Skip to content

Commit d83f5a9

Browse files
committed
JS: Update StringConcatenation tests after handling 0-arg join calls
1 parent a3e56de commit d83f5a9

File tree

3 files changed

+26
-0
lines changed

3 files changed

+26
-0
lines changed

javascript/ql/test/library-tests/StringConcatenation/ClassContainsTwo.expected

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,10 @@
3636
| tst.js:53:10:53:34 | `one ${ ... three` |
3737
| tst.js:53:19:53:23 | two |
3838
| tst.js:71:14:71:18 | "two" |
39+
| tst.js:77:15:77:37 | ["one", ... three"] |
3940
| tst.js:77:23:77:27 | "two" |
41+
| tst.js:79:12:79:16 | array |
42+
| tst.js:79:12:79:23 | array.join() |
4043
| tst.js:87:5:87:14 | x += 'two' |
4144
| tst.js:87:10:87:14 | 'two' |
4245
| tst.js:89:3:89:3 | x |

javascript/ql/test/library-tests/StringConcatenation/ContainsTwo.expected

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,10 @@
3636
| tst.js:53:10:53:34 | `one ${ ... three` |
3737
| tst.js:53:19:53:23 | two |
3838
| tst.js:71:14:71:18 | "two" |
39+
| tst.js:77:15:77:37 | ["one", ... three"] |
3940
| tst.js:77:23:77:27 | "two" |
41+
| tst.js:79:12:79:16 | array |
42+
| tst.js:79:12:79:23 | array.join() |
4043
| tst.js:87:5:87:14 | x += 'two' |
4144
| tst.js:87:10:87:14 | 'two' |
4245
| tst.js:89:3:89:3 | x |

javascript/ql/test/library-tests/StringConcatenation/StringOps.expected

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@ concatenation
4141
| tst.js:49:10:49:20 | xs.join("") |
4242
| tst.js:53:10:53:34 | `one ${ ... three` |
4343
| tst.js:61:10:61:34 | `first ... } last` |
44+
| tst.js:77:15:77:37 | ["one", ... three"] |
45+
| tst.js:79:12:79:23 | array.join() |
4446
| tst.js:87:5:87:14 | x |
4547
| tst.js:87:5:87:14 | x += 'two' |
4648
| tst.js:89:3:89:14 | x |
@@ -126,6 +128,10 @@ concatenationOperand
126128
| tst.js:61:23:61:23 | x |
127129
| tst.js:61:27:61:27 | x |
128130
| tst.js:61:29:61:33 | last |
131+
| tst.js:77:15:77:37 | ["one", ... three"] |
132+
| tst.js:77:16:77:20 | "one" |
133+
| tst.js:77:23:77:27 | "two" |
134+
| tst.js:77:30:77:36 | "three" |
129135
| tst.js:87:5:87:5 | x |
130136
| tst.js:87:10:87:14 | 'two' |
131137
| tst.js:89:3:89:3 | x |
@@ -210,6 +216,9 @@ concatenationLeaf
210216
| tst.js:61:23:61:23 | x |
211217
| tst.js:61:27:61:27 | x |
212218
| tst.js:61:29:61:33 | last |
219+
| tst.js:77:16:77:20 | "one" |
220+
| tst.js:77:23:77:27 | "two" |
221+
| tst.js:77:30:77:36 | "three" |
213222
| tst.js:87:5:87:5 | x |
214223
| tst.js:87:10:87:14 | 'two' |
215224
| tst.js:89:3:89:3 | x |
@@ -334,6 +343,11 @@ concatenationNode
334343
| tst.js:61:23:61:23 | x |
335344
| tst.js:61:27:61:27 | x |
336345
| tst.js:61:29:61:33 | last |
346+
| tst.js:77:15:77:37 | ["one", ... three"] |
347+
| tst.js:77:16:77:20 | "one" |
348+
| tst.js:77:23:77:27 | "two" |
349+
| tst.js:77:30:77:36 | "three" |
350+
| tst.js:79:12:79:23 | array.join() |
337351
| tst.js:87:5:87:5 | x |
338352
| tst.js:87:5:87:14 | x |
339353
| tst.js:87:5:87:14 | x += 'two' |
@@ -454,6 +468,10 @@ operand
454468
| tst.js:61:10:61:34 | `first ... } last` | 2 | tst.js:61:23:61:23 | x |
455469
| tst.js:61:10:61:34 | `first ... } last` | 3 | tst.js:61:27:61:27 | x |
456470
| tst.js:61:10:61:34 | `first ... } last` | 4 | tst.js:61:29:61:33 | last |
471+
| tst.js:77:15:77:37 | ["one", ... three"] | 0 | tst.js:77:16:77:20 | "one" |
472+
| tst.js:77:15:77:37 | ["one", ... three"] | 1 | tst.js:77:23:77:27 | "two" |
473+
| tst.js:77:15:77:37 | ["one", ... three"] | 2 | tst.js:77:30:77:36 | "three" |
474+
| tst.js:79:12:79:23 | array.join() | 0 | tst.js:77:15:77:37 | ["one", ... three"] |
457475
| tst.js:87:5:87:14 | x | 0 | tst.js:87:5:87:5 | x |
458476
| tst.js:87:5:87:14 | x | 1 | tst.js:87:10:87:14 | 'two' |
459477
| tst.js:87:5:87:14 | x += 'two' | 0 | tst.js:87:5:87:5 | x |
@@ -518,6 +536,8 @@ nextLeaf
518536
| tst.js:61:19:61:19 | x | tst.js:61:23:61:23 | x |
519537
| tst.js:61:23:61:23 | x | tst.js:61:27:61:27 | x |
520538
| tst.js:61:27:61:27 | x | tst.js:61:29:61:33 | last |
539+
| tst.js:77:16:77:20 | "one" | tst.js:77:23:77:27 | "two" |
540+
| tst.js:77:23:77:27 | "two" | tst.js:77:30:77:36 | "three" |
521541
| tst.js:87:5:87:5 | x | tst.js:87:10:87:14 | 'two' |
522542
| tst.js:89:3:89:3 | x | tst.js:89:8:89:14 | 'three' |
523543
| tst.js:95:7:95:7 | x | tst.js:95:16:95:20 | 'two' |

0 commit comments

Comments
 (0)