Skip to content

Commit 3d0cfdd

Browse files
asgerfasgerf
committed
Raw test update
1 parent d35f3d0 commit 3d0cfdd

File tree

8 files changed

+113
-37
lines changed

8 files changed

+113
-37
lines changed

‎javascript/ql/test/query-tests/AngularJS/InsecureUrlWhitelist/InsecureUrlWhitelist.expected‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,18 @@
1+
#select
12
| tst.js:3:9:23:10 | $sceDel ... ]) | $@ is not a secure whitelist entry, because the TLD '*' is insecurely specified. | tst.js:8:13:8:31 | "https://example.*" | "https://example.*" |
23
| tst.js:3:9:23:10 | $sceDel ... ]) | $@ is not a secure whitelist entry, because the domain '**.example.com' is insecurely specified. | tst.js:6:13:6:38 | "https: ... .com/*" | "https: ... .com/*" |
34
| tst.js:3:9:23:10 | $sceDel ... ]) | $@ is not a secure whitelist entry, because the domain 'example.**' is insecurely specified. | tst.js:7:13:7:32 | "https://example.**" | "https://example.**" |
45
| tst.js:3:9:23:10 | $sceDel ... ]) | $@ is not a secure whitelist entry, because the scheme '*' is insecurely specified. | tst.js:5:13:5:31 | "*://example.org/*" | "*://example.org/*" |
56
| tst.js:3:9:23:10 | $sceDel ... ]) | $@ is not a secure whitelist entry, because the scheme '**' is insecurely specified. | tst.js:4:13:4:32 | "**://example.com/*" | "**://example.com/*" |
7+
testFailures
8+
| tst.js:3:9:23:10 | $@ is not a secure whitelist entry, because the TLD '*' is insecurely specified. | Unexpected result: Alert |
9+
| tst.js:3:9:23:10 | $@ is not a secure whitelist entry, because the domain '**.example.com' is insecurely specified. | Unexpected result: Alert |
10+
| tst.js:3:9:23:10 | $@ is not a secure whitelist entry, because the domain 'example.**' is insecurely specified. | Unexpected result: Alert |
11+
| tst.js:3:9:23:10 | $@ is not a secure whitelist entry, because the scheme '*' is insecurely specified. | Unexpected result: Alert |
12+
| tst.js:3:9:23:10 | $@ is not a secure whitelist entry, because the scheme '**' is insecurely specified. | Unexpected result: Alert |
13+
| tst.js:4:35:4:138 | // $ Al ... .org/a) | Missing result: Alert |
14+
| tst.js:5:34:5:154 | // $ Al ... "//"!) | Missing result: Alert |
15+
| tst.js:6:41:6:103 | // $ Al ... e.com/a | Missing result: Alert |
16+
| tst.js:7:35:7:112 | // $ Al ... vil.com | Missing result: Alert |
17+
| tst.js:8:34:8:84 | // $ Al ... ctedTLD | Missing result: Alert |
18+
| tst.js:22:41:22:108 | // $ Al ... caped)) | Missing result: Alert |

