Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit a66d1c0

Browse files
committed
fixes test errors
1 parent d4f2172 commit a66d1c0

2 files changed

Lines changed: 50 additions & 43 deletions

File tree

javascript/ql/test/library-tests/Expr/nullSensitiveContexts.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ x -= y
2626
x *= y
2727
x /= y
2828
x %= y
29-
x , y = p
29+
[x , y] = p
3030
//[1,2,...xs]
3131
x & y
3232
x | y

javascript/ql/test/library-tests/Expr/tests.expected

Lines changed: 49 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -277,14 +277,15 @@ test_getParent
277277
| nullSensitiveContexts.js:27:1:27:1 | x | nullSensitiveContexts.js:27:1:27:6 | x /= y |
278278
| nullSensitiveContexts.js:27:1:27:6 | x /= y | nullSensitiveContexts.js:27:1:27:6 | x /= y |
279279
| nullSensitiveContexts.js:27:6:27:6 | y | nullSensitiveContexts.js:27:1:27:6 | x /= y |
280-
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:28:1:28:6 | x %= y |
281-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | nullSensitiveContexts.js:28:1:28:6 | x %= y |
282-
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:28:1:28:6 | x %= y |
283-
| nullSensitiveContexts.js:29:1:29:1 | x | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
284-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
285-
| nullSensitiveContexts.js:29:5:29:5 | y | nullSensitiveContexts.js:29:5:29:9 | y = p |
286-
| nullSensitiveContexts.js:29:5:29:9 | y = p | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
287-
| nullSensitiveContexts.js:29:9:29:9 | p | nullSensitiveContexts.js:29:5:29:9 | y = p |
280+
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
281+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
282+
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] |
283+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p |
284+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
285+
| nullSensitiveContexts.js:29:2:29:2 | x | nullSensitiveContexts.js:29:2:29:6 | x , y |
286+
| nullSensitiveContexts.js:29:2:29:6 | x , y | nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] |
287+
| nullSensitiveContexts.js:29:6:29:6 | y | nullSensitiveContexts.js:29:2:29:6 | x , y |
288+
| nullSensitiveContexts.js:29:11:29:11 | p | nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p |
288289
| nullSensitiveContexts.js:31:1:31:1 | x | nullSensitiveContexts.js:31:1:31:5 | x & y |
289290
| nullSensitiveContexts.js:31:1:31:5 | x & y | nullSensitiveContexts.js:31:1:31:5 | x & y |
290291
| nullSensitiveContexts.js:31:5:31:5 | y | nullSensitiveContexts.js:31:1:31:5 | x & y |
@@ -731,13 +732,14 @@ test_getTopLevel
731732
| nullSensitiveContexts.js:27:1:27:6 | x /= y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
732733
| nullSensitiveContexts.js:27:6:27:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
733734
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
734-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
735+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
735736
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
736-
| nullSensitiveContexts.js:29:1:29:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
737-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
738-
| nullSensitiveContexts.js:29:5:29:5 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
739-
| nullSensitiveContexts.js:29:5:29:9 | y = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
740-
| nullSensitiveContexts.js:29:9:29:9 | p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
737+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
738+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
739+
| nullSensitiveContexts.js:29:2:29:2 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
740+
| nullSensitiveContexts.js:29:2:29:6 | x , y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
741+
| nullSensitiveContexts.js:29:6:29:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
742+
| nullSensitiveContexts.js:29:11:29:11 | p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
741743
| nullSensitiveContexts.js:31:1:31:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
742744
| nullSensitiveContexts.js:31:1:31:5 | x & y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
743745
| nullSensitiveContexts.js:31:5:31:5 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
@@ -1088,12 +1090,14 @@ test_getChild
10881090
| nullSensitiveContexts.js:26:1:26:6 | x *= y | 1 | nullSensitiveContexts.js:26:6:26:6 | y |
10891091
| nullSensitiveContexts.js:27:1:27:6 | x /= y | 0 | nullSensitiveContexts.js:27:1:27:1 | x |
10901092
| nullSensitiveContexts.js:27:1:27:6 | x /= y | 1 | nullSensitiveContexts.js:27:6:27:6 | y |
1091-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | 0 | nullSensitiveContexts.js:28:1:28:1 | x |
1092-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | 1 | nullSensitiveContexts.js:28:6:28:6 | y |
1093-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | 0 | nullSensitiveContexts.js:29:1:29:1 | x |
1094-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | 1 | nullSensitiveContexts.js:29:5:29:9 | y = p |
1095-
| nullSensitiveContexts.js:29:5:29:9 | y = p | 0 | nullSensitiveContexts.js:29:5:29:5 | y |
1096-
| nullSensitiveContexts.js:29:5:29:9 | y = p | 1 | nullSensitiveContexts.js:29:9:29:9 | p |
1093+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | 0 | nullSensitiveContexts.js:28:1:28:1 | x |
1094+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | 1 | nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p |
1095+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | 0 | nullSensitiveContexts.js:28:6:28:6 | y |
1096+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | 1 | nullSensitiveContexts.js:29:2:29:6 | x , y |
1097+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | 0 | nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] |
1098+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | 1 | nullSensitiveContexts.js:29:11:29:11 | p |
1099+
| nullSensitiveContexts.js:29:2:29:6 | x , y | 0 | nullSensitiveContexts.js:29:2:29:2 | x |
1100+
| nullSensitiveContexts.js:29:2:29:6 | x , y | 1 | nullSensitiveContexts.js:29:6:29:6 | y |
10971101
| nullSensitiveContexts.js:31:1:31:5 | x & y | 0 | nullSensitiveContexts.js:31:1:31:1 | x |
10981102
| nullSensitiveContexts.js:31:1:31:5 | x & y | 1 | nullSensitiveContexts.js:31:5:31:5 | y |
10991103
| nullSensitiveContexts.js:32:1:32:5 | x \| y | 0 | nullSensitiveContexts.js:32:1:32:1 | x |
@@ -1415,9 +1419,11 @@ test_isPure
14151419
| nullSensitiveContexts.js:27:6:27:6 | y |
14161420
| nullSensitiveContexts.js:28:1:28:1 | x |
14171421
| nullSensitiveContexts.js:28:6:28:6 | y |
1418-
| nullSensitiveContexts.js:29:1:29:1 | x |
1419-
| nullSensitiveContexts.js:29:5:29:5 | y |
1420-
| nullSensitiveContexts.js:29:9:29:9 | p |
1422+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] |
1423+
| nullSensitiveContexts.js:29:2:29:2 | x |
1424+
| nullSensitiveContexts.js:29:2:29:6 | x , y |
1425+
| nullSensitiveContexts.js:29:6:29:6 | y |
1426+
| nullSensitiveContexts.js:29:11:29:11 | p |
14211427
| nullSensitiveContexts.js:31:1:31:1 | x |
14221428
| nullSensitiveContexts.js:31:1:31:5 | x & y |
14231429
| nullSensitiveContexts.js:31:5:31:5 | y |
@@ -1934,13 +1940,14 @@ test_getContainer
19341940
| nullSensitiveContexts.js:27:1:27:6 | x /= y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19351941
| nullSensitiveContexts.js:27:6:27:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19361942
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1937-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1943+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19381944
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1939-
| nullSensitiveContexts.js:29:1:29:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1940-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1941-
| nullSensitiveContexts.js:29:5:29:5 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1942-
| nullSensitiveContexts.js:29:5:29:9 | y = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1943-
| nullSensitiveContexts.js:29:9:29:9 | p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1945+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1946+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1947+
| nullSensitiveContexts.js:29:2:29:2 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1948+
| nullSensitiveContexts.js:29:2:29:6 | x , y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1949+
| nullSensitiveContexts.js:29:6:29:6 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
1950+
| nullSensitiveContexts.js:29:11:29:11 | p | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19441951
| nullSensitiveContexts.js:31:1:31:1 | x | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19451952
| nullSensitiveContexts.js:31:1:31:5 | x & y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
19461953
| nullSensitiveContexts.js:31:5:31:5 | y | nullSensitiveContexts.js:1:1:61:0 | <toplevel> |
@@ -2357,14 +2364,15 @@ test_getEnclosingStmt
23572364
| nullSensitiveContexts.js:27:1:27:1 | x | nullSensitiveContexts.js:27:1:27:6 | x /= y |
23582365
| nullSensitiveContexts.js:27:1:27:6 | x /= y | nullSensitiveContexts.js:27:1:27:6 | x /= y |
23592366
| nullSensitiveContexts.js:27:6:27:6 | y | nullSensitiveContexts.js:27:1:27:6 | x /= y |
2360-
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:28:1:28:6 | x %= y |
2361-
| nullSensitiveContexts.js:28:1:28:6 | x %= y | nullSensitiveContexts.js:28:1:28:6 | x %= y |
2362-
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:28:1:28:6 | x %= y |
2363-
| nullSensitiveContexts.js:29:1:29:1 | x | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
2364-
| nullSensitiveContexts.js:29:1:29:9 | x , y = p | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
2365-
| nullSensitiveContexts.js:29:5:29:5 | y | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
2366-
| nullSensitiveContexts.js:29:5:29:9 | y = p | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
2367-
| nullSensitiveContexts.js:29:9:29:9 | p | nullSensitiveContexts.js:29:1:29:9 | x , y = p |
2367+
| nullSensitiveContexts.js:28:1:28:1 | x | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2368+
| nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2369+
| nullSensitiveContexts.js:28:6:28:6 | y | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2370+
| nullSensitiveContexts.js:28:6:29:7 | y\\n[x , y] | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2371+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2372+
| nullSensitiveContexts.js:29:2:29:2 | x | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2373+
| nullSensitiveContexts.js:29:2:29:6 | x , y | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2374+
| nullSensitiveContexts.js:29:6:29:6 | y | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
2375+
| nullSensitiveContexts.js:29:11:29:11 | p | nullSensitiveContexts.js:28:1:29:11 | x %= y\\n[x , y] = p |
23682376
| nullSensitiveContexts.js:31:1:31:1 | x | nullSensitiveContexts.js:31:1:31:5 | x & y |
23692377
| nullSensitiveContexts.js:31:1:31:5 | x & y | nullSensitiveContexts.js:31:1:31:5 | x & y |
23702378
| nullSensitiveContexts.js:31:5:31:5 | y | nullSensitiveContexts.js:31:1:31:5 | x & y |
@@ -2531,7 +2539,6 @@ test_getEnclosingStmt
25312539
| update.js:4:1:4:1 | b | update.js:4:1:4:4 | b--; |
25322540
| update.js:4:1:4:3 | b-- | update.js:4:1:4:4 | b--; |
25332541
test_inNullSensitiveContext
2534-
| assignment.js:1:5:1:6 | 23 |
25352542
| assignment.js:2:1:2:1 | a |
25362543
| assignment.js:2:6:2:7 | 19 |
25372544
| assignment.js:3:1:3:1 | a |
@@ -2607,6 +2614,7 @@ test_inNullSensitiveContext
26072614
| es2015.js:28:19:28:19 | x |
26082615
| es2015.js:28:21:28:22 | 19 |
26092616
| functions.js:4:3:6:4 | (functi ... f;\\n\\t\\t}) |
2617+
| functions.js:4:4:6:3 | functio ... _f;\\n\\t\\t} |
26102618
| legacyletexpr.js:1:1:1:7 | console |
26112619
| legacyletexpr.js:1:1:1:11 | console.log |
26122620
| legacyletexpr.js:1:34:1:34 | x |
@@ -2621,7 +2629,6 @@ test_inNullSensitiveContext
26212629
| nullSensitiveContexts.js:9:5:9:7 | Foo |
26222630
| nullSensitiveContexts.js:10:5:10:7 | Foo |
26232631
| nullSensitiveContexts.js:11:1:11:3 | foo |
2624-
| nullSensitiveContexts.js:11:11:11:11 | 5 |
26252632
| nullSensitiveContexts.js:12:1:12:3 | foo |
26262633
| nullSensitiveContexts.js:13:1:13:1 | x |
26272634
| nullSensitiveContexts.js:13:5:13:5 | y |
@@ -2651,7 +2658,9 @@ test_inNullSensitiveContext
26512658
| nullSensitiveContexts.js:27:6:27:6 | y |
26522659
| nullSensitiveContexts.js:28:1:28:1 | x |
26532660
| nullSensitiveContexts.js:28:6:28:6 | y |
2654-
| nullSensitiveContexts.js:29:9:29:9 | p |
2661+
| nullSensitiveContexts.js:28:6:29:11 | y\\n[x , y] = p |
2662+
| nullSensitiveContexts.js:29:2:29:6 | x , y |
2663+
| nullSensitiveContexts.js:29:6:29:6 | y |
26552664
| nullSensitiveContexts.js:31:1:31:1 | x |
26562665
| nullSensitiveContexts.js:31:5:31:5 | y |
26572666
| nullSensitiveContexts.js:32:1:32:1 | x |
@@ -2691,10 +2700,8 @@ test_inNullSensitiveContext
26912700
| surrogates.js:2:1:2:8 | /\\uD800/ |
26922701
| surrogates.js:2:1:2:13 | /\\uD800/.test |
26932702
| unary.js:1:2:1:3 | 23 |
2694-
| unary.js:2:2:2:3 | 42 |
26952703
| unary.js:4:2:4:2 | 2 |
26962704
| unary.js:5:8:5:13 | Object |
2697-
| unary.js:6:5:6:7 | (0) |
26982705
| unary.js:7:8:7:13 | Object |
26992706
| unary.js:7:8:7:23 | Object.prototype |
27002707
| update.js:1:3:1:3 | a |

0 commit comments

Comments
 (0)