Skip to content

Conversation

@cklin
Copy link
Contributor

@cklin cklin commented Apr 24, 2025

This PR adds expected files for diff-informed testing. These expected files describe how diff-informed queries produce alerts that are not completely in accordance with the given diff ranges.

This commit adds expected files for diff-informed testing. These
expected files describe how diff-informed queries produce alerts that
are not completely in accordance with the given diff ranges.
@github-actions github-actions bot added the JS label Apr 24, 2025
@cklin cklin marked this pull request as ready for review April 24, 2025 21:27
Copilot AI review requested due to automatic review settings April 24, 2025 21:27
@cklin cklin requested a review from a team as a code owner April 24, 2025 21:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 6 out of 26 changed files in this pull request and generated no comments.

Files not reviewed (20)
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/47d3c.expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/ad592.expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/main.js/(16,21)-(16,35).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/main.js/(17,48)-(17,50).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/main.js/16.expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-079/UnsafeHtmlConstruction/DIFF-INFORMED/UnsafeHtmlConstruction/main.js/17.expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(15,28)-(15,35).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(15,41)-(15,43).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(17,11)-(17,12).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(17,5)-(17,6).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(25,37)-(25,56).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(25,63)-(25,64).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(30,19)-(30,22).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(30,23)-(30,24).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(36,18)-(36,19).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(36,35)-(36,36).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(37,18)-(37,19).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(37,35)-(37,36).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(38,18)-(38,19).expected: Language not supported
  • javascript/ql/test/query-tests/Security/CWE-400/ReDoS/DIFF-INFORMED/PolynomialReDoS/polynomial-redos.js/(38,34)-(38,35).expected: Language not supported

@cklin cklin closed this Apr 24, 2025
@cklin cklin deleted the cklin/check-diff-informed-js branch August 12, 2025 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants