Skip to content

Commit 601194e

Browse files
asgerfasgerf
committed
JS: Accept raw test output
1 parent ced213f commit 601194e

File tree

197 files changed

+7128
-4308
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

197 files changed

+7128
-4308
lines changed

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

+3
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
1+
#select
12
| bad.js:14:7:14:23 | $compile(element) | This call to $compile may cause double compilation of 'element'. |
3+
testFailures
4+
| bad.js:14:7:14:23 | This call to $compile may cause double compilation of 'element'. | Unexpected result: Alert |

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

+5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
1+
#select
12
| duplicates.js:2:5:2:18 | function f(){} | This function has a duplicate dependency $@. | duplicates.js:3:26:3:31 | 'dup5' | dup5 |
23
| duplicates.js:6:33:6:56 | functio ... up2b){} | This function has a duplicate dependency $@. | duplicates.js:6:24:6:30 | 'dup2a' | dup2a |
34
| duplicates.js:7:33:7:56 | functio ... up3b){} | This function has a duplicate dependency $@. | duplicates.js:7:24:7:30 | 'dup3b' | dup3b |
45
| duplicates.js:8:43:8:78 | functio ... up4C){} | This function has a duplicate dependency $@. | duplicates.js:8:35:8:40 | 'dup4' | dup4 |
56
| duplicates.js:15:35:15:113 | functio ... } | This function has a duplicate dependency $@. | duplicates.js:15:25:15:32 | 'dup11a' | dup11a |
7+
testFailures
8+
| duplicates.js:2:5:2:18 | This function has a duplicate dependency $@. | Unexpected result: Alert |
9+
| duplicates.js:3:35:3:44 | // $ Alert | Missing result: Alert |
10+
| duplicates.js:15:61:15:113 | // $ Al ... unction | Missing result: Alert |

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