‎javascript/ql/test/query-tests/Electron/NodeIntegration/EnablingNodeIntegration.expected‎

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,16 @@
1+
#select
12
| EnablingNodeIntegration.js:5:28:11:9 | {\\n ... } | The `nodeIntegrationInWorker` feature has been enabled. |
23
| EnablingNodeIntegration.js:5:28:11:9 | {\\n ... } | The `nodeIntegration` feature has been enabled. |
34
| EnablingNodeIntegration.js:15:22:20:9 | {\\n ... } | The `nodeIntegration` feature is enabled by default. |
45
| EnablingNodeIntegration.js:23:16:27:9 | { // $ ... } | The `nodeIntegration` feature is enabled by default. |
56
| EnablingNodeIntegration.js:49:74:49:96 | {nodeIn ... : true} | The `nodeIntegration` feature has been enabled. |
7+
testFailures
8+
| EnablingNodeIntegration.js:4:22:4:46 | // $ Al ... enabled | Missing result: Alert |
9+
| EnablingNodeIntegration.js:5:28:11:9 | The `nodeIntegrationInWorker` feature has been enabled. | Unexpected result: Alert |
10+
| EnablingNodeIntegration.js:5:28:11:9 | The `nodeIntegration` feature has been enabled. | Unexpected result: Alert |
11+
| EnablingNodeIntegration.js:14:23:14:68 | // $ Al ... enabled | Missing result: Alert |
12+
| EnablingNodeIntegration.js:15:22:20:9 | The `nodeIntegration` feature is enabled by default. | Unexpected result: Alert |
13+
| EnablingNodeIntegration.js:23:16:27:9 | The `nodeIntegration` feature is enabled by default. | Unexpected result: Alert |
14+
| EnablingNodeIntegration.js:23:18:23:48 | // $ Al ... enabled | Missing result: Alert |
15+
| EnablingNodeIntegration.js:29:24:29:54 | // $ Al ... enabled | Missing result: Alert |
16+
| EnablingNodeIntegration.js:37:23:37:54 | // $ Al ... isabled | Missing result: Alert |

‎javascript/ql/test/query-tests/Expressions/BitwiseSignCheck/BitwiseSignCheck.expected‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
#select
21
| tst.js:2:9:2:24 | (x & (1<<n)) > 0 | Potentially unsafe sign check of a bitwise operation. |
32
| tst.js:13:13:13:25 | (x >>> 0) > 0 | Potentially unsafe sign check of a bitwise operation. |
43
| tst.js:21:1:21:21 | (x & 0x ... 00) > 0 | Potentially unsafe sign check of a bitwise operation. |
5-
testFailures
6-
| tst.js:13:13:13:25 | Potentially unsafe sign check of a bitwise operation. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Expressions/SuspiciousPropAccess/SuspiciousPropAccess.expected‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
1-
#select
21
| SuspiciousPropAccess.js:4:10:4:21 | result.value | The base expression of this property access is always undefined. |
32
| optional-chaining.js:3:5:3:7 | a.p | The base expression of this property access is always null. |
43
| optional-chaining.js:7:5:7:7 | b.p | The base expression of this property access is always undefined. |
54
| tst2.ts:19:3:19:5 | x.p | The base expression of this property access is always undefined. |
65
| tst.js:32:32:32:38 | a(1)[0] | The base expression of this property access is always null. |
76
| typeassertion.ts:14:3:14:9 | z.field | The base expression of this property access is always null. |
8-
testFailures
9-
| typeoftype.ts:3:15:3:70 | // $ MI ... karound | Missing result: Alert |

‎javascript/ql/test/query-tests/Expressions/UnboundEventHandlerReceiver/UnboundEventHandlerReceiver.expected‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
#select
21
| tst.js:27:18:27:40 | onClick ... bound1} | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | tst.js:56:9:56:12 | this | this | tst.js:55:5:57:5 | unbound ... ;\\n } | unbound1 |
32
| tst.js:28:18:28:40 | onClick ... bound2} | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | tst.js:60:15:60:18 | this | this | tst.js:59:5:61:5 | unbound ... ;\\n } | unbound2 |
43
| tst.js:29:18:29:35 | onClick={unbound3} | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | tst.js:64:15:64:18 | this | this | tst.js:63:5:65:5 | unbound ... ;\\n } | unbound3 |
5-
testFailures
6-
| tst.js:27:18:27:40 | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | Unexpected result: Alert |
7-
| tst.js:28:18:28:40 | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | Unexpected result: Alert |
8-
| tst.js:29:18:29:35 | The receiver of this event handler call is unbound, `$@` will be `undefined` in the call to $@. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Expressions/UnneededDefensiveProgramming/UnneededDefensiveProgramming.expected‎

