-
Notifications
You must be signed in to change notification settings - Fork 1.9k
JS: Update test suite to use post-processed inline expectations #18670
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
116 commits
Select commit
Hold shift + click to select a range
79e2a75
JS: Allow more kinds of expectation comments
asgerf 7e5c24a
JS: Remove uses of old inline expectation test library
asgerf 9be041e
JS: Update OK-style comments to $-style
asgerf 426edd5
JS: Update output after line number change
asgerf d0ce53e
JS: Enable post-processing for all .qlref files
asgerf ac6547f
JS: Disable for comment-related alerts
asgerf 789a7bd
JS: Disable for test with alerts in a JSON file
asgerf 795c110
JS: Disable for SyntaxError
asgerf f5911c9
JS: Accept raw test output
asgerf 86932c5
JS: Move some alerts to their correct location
asgerf 0453ded
JS: Add query ID to some alerts
asgerf 10a7294
JS: Accept trivial test changes
asgerf 07a876b
JS: Accept some alerts at the SystemCommandExecution location
asgerf f395651
JS: Mark alert as MISSING
asgerf 1f3c496
JS: Accept some less obvious alerts
asgerf 68fae9d
JS: Accept alerts about newline replacement
asgerf e5bee19
JS: Accept a double-flagged line
asgerf e026b9e
JS: Mark regressions due to lack of local field steps
asgerf 2c46e10
JS: Mark an alert as missing
asgerf 426a871
JS: Remove incorrect Alert marker
asgerf 2877531
JS: Remove invalid syntax from test
asgerf a9b263f
JS: Remove incorrect alert expectation
asgerf 22c218d
JS: Mark a 'good' test as 'bad' and add Alert marker
asgerf b095fe2
JS: Fix some bugs in a test case
asgerf c051b4c
JS: Add spurious alert marker
asgerf cf33db7
JS: Fix the spurious flow
asgerf c593853
JS: Record some missing alerts in FileAccessToHttp
asgerf 319ee2c
JS: Track deep flow through qs.stringify
asgerf e745f42
JS: Remove alert expectation from step
asgerf d3de6d1
JS: Accept other changes to UnusedVariable
asgerf f72cd21
JS: Some more test changes in SpuriousArguments
asgerf 576dbcb
JS: Stop overriding entire module.exports object in test
asgerf b54ff3b
JS: Accept an alert
asgerf e91a046
JS: Mark a spurious alert
asgerf 51b4559
JS: Move an alert and add query ID
asgerf fc95702
JS: Accept some more alerts from CleartextStorage
asgerf 0f8e85f
JS: Accept alerts for InsufficientKeySize
asgerf 92c3939
JS: Accept InsecureRandomness alerts
asgerf 283b142
JS: Accept some ReDoS alerts
asgerf d298d87
JS: Accept some exponenital redos alerts in the polynomial redos test…
asgerf 51fb3da
JS: Accept ReDoS alerts in regexplib
asgerf 607b184
JS: Fix a bug in test case
asgerf 082e16b
JS: More Alert comments in ReDoS/tst.js based on variable naming
asgerf 266ac09
JS: Add query iDs
asgerf 1fcebce
JS: Move some ReDoS alerts
asgerf a0f8e28
JS: Accept a fixed FN
asgerf cded757
JS: Add a query ID
asgerf cea5337
JS: Accept alerts for missing-x-frame-options
asgerf 260c66b
JS: Mark a spurious alert in missing-x-frame-options
asgerf 48760d6
JS: Accept alerts for HardcodedDataInterpretedAsCode
asgerf 3f7f74b
JS: Accept alerts for DecompressionBomb
asgerf cd788bc
JS: Mark what seems to be missing alerts for fflate
asgerf 0f23c33
JS: Fix a comment
asgerf 1ee93cf
JS: Manually fix two comments in JSX
asgerf b734a3d
JS: Fix a test case bug due to a typo
asgerf 7c2394f
JS: Accept some alerts
asgerf 38be524
JS: Temporarily accept a spurious alert
asgerf 2965964
JS: Fix barrier guards for ServerSideUrlRedirect
asgerf 4ca49fb
JS: Change note about guards in server-side url redirect
asgerf dbc079c
JS: Accept a spurious alert
asgerf 360c765
JS: Accept some suboptimal alert locations
asgerf 69b2d19
JS: Move an alert and add RelatedLocations
asgerf 7fa63fa
JS: Update alerts in HardcodedCredentials test
asgerf 9ef5a97
JS: Accept alerts in HardcodedCredentials and add Sink tags
asgerf 2bed3a4
JS: Mark some missing alerts in HardcodedCredentials
asgerf a1796bd
JS: Accept some new alerts in HardcodedCredentials
asgerf 7ed5398
JS: Accept some different-kind comparison-bypass alerts
asgerf c79d355
JS: Update alerts in example_bypass.js
asgerf db98dc6
JS: Move some alerts and add query IDs
asgerf 3a535db
JS: Accept another alert
asgerf 9f87446
JS: Remove a fixed spurious alert
asgerf 2d1aa3e
JS: Accept missing alert and clarify reason
asgerf 49274d5
JS: Accept an alert
asgerf 9760965
JS: Accept an alert
asgerf 764eb98
JS: Move two alerts and add query ID
asgerf 4d7cbe6
JS: Accept to web socket-based SSRF alerts
asgerf e634b31
JS: Accept some UselessConditional alerts
asgerf 507a091
JS: More alert updates in UselessConditional
asgerf 7623ebb
JS: Accept changes in UseOfReturnlessFunction
asgerf 6cf1334
Fix comment style in UnboundEventHandlerReceiver
asgerf 2b33ed3
JS: Remove incorrect alert marker
asgerf a1c13f0
JS: Remove alert marker that's reported on another line
asgerf dc28bb5
JS: Fix alert location and use RelatedLocation in InsecureUrlWhitelist
asgerf 6059994
JS: Accept more results in SpuriousArguments
asgerf 87ed86e
JS: Update UnusedOrUndefinedStateProperty
asgerf 8ef51c4
JS: Fix wrong expectation in UnusedOrUndefinedStateProperty
asgerf aade1e8
JS: Add a related location in UnusedOrUndefinedStateProperty
asgerf 0496de6
JS: Accept alerts in UselessCharacterEscape
asgerf bb67a0e
JS: Remove outdated comment
asgerf 87518ba
JS: Update tainted-sendFile.js
asgerf b4ac2f7
JS: Add a query ID
asgerf fd6a9c6
JS: Accept an alert
asgerf 19cada3
JS: Migrate a new file from OK-style comments
asgerf 64d39da
JS: Accept Sources/Sink tags
asgerf 7bd01bf
JS: Bulk update in UnneededDefensiveProgramming test
asgerf c67c585
Disable for more queries with alerts in JSON
asgerf 33602ee
JS: Convert some comments to JSX
asgerf 193b26e
JS: Add query IDs
asgerf 2a194a5
raw test output
asgerf 24c9b2e
Update javascript/ql/test/query-tests/Security/CWE-078/UselessUseOfCa…
asgerf 017f458
Update javascript/ql/test/query-tests/Security/CWE-078/UselessUseOfCa…
asgerf dad4838
Update javascript/ql/test/query-tests/Security/CWE-079/DomBasedXss/ts…
asgerf 21d42bc
Update javascript/ql/test/query-tests/Security/CWE-079/ExceptionXss/e…
asgerf 92dfdc8
Update javascript/ql/test/query-tests/Security/CWE-116/IncompleteSani…
asgerf 8ee5b23
Update javascript/ql/test/query-tests/Security/CWE-730/server-crash.js
asgerf 122f68e
Update javascript/ql/test/query-tests/Security/CWE-400/ReDoS/polynomi…
asgerf f7532c0
JS: Remove blank line
asgerf b9dd594
JS: Remove blank lines
asgerf 6fe3a36
JS: Remove blank line and add trailing newline to file
asgerf 0df893e
JS: Remove blank lines and add trailing newline
asgerf c88eac4
JS: Remove stray $ Alert comment inside a doc comment
asgerf 6a47678
JS: Fix broken alert comment in HeterogenousComparison
asgerf 75ed0d0
JS: Remove duplicate '$ Alert' in libxml test
asgerf 0f201d2
JS: Line number changes in redos test case
asgerf e8c5e4d
Merge branch 'main' into js/test-suite
asgerf 6499e54
JS: Restore line lost in merge
asgerf File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.