+3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
1+
#select
12
| repeated-injection.js:6:5:6:31 | functio ... name){} | This function has $@ defined in multiple places. | repeated-injection.js:7:5:7:22 | $Injected2.$inject | dependency injections |
23
| repeated-injection.js:6:5:6:31 | functio ... name){} | This function has $@ defined in multiple places. | repeated-injection.js:8:54:8:73 | ['name', $Injected2] | dependency injections |
34
| repeated-injection.js:10:5:10:31 | functio ... name){} | This function has $@ defined in multiple places. | repeated-injection.js:11:5:11:22 | $Injected3.$inject | dependency injections |
45
| repeated-injection.js:10:5:10:31 | functio ... name){} | This function has $@ defined in multiple places. | repeated-injection.js:12:5:12:22 | $Injected3.$inject | dependency injections |
56
| repeated-injection.js:33:5:33:85 | functio ... n\\n } | This function has $@ defined in multiple places. | repeated-injection.js:35:5:35:23 | $Injected10.$inject | dependency injections |
67
| repeated-injection.js:33:5:33:85 | functio ... n\\n } | This function has $@ defined in multiple places. | repeated-injection.js:36:56:36:76 | ['name' ... cted10] | dependency injections |
8+
testFailures
9+
| repeated-injection.js:33:33:33:85 | // $ Al ... unction | Missing result: Alert |

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

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
1+
#select
12
| unused-angular-dependency.js:5:17:5:23 | unused7 | Unused dependency unused7. |
23
| unused-angular-dependency.js:14:14:14:39 | ["unuse ... n() {}] | This function has 0 parameters, but 1 dependency is injected into it. |
34
| unused-angular-dependency.js:16:14:16:53 | ["used2 ... d2) {}] | This function has 1 parameter, but 2 dependencies are injected into it. |
45
| unused-angular-dependency.js:17:14:17:52 | ["unuse ... n() {}] | This function has 0 parameters, but 2 dependencies are injected into it. |
56
| unused-angular-dependency.js:18:14:18:106 | ["used2 ... }] | This function has 1 parameter, but 2 dependencies are injected into it. |
7+
testFailures
8+
| unused-angular-dependency.js:18:54:18:106 | // $ Al ... unction | Missing result: Alert |

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

+7
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
1+
#select
12
| tst2.html:3:6:3:24 | href={{help_url}} | Use 'ng-href' instead of 'href'. |
23
| tst.html:8:6:8:24 | href={{help_url}} | Use 'ng-href' instead of 'href'. |
34
| tst.html:10:40:10:83 | srcset=#/resources/pics-large/{{item._id}} | Use 'ng-srcset' instead of 'srcset'. |
45
| tst.html:11:10:11:52 | src=#/resources/pics-default/{{item._id}} | Use 'ng-src' instead of 'src'. |
56
| tst_fragment.html:3:6:3:24 | href={{help_url}} | Use 'ng-href' instead of 'href'. |
7+
testFailures
8+
| tst.html:8:6:8:24 | Use 'ng-href' instead of 'href'. | Unexpected result: Alert |
9+
| tst.html:10:40:10:83 | Use 'ng-srcset' instead of 'srcset'. | Unexpected result: Alert |
10+
| tst.html:11:10:11:52 | Use 'ng-src' instead of 'src'. | Unexpected result: Alert |
11+
| tst2.html:3:6:3:24 | Use 'ng-href' instead of 'href'. | Unexpected result: Alert |
12+
| tst_fragment.html:3:6:3:24 | Use 'ng-href' instead of 'href'. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/DOM/HTML/AmbiguousIdAttribute.expected

+9
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,12 @@
1+
#select
12
| AmbiguousIdAttribute.html:4:5:4:14 | id=first | This element has the same id as $@. | AmbiguousIdAttribute.html:5:5:5:14 | id=first | another element |
23
| AmbiguousIdAttribute_fragment.html:2:7:2:16 | id=first | This element has the same id as $@. | AmbiguousIdAttribute_fragment.html:3:7:3:16 | id=first | another element |
34
| tst.js:17:22:17:33 | id="theDiff" | This element has the same id as $@. | tst.js:17:46:17:57 | id="theDiff" | another element |
5+
testFailures
6+
| AmbiguousIdAttribute.html:4:5:4:14 | This element has the same id as $@. | Unexpected result: Alert |
7+
| AmbiguousIdAttribute_fragment.html:2:7:2:16 | This element has the same id as $@. | Unexpected result: Alert |
8+
| tst.js:5:67:5:76 | // $ Alert | Missing result: Alert |
9+
| tst.js:7:68:7:77 | // $ Alert | Missing result: Alert |
10+
| tst.js:9:20:9:29 | // $ Alert | Missing result: Alert |
11+
| tst.js:12:60:12:69 | // $ Alert | Missing result: Alert |
12+
| tst.js:19:51:19:60 | // $ Alert | Missing result: Alert |

‎javascript/ql/test/query-tests/DOM/HTML/ConflictingAttributes.expected

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
#select
12
| ConflictingAttributes.html:1:4:1:27 | href=http://semmle.com | This attribute has the same name as $@ of the same element, but a different value. | ConflictingAttributes.html:1:29:1:53 | href=https://semmle.com | another attribute |
23
| tst.js:5:4:5:27 | href="h ... le.com" | This attribute has the same name as $@ of the same element, but a different value. | tst.js:5:29:5:53 | href="h ... le.com" | another attribute |
34
| tst.js:12:4:12:27 | href="h ... le.com" | This attribute has the same name as $@ of the same element, but a different value. | tst.js:12:29:12:46 | href={someValue()} | another attribute |
5+
testFailures
6+
| ConflictingAttributes.html:1:4:1:27 | This attribute has the same name as $@ of the same element, but a different value. | Unexpected result: Alert |
7+
| tst.js:7:68:7:77 | // $ Alert | Missing result: Alert |
8+
| tst.js:9:20:9:29 | // $ Alert | Missing result: Alert |
9+
| tst.js:17:73:17:82 | // $ Alert | Missing result: Alert |
10+
| tst.js:19:51:19:60 | // $ Alert | Missing result: Alert |

‎javascript/ql/test/query-tests/DOM/HTML/DuplicateAttributes.expected

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
#select
12
| DuplicateAttributes.html:1:4:1:28 | href=https://semmle.com | This attribute $@. | DuplicateAttributes.html:1:30:1:54 | href=https://semmle.com | is duplicated later |
23
| tst.js:7:4:7:28 | href="h ... le.com" | This attribute $@. | tst.js:7:30:7:54 | href="h ... le.com" | is duplicated later |
34
| tst.js:19:17:19:28 | id="theDiff" | This attribute $@. | tst.js:19:30:19:41 | id="theDiff" | is duplicated later |
5+
testFailures
6+
| DuplicateAttributes.html:1:4:1:28 | This attribute $@. | Unexpected result: Alert |
7+
| tst.js:5:67:5:76 | // $ Alert | Missing result: Alert |
8+
| tst.js:9:20:9:29 | // $ Alert | Missing result: Alert |
9+
| tst.js:12:60:12:69 | // $ Alert | Missing result: Alert |
10+
| tst.js:17:73:17:82 | // $ Alert | Missing result: Alert |

‎javascript/ql/test/query-tests/DOM/HTML/MalformedIdAttribute.expected

+11
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,16 @@
1+
#select
12
| AmbiguousIdAttributeGood.html:8:5:8:19 | id=invalid id | The value of the id attribute must not contain any space characters. |
23
| AmbiguousIdAttributeGood.html:9:5:9:19 | id=invalid id | The value of the id attribute must not contain any space characters. |
34
| MalformedIdAttribute.html:1:6:1:27 | id=heading important | The value of the id attribute must not contain any space characters. |
45
| tst.js:9:6:9:10 | id="" | The value of the id attribute must contain at least one character. |
56
| tst.js:10:6:10:13 | id="a b" | The value of the id attribute must not contain any space characters. |
7+
testFailures
8+
| AmbiguousIdAttributeGood.html:8:5:8:19 | The value of the id attribute must not contain any space characters. | Unexpected result: Alert |
9+
| AmbiguousIdAttributeGood.html:9:5:9:19 | The value of the id attribute must not contain any space characters. | Unexpected result: Alert |
10+
| MalformedIdAttribute.html:1:6:1:27 | The value of the id attribute must not contain any space characters. | Unexpected result: Alert |
11+
| tst.js:5:67:5:76 | // $ Alert | Missing result: Alert |
12+
| tst.js:7:68:7:77 | // $ Alert | Missing result: Alert |
13+
| tst.js:10:6:10:13 | The value of the id attribute must not contain any space characters. | Unexpected result: Alert |
14+
| tst.js:12:60:12:69 | // $ Alert | Missing result: Alert |
15+
| tst.js:17:73:17:82 | // $ Alert | Missing result: Alert |
16+
| tst.js:19:51:19:60 | // $ Alert | Missing result: Alert |

‎javascript/ql/test/query-tests/DOM/PseudoEval/PseudoEval.expected

+7
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
1+
#select
12
| tst.js:1:1:1:22 | window. ... ut(";") | Avoid using functions that evaluate strings as code. |
23
| tst.js:2:1:2:24 | setInte ... te();") | Avoid using functions that evaluate strings as code. |
34
| tst.js:4:1:4:31 | documen ... i!');") | Avoid using functions that evaluate strings as code. |
45
| tst.js:5:1:5:30 | window. ... gger;") | Avoid using functions that evaluate strings as code. |
56
| tst.js:9:3:9:33 | documen ... d = 2") | Avoid using functions that evaluate strings as code. |
7+
testFailures
8+
| tst.js:1:1:1:22 | Avoid using functions that evaluate strings as code. | Unexpected result: Alert |
9+
| tst.js:2:1:2:24 | Avoid using functions that evaluate strings as code. | Unexpected result: Alert |
10+
| tst.js:4:1:4:31 | Avoid using functions that evaluate strings as code. | Unexpected result: Alert |
11+
| tst.js:5:1:5:30 | Avoid using functions that evaluate strings as code. | Unexpected result: Alert |
12+
| tst.js:9:3:9:33 | Avoid using functions that evaluate strings as code. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/DOM/TargetBlank/TargetBlank.expected

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| tst.html:23:1:23:61 | <a>...</> | External links without noopener/noreferrer are a potential security risk. |
23
| tst.html:24:1:24:48 | <a>...</> | External links without noopener/noreferrer are a potential security risk. |
34
| tst.html:25:1:25:36 | <a>...</> | External links without noopener/noreferrer are a potential security risk. |
@@ -7,3 +8,9 @@
78
| tst.js:19:1:19:51 | <a data ... ple</a> | External links without noopener/noreferrer are a potential security risk. |
89
| tst.js:31:12:31:39 | $("<a/> ... X}}" }) | External links without noopener/noreferrer are a potential security risk. |
910
| tst.js:39:12:39:20 | $("<a/>") | External links without noopener/noreferrer are a potential security risk. |
11+
testFailures
12+
| tst.html:23:1:23:61 | External links without noopener/noreferrer are a potential security risk. | Unexpected result: Alert |
13+
| tst.html:24:1:24:48 | External links without noopener/noreferrer are a potential security risk. | Unexpected result: Alert |
14+
| tst.html:30:1:30:61 | External links without noopener/noreferrer are a potential security risk. | Unexpected result: Alert |
15+
| tst.js:18:1:18:58 | External links without noopener/noreferrer are a potential security risk. | Unexpected result: Alert |
16+
| tst.js:19:1:19:51 | External links without noopener/noreferrer are a potential security risk. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/ArgumentsRedefined/ArgumentsRedefined.expected

+4
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1+
#select
12
| tst.js:3:5:3:13 | arguments | Redefinition of arguments. |
23
| tst.js:7:7:7:15 | arguments | Redefinition of arguments. |
34
| tst.js:11:11:11:19 | arguments | Redefinition of arguments. |
45
| tst.js:12:11:12:19 | arguments | Redefinition of arguments. |
6+
testFailures
7+
| tst.js:11:11:11:19 | Redefinition of arguments. | Unexpected result: Alert |
8+
| tst.js:12:11:12:19 | Redefinition of arguments. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/AssignmentToConst/AssignmentToConst.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
1+
#select
12
| classes.js:3:1:3:10 | class C {} | Assignment to variable C, which is $@ constant. | classes.js:1:1:1:13 | const C = 45; | declared |
23
| functions.js:3:10:3:10 | C | Assignment to variable C, which is $@ constant. | functions.js:1:1:1:13 | const C = 45; | declared |
34
| tst.js:3:1:3:6 | x = 42 | Assignment to variable x, which is $@ constant. | tst.js:1:1:1:21 | const x ... y = 42; | declared |
45
| tst.js:5:1:5:6 | y = 23 | Assignment to variable y, which is $@ constant. | tst.js:1:1:1:21 | const x ... y = 42; | declared |
56
| tst.js:7:5:7:10 | y = -1 | Assignment to variable y, which is $@ constant. | tst.js:1:1:1:21 | const x ... y = 42; | declared |
67
| tst.js:9:1:9:3 | ++x | Assignment to variable x, which is $@ constant. | tst.js:1:1:1:21 | const x ... y = 42; | declared |
78
| tst.js:21:10:21:14 | [ c ] | Assignment to variable c, which is $@ constant. | tst.js:20:5:20:19 | const c = null; | declared |
9+
testFailures
10+
| tst.js:21:10:21:14 | Assignment to variable c, which is $@ constant. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/ConflictingFunctions/ConflictingFunctions.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
1+
#select
12
| tst.js:3:12:3:12 | g | Declaration of function g conflicts with $@ in the same scope. | tst.js:7:12:7:12 | g | another declaration |
3+
testFailures
4+
| tst.js:3:12:3:12 | Declaration of function g conflicts with $@ in the same scope. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/DeadStoreOfGlobal/DeadStoreOfGlobal.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,5 @@
1+
#select
12
| tst.js:1:1:1:1 | g | This definition of g is useless, since its value is never read. |
23
| worker.js:3:1:3:9 | onmissage | This definition of onmissage is useless, since its value is never read. |
4+
testFailures
5+
| worker.js:3:1:3:9 | This definition of onmissage is useless, since its value is never read. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/DeadStoreOfLocal/DeadStoreOfLocal.expected

+8
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| overload.ts:10:12:10:14 | baz | The value assigned to baz here is unused. |
23
| tst2.js:25:9:25:14 | x = 23 | The initial value of x is unused, since it is always overwritten. |
34
| tst2.js:27:9:27:14 | x = 42 | The value assigned to x here is unused. |
@@ -11,3 +12,10 @@
1112
| tst.js:130:7:130:13 | {x} = o | The initial value of x is unused, since it is always overwritten. |
1213
| tst.js:160:6:160:14 | [x] = [0] | The initial value of x is unused, since it is always overwritten. |
1314
| tst.js:170:7:170:17 | nSign = foo | The value assigned to nSign here is unused. |
15+
testFailures
16+
| tst.js:43:6:43:11 | The initial value of x is unused, since it is always overwritten. | Unexpected result: Alert |
17+
| tst.js:49:6:49:11 | The initial value of x is unused, since it is always overwritten. | Unexpected result: Alert |
18+
| tst.js:130:7:130:13 | The initial value of x is unused, since it is always overwritten. | Unexpected result: Alert |
19+
| tst.js:160:6:160:14 | The initial value of x is unused, since it is always overwritten. | Unexpected result: Alert |
20+
| tst.js:170:7:170:17 | The value assigned to nSign here is unused. | Unexpected result: Alert |
21+
| tst2.js:27:9:27:14 | The value assigned to x here is unused. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/DeadStoreOfProperty/DeadStoreOfProperty.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| exports.js:2:1:2:22 | exports ... = "yes" | This write to property 'answer' is useless, since $@ always overrides it. | exports.js:3:1:3:21 | exports ... = "no" | another property write |
23
| fieldInit.ts:10:3:10:8 | f = 4; | This write to property 'f' is useless, since $@ always overrides it. | fieldInit.ts:13:5:13:14 | this.f = 5 | another property write |
34
| fieldInit.ts:18:22:18:22 | h | This write to property 'h' is useless, since $@ always overrides it. | fieldInit.ts:19:5:19:14 | this.h = h | another property write |
@@ -16,3 +17,5 @@
1617
| tst.js:96:5:96:17 | o.pure18 = 42 | This write to property 'pure18' is useless, since $@ always overrides it. | tst.js:97:5:97:17 | o.pure18 = 42 | another property write |
1718
| tst.js:114:2:114:14 | o.setter = 42 | This write to property 'setter' is useless, since $@ always overrides it. | tst.js:115:2:115:14 | o.setter = 87 | another property write |
1819
| tst.js:118:2:118:104 | Object. ... lue()}) | This write to property 'prop' is useless, since $@ always overrides it. | tst.js:119:2:119:12 | o.prop = 42 | another property write |
20+
testFailures
21+
| tst.js:114:2:114:14 | This write to property 'setter' is useless, since $@ always overrides it. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/DeclBeforeUse/DeclBeforeUse.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,5 @@
1+
#select
12
| jslint.js:7:1:7:1 | z | Variable 'z' is used before its $@. | jslint.js:8:11:8:11 | z | declaration |
23
| tst2.js:5:13:5:13 | x | Variable 'x' is used before its $@. | tst2.js:6:5:6:5 | x | declaration |
4+
testFailures
5+
| jslint.js:6:4:6:13 | // $ Alert | Missing result: Alert |

‎javascript/ql/test/query-tests/Declarations/DefaultArgumentReferencesNestedFunction/DefaultArgumentReferencesNestedFunction.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
1+
#select
12
| tst.js:1:19:1:28 | defaultVal | This expression refers to $@ before it is defined. | tst.js:2:5:4:5 | functio ... ;\\n } | defaultVal |
3+
testFailures
4+
| tst.js:1:19:1:28 | This expression refers to $@ before it is defined. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/MissingThisQualifier/MissingThisQualifier.expected

