JS: Add some more missing cases

asgerf · asgerf · commit 87dfaa6f0ef5 · 2025-03-13T13:24:05.000+01:00
diff --git a/javascript/ql/test/library-tests/frameworks/data/test.js b/javascript/ql/test/library-tests/frameworks/data/test.js
@@ -282,4 +282,7 @@ function dangerConstant() {
 function arraySource() {
   const source = testlib.getSourceArray();
   sink(source[0]); // NOT OK
+  sink(source.pop()); // NOT OK [INCONSISTENCY]
+  source.forEach(e => sink(e)); // // NOT OK [INCONSISTENCY]
+  source.map(e => sink(e)); // // NOT OK [INCONSISTENCY]
 }

Original file line number	Diff line number	Diff line change
`@@ -282,4 +282,7 @@ function dangerConstant() {`
`282`	`282`	`function arraySource() {`
`283`	`283`	`const source = testlib.getSourceArray();`
`284`	`284`	`sink(source[0]); // NOT OK`
	`285`	`+ sink(source.pop()); // NOT OK [INCONSISTENCY]`
	`286`	`+ source.forEach(e => sink(e)); // // NOT OK [INCONSISTENCY]`
	`287`	`+ source.map(e => sink(e)); // // NOT OK [INCONSISTENCY]`
`285`	`288`	`}`