Skip to content

Commit c0997c2

Browse files
committed
JS: Reveal issue with immutable.js test
Fixed in the next commit
1 parent 4473e6d commit c0997c2

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

javascript/ql/test/library-tests/frameworks/Immutable/immutable.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ sink(map2.get("b")); // OK - but still flagged [INCONSISTENCY]
1414

1515
const map3 = map2.set("d", source("d"));
1616
sink(map1.get("d")); // OK
17-
sink(map3.get("d")); // NOT OK
17+
sink(map3.get("d")); // NOT OK [INCONSISTENCY]
1818

1919

2020
sink(map3.toJS()["a"]); // NOT OK
@@ -34,7 +34,7 @@ List(["safe"]).push(source()).forEach(x => sink(x)); // NOT OK
3434

3535

3636
const map4 = OrderedMap({}).set("f", source());
37-
sink(map4.get("f")); // NOT OK
37+
sink(map4.get("f")); // NOT OK [INCONSISTENCY]
3838

3939
const map5 = Record({a: source(), b: null, c: null})({b: source()});
4040
sink(map5.get("a")); // NOT OK

javascript/ql/test/library-tests/frameworks/Immutable/tests.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
11
legacyDataFlowDifference
2+
| immutable.js:15:28:15:38 | source("d") | immutable.js:17:6:17:18 | map3.get("d") | only flow with OLD data flow library |
3+
| immutable.js:36:38:36:45 | source() | immutable.js:37:6:37:18 | map4.get("f") | only flow with OLD data flow library |
24
dataFlow
35
| immutable.js:1:16:1:26 | source("a") | immutable.js:2:6:2:13 | obj["a"] |
46
| immutable.js:1:16:1:26 | source("a") | immutable.js:11:6:11:18 | map1.get("a") |
57
| immutable.js:1:16:1:26 | source("a") | immutable.js:12:6:12:18 | map2.get("a") |
68
| immutable.js:1:16:1:26 | source("a") | immutable.js:20:6:20:21 | map3.toJS()["a"] |
79
| immutable.js:1:32:1:43 | source("b1") | immutable.js:8:6:8:18 | map1.get("b") |
810
| immutable.js:1:32:1:43 | source("b1") | immutable.js:13:6:13:18 | map2.get("b") |
9-
| immutable.js:15:28:15:38 | source("d") | immutable.js:17:6:17:18 | map3.get("d") |
1011
| immutable.js:22:19:22:29 | source("e") | immutable.js:22:6:22:40 | fromJS( ... et("e") |
1112
| immutable.js:24:18:24:25 | source() | immutable.js:25:22:25:22 | x |
1213
| immutable.js:29:25:29:32 | source() | immutable.js:29:53:29:53 | x |
1314
| immutable.js:31:7:31:14 | source() | immutable.js:31:75:31:75 | x |
1415
| immutable.js:33:21:33:28 | source() | immutable.js:33:49:33:49 | x |
15-
| immutable.js:36:38:36:45 | source() | immutable.js:37:6:37:18 | map4.get("f") |
1616
| immutable.js:39:25:39:32 | source() | immutable.js:40:6:40:18 | map5.get("a") |
1717
| immutable.js:39:58:39:65 | source() | immutable.js:41:6:41:18 | map5.get("b") |
1818
| immutable.js:44:40:44:47 | source() | immutable.js:45:6:45:18 | map6.get("a") |

0 commit comments

Comments
 (0)