+3
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| abstract-missing.ts:3:5:3:24 | setAudioProperties() | This call refers to a global function, and not the local method $@. | abstract-missing.ts:6:3:6:32 | abstrac ... ties(); | setAudioProperties |
23
| indirection.js:7:9:7:20 | m("default") | This call refers to a global function, and not the local method $@. | indirection.js:2:5:4:5 | m() {\\n ... ;\\n } | m |
34
| missing1.js:3:5:3:24 | setAudioProperties() | This call refers to a global function, and not the local method $@. | missing1.js:6:3:7:3 | setAudi ... (){\\n } | setAudioProperties |
@@ -8,3 +9,5 @@
89
| namespaces-uses.ts:16:7:16:22 | globalFunction() | This call refers to a global function, and not the local method $@. | namespaces-uses.ts:15:5:17:5 | globalF ... t\\n } | globalFunction |
910
| namespaces-uses.ts:30:7:30:22 | globalFunction() | This call refers to a global function, and not the local method $@. | namespaces-uses.ts:29:5:31:5 | globalF ... t\\n } | globalFunction |
1011
| not-ignored-by-jslint.js:4:5:4:24 | setAudioProperties() | This call refers to a global function, and not the local method $@. | not-ignored-by-jslint.js:7:3:8:3 | setAudi ... (){\\n } | setAudioProperties |
12+
testFailures
13+
| namespaces-uses.ts:22:34:22:103 | // $ Al ... ignored | Missing result: Alert |