Lines changed: 76 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,79 @@
7676
| tst.js:177:2:177:2 | u | This guard always evaluates to false. |
7777
| tst.js:178:2:178:2 | u | This guard always evaluates to false. |
7878
testFailures
79-
| global-module-definition.js:10:30:10:39 | // $ Alert | Missing result: Alert |
80-
| tst.js:13:19:13:28 | // $ Alert | Missing result: Alert |
81-
| tst.js:14:19:14:28 | // $ Alert | Missing result: Alert |
82-
| tst.js:15:19:15:28 | // $ Alert | Missing result: Alert |
83-
| tst.js:165:26:165:35 | // $ Alert | Missing result: Alert |
79+
| module-environment-detection.js:23:8:23:36 | This guard always evaluates to true. | Unexpected result: Alert |
80+
| regression.js:9:9:9:12 | This guard always evaluates to true. | Unexpected result: Alert |
81+
| regression.js:13:25:13:40 | This guard always evaluates to true. | Unexpected result: Alert |
82+
| tst.js:18:5:18:5 | This guard always evaluates to false. | Fixed missing result: Alert |
83+
| tst.js:19:5:19:5 | This guard always evaluates to false. | Fixed missing result: Alert |
84+
| tst.js:20:5:20:5 | This guard always evaluates to true. | Fixed missing result: Alert |
85+
| tst.js:23:5:23:5 | This guard always evaluates to false. | Fixed missing result: Alert |
86+
| tst.js:24:5:24:5 | This guard always evaluates to false. | Fixed missing result: Alert |
87+
| tst.js:25:5:25:5 | This guard always evaluates to true. | Fixed missing result: Alert |
88+
| tst.js:28:5:28:6 | This guard always evaluates to true. | Fixed missing result: Alert |
89+
| tst.js:29:5:29:6 | This guard always evaluates to true. | Fixed missing result: Alert |
90+
| tst.js:30:5:30:6 | This guard always evaluates to false. | Fixed missing result: Alert |
91+
| tst.js:33:5:33:7 | This guard always evaluates to false. | Fixed missing result: Alert |
92+
| tst.js:34:5:34:7 | This guard always evaluates to false. | Fixed missing result: Alert |
93+
| tst.js:35:5:35:7 | This guard always evaluates to true. | Fixed missing result: Alert |
94+
| tst.js:38:5:38:18 | This guard always evaluates to false. | Fixed missing result: Alert |
95+
| tst.js:39:5:39:18 | This guard always evaluates to false. | Fixed missing result: Alert |
96+
| tst.js:40:5:40:18 | This guard always evaluates to true. | Fixed missing result: Alert |
97+
| tst.js:43:5:43:18 | This guard always evaluates to true. | Fixed missing result: Alert |
98+
| tst.js:44:5:44:18 | This guard always evaluates to true. | Fixed missing result: Alert |
99+
| tst.js:45:5:45:18 | This guard always evaluates to false. | Fixed missing result: Alert |
100+
| tst.js:48:5:48:19 | This guard always evaluates to true. | Fixed missing result: Alert |
101+
| tst.js:49:5:49:19 | This guard always evaluates to false. | Fixed missing result: Alert |
102+
| tst.js:50:5:50:19 | This guard always evaluates to false. | Fixed missing result: Alert |
103+
| tst.js:53:9:53:9 | This guard always evaluates to false. | Fixed missing result: Alert |
104+
| tst.js:56:9:56:9 | This guard always evaluates to false. | Fixed missing result: Alert |
105+
| tst.js:59:9:59:9 | This guard always evaluates to true. | Fixed missing result: Alert |
106+
| tst.js:66:5:66:5 | This guard always evaluates to false. | Fixed missing result: Alert |
107+
| tst.js:67:5:67:5 | This guard always evaluates to false. | Fixed missing result: Alert |
108+
| tst.js:68:5:68:5 | This guard always evaluates to true. | Fixed missing result: Alert |
109+
| tst.js:71:9:71:23 | This guard always evaluates to false. | Fixed missing result: Alert |
110+
| tst.js:74:9:74:23 | This guard always evaluates to true. | Fixed missing result: Alert |
111+
| tst.js:77:9:77:23 | This guard always evaluates to true. | Fixed missing result: Alert |
112+
| tst.js:84:9:84:22 | This guard always evaluates to true. | Fixed missing result: Alert |
113+
| tst.js:85:9:85:22 | This guard always evaluates to true. | Fixed missing result: Alert |
114+
| tst.js:86:9:86:22 | This guard always evaluates to false. | Fixed missing result: Alert |
115+
| tst.js:89:9:89:22 | This guard always evaluates to false. | Fixed missing result: Alert |
116+
| tst.js:90:9:90:22 | This guard always evaluates to false. | Fixed missing result: Alert |
117+
| tst.js:91:9:91:22 | This guard always evaluates to true. | Fixed missing result: Alert |
118+
| tst.js:94:9:94:32 | This guard always evaluates to true. | Fixed missing result: Alert |
119+
| tst.js:95:9:95:32 | This guard always evaluates to false. | Fixed missing result: Alert |
120+
| tst.js:96:9:96:32 | This guard always evaluates to false. | Fixed missing result: Alert |
121+
| tst.js:100:5:100:27 | This guard always evaluates to true. | Fixed missing result: Alert |
122+
| tst.js:101:5:101:27 | This guard always evaluates to false. | Fixed missing result: Alert |
123+
| tst.js:114:5:114:15 | This guard always evaluates to true. | Fixed missing result: Alert |
124+
| tst.js:115:5:115:22 | This guard always evaluates to true. | Fixed missing result: Alert |
125+
| tst.js:116:5:116:19 | This guard always evaluates to true. | Fixed missing result: Alert |
126+
| tst.js:124:6:124:20 | This guard always evaluates to false. | Fixed missing result: Alert |
127+
| tst.js:124:25:124:34 | This guard always evaluates to true. | Fixed missing result: Alert |
128+
| tst.js:125:5:125:19 | This guard always evaluates to false. | Fixed missing result: Alert |
129+
| tst.js:125:24:125:33 | This guard always evaluates to true. | Fixed missing result: Alert |
130+
| tst.js:127:5:127:18 | This guard always evaluates to false. | Fixed missing result: Alert |
131+
| tst.js:127:23:127:31 | This guard always evaluates to false. | Fixed missing result: Alert |
132+
| tst.js:127:23:127:31 | This guard always evaluates to true. | Fixed missing result: Alert |
133+
| tst.js:128:5:128:18 | This guard always evaluates to true. | Fixed missing result: Alert |
134+
| tst.js:128:23:128:31 | This guard always evaluates to false. | Fixed missing result: Alert |
135+
| tst.js:128:23:128:31 | This guard always evaluates to true. | Fixed missing result: Alert |
136+
| tst.js:129:5:129:19 | This guard always evaluates to false. | Fixed missing result: Alert |
137+
| tst.js:129:24:129:33 | This guard always evaluates to true. | Fixed missing result: Alert |
138+
| tst.js:130:5:130:22 | This guard always evaluates to false. | Fixed missing result: Alert |
139+
| tst.js:130:27:130:39 | This guard always evaluates to true. | Fixed missing result: Alert |
140+
| tst.js:131:5:131:19 | This guard always evaluates to true. | Fixed missing result: Alert |
141+
| tst.js:131:24:131:33 | This guard always evaluates to false. | Fixed missing result: Alert |
142+
| tst.js:132:7:132:21 | This guard always evaluates to true. | Fixed missing result: Alert |
143+
| tst.js:132:26:132:35 | This guard always evaluates to false. | Fixed missing result: Alert |
144+
| tst.js:133:5:133:22 | This guard always evaluates to false. | Fixed missing result: Alert |
145+
| tst.js:133:27:133:36 | This guard always evaluates to true. | Fixed missing result: Alert |
146+
| tst.js:135:5:135:18 | This guard always evaluates to true. | Fixed missing result: Alert |
147+
| tst.js:135:23:135:31 | This guard always evaluates to true. | Fixed missing result: Alert |
148+
| tst.js:138:24:138:33 | This guard always evaluates to false. | Fixed missing result: Alert |
149+
| tst.js:140:13:140:22 | This guard always evaluates to false. | Fixed missing result: Alert |
150+
| tst.js:156:23:156:31 | This guard always evaluates to true. | Fixed missing result: Alert |
151+
| tst.js:158:13:158:21 | This guard always evaluates to true. | Fixed missing result: Alert |
152+
| tst.js:177:2:177:2 | This guard always evaluates to false. | Fixed missing result: Alert |
153+
| tst.js:178:2:178:2 | This guard always evaluates to false. | Fixed missing result: Alert |
154+
| tst2.js:4:12:4:35 | This guard always evaluates to true. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/JSDoc/BadParamTag/BadParamTag.expected‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
1+
#select
12
| tst.js:3:4:3:9 | @param | @param tag is missing description. |
23
| tst.js:3:4:3:9 | @param | @param tag is missing name. |
34
| tst.js:7:4:7:9 | @param | @param tag is missing description. |
45
| tst.js:11:4:11:9 | @param | @param tag is missing description. |
6+
testFailures
7+
| tst.js:3:4:3:9 | @param tag is missing description. | Unexpected result: Alert |
8+
| tst.js:3:4:3:9 | @param tag is missing name. | Unexpected result: Alert |
9+
| tst.js:7:4:7:9 | @param tag is missing description. | Unexpected result: Alert |
10+
| tst.js:11:4:11:9 | @param tag is missing description. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/React/UnusedOrUndefinedStateProperty/UnusedOrUndefinedStateProperty.expected‎

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,27 +3,13 @@
33
| undefined.js:1:1:1:34 | class C ... }\\n} | Component state property 'notWritten' is $@, but it is never written. | undefined.js:8:9:8:29 | this.st ... Written | read |
44
| undefined.js:1:1:1:34 | class C ... }\\n} | Component state property 'notWrittenButReadInChain' is $@, but it is never written. | undefined.js:9:9:9:43 | this.st ... InChain | read |
55
| undefined.js:32:1:32:34 | class C ... }\\n} | Component state property 'notWrittenThrougExternalPropertyAccess' is $@, but it is never written. | undefined.js:35:9:35:57 | this.st ... yAccess | read |
6-
| undefined.js:61:19:61:19 | {\\n r ... ;\\n }\\n} | Component state property 'notWrittenInKnownInitializerObject' is $@, but it is never written. | undefined.js:64:9:64:53 | this.st ... rObject | read |
6+
| undefined.js:61:19:61:30 | { // $ ... ;\\n }\\n} | Component state property 'notWrittenInKnownInitializerObject' is $@, but it is never written. | undefined.js:64:9:64:53 | this.st ... rObject | read |
77
| unused.js:1:1:1:34 | class C ... }\\n} | Component state property 'notRead' is $@, but it is never read. | unused.js:6:9:6:26 | this.state.notRead | written |
88
| unused.js:27:1:27:34 | class C ... }\\n} | Component state property 'notReadThrougExternaPropertyAccess' is $@, but it is never read. | unused.js:30:9:30:53 | this.st ... yAccess | written |
99
testFailures
10-
| undefined.js:8:32:8:41 | // $ Alert | Missing result: Alert |
11-
| undefined.js:9:46:9:55 | // $ Alert | Missing result: Alert |
12-
| undefined.js:27:43:27:84 | // $ Al ... P above | Missing result: Alert |
13-
| undefined.js:35:60:35:69 | // $ Alert | Missing result: Alert |
14-
| undefined.js:48:58:48:99 | // $ Al ... P above | Missing result: Alert |
15-
| undefined.js:56:58:56:99 | // $ Al ... P above | Missing result: Alert |
16-
| undefined.js:61:19:61:19 | Component state property 'notWrittenInKnownInitializerObject' is $@, but it is never written. | Unexpected result: Alert |
17-
| undefined.js:64:56:64:65 | // $ Alert | Missing result: Alert |
18-
| undefined.js:78:58:78:99 | // $ Al ... P above | Missing result: Alert |
19-
| undefined.js:90:46:90:87 | // $ Al ... P above | Missing result: Alert |
20-
| undefined.js:100:40:100:81 | // $ Al ... P above | Missing result: Alert |
21-
| undefined.js:124:42:124:83 | // $ Al ... P above | Missing result: Alert |
22-
| undefined.js:135:67:135:108 | // $ Al ... P above | Missing result: Alert |
23-
| undefined.js:145:56:145:97 | // $ Al ... P above | Missing result: Alert |
24-
| undefined.js:152:45:152:86 | // $ Al ... P above | Missing result: Alert |
25-
| unused.js:6:34:6:43 | // $ Alert | Missing result: Alert |
26-
| unused.js:21:42:21:51 | // $ Alert | Missing result: Alert |
27-
| unused.js:30:61:30:70 | // $ Alert | Missing result: Alert |
28-
| unused.js:40:69:40:110 | // $ Al ... P above | Missing result: Alert |
29-
| unused.js:57:47:57:88 | // $ Al ... P above | Missing result: Alert |
10+
| issue7506.js:12:1:12:34 | Component state property 'p1' is $@, but it is never read. | Unexpected result: Alert |
11+
| undefined.js:1:1:1:34 | Component state property 'notWritten' is $@, but it is never written. | Unexpected result: Alert |
12+
| undefined.js:1:1:1:34 | Component state property 'notWrittenButReadInChain' is $@, but it is never written. | Unexpected result: Alert |
13+
| undefined.js:32:1:32:34 | Component state property 'notWrittenThrougExternalPropertyAccess' is $@, but it is never written. | Unexpected result: Alert |
14+
| unused.js:1:1:1:34 | Component state property 'notRead' is $@, but it is never read. | Unexpected result: Alert |
15+
| unused.js:27:1:27:34 | Component state property 'notReadThrougExternaPropertyAccess' is $@, but it is never read. | Unexpected result: Alert |

0 commit comments

Comments
 (0)