‎javascript/ql/test/query-tests/Declarations/MissingVarDecl/MissingVarDecl.expected

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| test.js:5:7:5:7 | i | Variable i is used like a local variable, but is missing a declaration. |
23
| test.js:12:7:12:7 | i | Variable i is used like a local variable, but is missing a declaration. |
34
| test.js:20:2:20:2 | y | Variable y is used like a local variable, but is missing a declaration. |
@@ -7,3 +8,9 @@
78
| test.js:66:9:66:20 | unresolvable | Variable unresolvable is used like a local variable, but is missing a declaration. |
89
| tst3.js:7:10:7:10 | x | Variable x is used like a local variable, but is missing a declaration. |
910
| tst3.js:7:16:7:19 | rest | Variable rest is used like a local variable, but is missing a declaration. |
11+
testFailures
12+
| test.js:54:9:54:18 | // $ Alert | Missing result: Alert |
13+
| test.js:55:6:55:6 | Variable y is used like a local variable, but is missing a declaration. | Unexpected result: Alert |
14+
| test.js:66:9:66:20 | Variable unresolvable is used like a local variable, but is missing a declaration. | Unexpected result: Alert |
15+
| tst3.js:7:10:7:10 | Variable x is used like a local variable, but is missing a declaration. | Unexpected result: Alert |
16+
| tst3.js:7:16:7:19 | Variable rest is used like a local variable, but is missing a declaration. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/RedeclaredVariable/RedeclaredVariable.expected

+5
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#select
12
| sameNameForFunctionAndVariable.js:2:10:2:13 | fun1 | This variable has already $@. | sameNameForFunctionAndVariable.js:1:5:1:8 | fun1 | been declared |
23
| sameNameForFunctionAndVariable.js:7:5:7:8 | fun2 | This variable has already $@. | sameNameForFunctionAndVariable.js:5:10:5:13 | fun2 | been declared |
34
| tst3.js:9:10:9:10 | f | This variable has already $@. | tst3.js:3:10:3:10 | f | been declared |
@@ -6,3 +7,7 @@
67
| tst4.js:12:12:12:12 | i | This variable has already $@. | tst4.js:11:7:11:7 | i | been declared |
78
| tst4.js:19:7:19:7 | y | This variable has already $@. | tst4.js:17:7:17:7 | y | been declared |
89
| tst4.js:22:5:22:5 | g | This variable has already $@. | tst4.js:6:10:6:10 | g | been declared |
10+
testFailures
11+
| sameNameForFunctionAndVariable.js:2:10:2:13 | This variable has already $@. | Unexpected result: Alert |
12+
| sameNameForFunctionAndVariable.js:7:5:7:8 | This variable has already $@. | Unexpected result: Alert |
13+
| tst4.js:22:5:22:5 | This variable has already $@. | Unexpected result: Alert |

‎javascript/ql/test/query-tests/Declarations/UniqueParameterNames/UniqueParameterNames.expected

+4
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,6 @@
1+
#select
12
| tst.js:2:1:2:1 | x | This parameter has the same name as $@ of the same function. | tst.js:4:1:4:6 | \\u0078 | another parameter |
23
| tst.js:3:1:3:1 | x | This parameter has the same name as $@ of the same function. | tst.js:4:1:4:6 | \\u0078 | another parameter |
4+
testFailures
5+
| tst.js:2:1:2:1 | This parameter has the same name as $@ of the same function. | Unexpected result: Alert |
6+
| tst.js:4:8:4:17 | // $ Alert | Missing result: Alert |

0 commit comments

Comments
 (0)