Skip to content

Commit 57d814f

Browse files
Esben Sparre AndreasenGitHub Enterprise
Esben Sparre Andreasen
authored and
GitHub Enterprise
committed
Merge pull request github#29 from max/js/generify-tests
JavaScript: Generify a few tests from unknown sources.
2 parents 8d7a3bf + 2d88805 commit 57d814f

Some content is hidden

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

57 files changed

+219
-466
lines changed

‎javascript/ql/test/library-tests/ES20xxFeatures/ES20xxFeatures.expected

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@
3131
| es2016.js:1:7:1:12 | y ** z | 2016 | exponentiation operator |
3232
| es2017.js:1:1:3:1 | functio ... (0,);\\n} | 2017 | trailing function comma |
3333
| es2017.js:2:3:2:7 | f(0,) | 2017 | trailing function comma |
34-
| es2017.js:5:1:13:1 | async f ... ret;\\n} | 2017 | async/await |
35-
| es2017.js:6:3:6:17 | let ret = null; | 2015 | let |
34+
| es2017.js:5:1:13:1 | async f ... rn v;\\n} | 2017 | async/await |
35+
| es2017.js:6:3:6:15 | let v = null; | 2015 | let |
3636
| es2017.js:8:5:10:5 | for(con ... ;\\n } | 2015 | for..of loops |
37-
| es2017.js:8:9:8:18 | const anim | 2015 | const |
38-
| es2017.js:9:13:9:28 | await anim(elem) | 2017 | async/await |
37+
| es2017.js:8:9:8:15 | const x | 2015 | const |
38+
| es2017.js:9:11:9:23 | await h(x, y) | 2017 | async/await |
3939
| es2018.js:1:5:1:8 | `\\u` | 2015 | template literals |
4040
| es2018.js:1:6:1:7 | \\u | 2018 | revised template literal syntax |
4141
| es2018.js:2:1:2:18 | /\\k<ws>(?<ws>\\w+)/ | 2018 | named capture groups |

‎javascript/ql/test/library-tests/ES20xxFeatures/es2017.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@ function f(x,) {
22
f(0,);
33
}
44

5-
async function chainAnimationsAsync(elem, animations) {
6-
let ret = null;
5+
async function g(y, xs) {
6+
let v = null;
77
try {
8-
for(const anim of animations) {
9-
ret = await anim(elem);
8+
for(const x of xs) {
9+
v = await h(x, y);
1010
}
11-
} catch(e) { /* ignore and keep going */ }
12-
return ret;
11+
} catch(e) { }
12+
return v;
1313
}

‎javascript/ql/test/library-tests/Functions/Function.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
| arrowfns.js:3:12:3:41 | () => { ... "); ; } |
44
| defaultargs.js:1:1:1:24 | functio ... +19) {} |
55
| exprclosures.js:1:7:1:21 | function(x) x+1 |
6-
| generators.js:1:1:5:1 | functio ... ex++;\\n} |
7-
| generators.js:7:2:7:19 | function* bar() {} |
6+
| generators.js:1:1:4:1 | functio ... i++;\\n} |
7+
| generators.js:6:2:6:19 | function* bar() {} |
88
| restparms.js:1:1:2:1 | functio ... ys) {\\n} |
99
| tst.js:1:1:1:15 | function A() {} |
1010
| tst.js:2:1:2:16 | function B(x) {} |

‎javascript/ql/test/library-tests/Functions/generators.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
function* foo(){
2-
var index = 0;
3-
while(index <= 2) // when index reaches 2, yield's done will be true and its value will be undefined;
4-
yield index++;
2+
for (var i=0; i<=2; ++i)
3+
yield i++;
54
}
65

76
(function* bar() {})();

‎javascript/ql/test/library-tests/Functions/getArgumentsVariable.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
| defaultargs.js:1:1:1:24 | functio ... +19) {} |
22
| exprclosures.js:1:7:1:21 | function(x) x+1 |
3-
| generators.js:1:1:5:1 | functio ... ex++;\\n} |
4-
| generators.js:7:2:7:19 | function* bar() {} |
3+
| generators.js:1:1:4:1 | functio ... i++;\\n} |
4+
| generators.js:6:2:6:19 | function* bar() {} |
55
| restparms.js:1:1:2:1 | functio ... ys) {\\n} |
66
| tst.js:1:1:1:15 | function A() {} |
77
| tst.js:2:1:2:16 | function B(x) {} |

‎javascript/ql/test/library-tests/Functions/getBody.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
| arrowfns.js:3:12:3:41 | () => { ... "); ; } | arrowfns.js:3:18:3:41 | { alert ... "); ; } |
44
| defaultargs.js:1:1:1:24 | functio ... +19) {} | defaultargs.js:1:23:1:24 | {} |
55
| exprclosures.js:1:7:1:21 | function(x) x+1 | exprclosures.js:1:19:1:21 | x+1 |
6-
| generators.js:1:1:5:1 | functio ... ex++;\\n} | generators.js:1:16:5:1 | {\\n var ... ex++;\\n} |
7-
| generators.js:7:2:7:19 | function* bar() {} | generators.js:7:18:7:19 | {} |
6+
| generators.js:1:1:4:1 | functio ... i++;\\n} | generators.js:1:16:4:1 | {\\n for ... i++;\\n} |
7+
| generators.js:6:2:6:19 | function* bar() {} | generators.js:6:18:6:19 | {} |
88
| restparms.js:1:1:2:1 | functio ... ys) {\\n} | restparms.js:1:22:2:1 | {\\n} |
99
| tst.js:1:1:1:15 | function A() {} | tst.js:1:14:1:15 | {} |
1010
| tst.js:2:1:2:16 | function B(x) {} | tst.js:2:15:2:16 | {} |

‎javascript/ql/test/library-tests/Functions/getBodyStmt.expected

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
| arrowfns.js:3:12:3:41 | () => { ... "); ; } | 0 | arrowfns.js:3:20:3:37 | alert("Wake up!"); |
22
| arrowfns.js:3:12:3:41 | () => { ... "); ; } | 1 | arrowfns.js:3:39:3:39 | ; |
3-
| generators.js:1:1:5:1 | functio ... ex++;\\n} | 0 | generators.js:2:3:2:16 | var index = 0; |
4-
| generators.js:1:1:5:1 | functio ... ex++;\\n} | 1 | generators.js:3:3:4:18 | while(i ... ndex++; |
3+
| generators.js:1:1:4:1 | functio ... i++;\\n} | 0 | generators.js:2:3:3:14 | for (va ... ld i++; |
54
| tst.js:10:1:10:31 | functio ... ents; } | 0 | tst.js:10:16:10:29 | var arguments; |
65
| tst.js:11:1:11:35 | functio ... ts; } } | 0 | tst.js:11:16:11:33 | { var arguments; } |
76
| tst.js:12:1:12:44 | functio ... s) {} } | 0 | tst.js:12:16:12:42 | try { } ... nts) {} |

‎javascript/ql/test/library-tests/Functions/getEnclosingStmt.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
| arrowfns.js:3:12:3:41 | () => { ... "); ; } | arrowfns.js:3:1:3:50 | setTime ... 60000); |
44
| defaultargs.js:1:1:1:24 | functio ... +19) {} | defaultargs.js:1:1:1:24 | functio ... +19) {} |
55
| exprclosures.js:1:7:1:21 | function(x) x+1 | exprclosures.js:1:1:1:23 | a.map(f ... ) x+1); |
6-
| generators.js:1:1:5:1 | functio ... ex++;\\n} | generators.js:1:1:5:1 | functio ... ex++;\\n} |
7-
| generators.js:7:2:7:19 | function* bar() {} | generators.js:7:1:7:23 | (functi ... {})(); |
6+
| generators.js:1:1:4:1 | functio ... i++;\\n} | generators.js:1:1:4:1 | functio ... i++;\\n} |
7+
| generators.js:6:2:6:19 | function* bar() {} | generators.js:6:1:6:23 | (functi ... {})(); |
88
| restparms.js:1:1:2:1 | functio ... ys) {\\n} | restparms.js:1:1:2:1 | functio ... ys) {\\n} |
99
| tst.js:1:1:1:15 | function A() {} | tst.js:1:1:1:15 | function A() {} |
1010
| tst.js:2:1:2:16 | function B(x) {} | tst.js:2:1:2:16 | function B(x) {} |

‎javascript/ql/test/library-tests/Functions/getId.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
| defaultargs.js:1:1:1:24 | functio ... +19) {} | defaultargs.js:1:10:1:10 | f | f |
2-
| generators.js:1:1:5:1 | functio ... ex++;\\n} | generators.js:1:11:1:13 | foo | foo |
3-
| generators.js:7:2:7:19 | function* bar() {} | generators.js:7:12:7:14 | bar | bar |
2+
| generators.js:1:1:4:1 | functio ... i++;\\n} | generators.js:1:11:1:13 | foo | foo |
3+
| generators.js:6:2:6:19 | function* bar() {} | generators.js:6:12:6:14 | bar | bar |
44
| restparms.js:1:1:2:1 | functio ... ys) {\\n} | restparms.js:1:10:1:10 | r | r |
55
| tst.js:1:1:1:15 | function A() {} | tst.js:1:10:1:10 | A | A |
66
| tst.js:2:1:2:16 | function B(x) {} | tst.js:2:10:2:10 | B | B |

‎javascript/ql/test/library-tests/Functions/getScope.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
| arrowfns.js:3:12:3:41 | () => { ... "); ; } |
44
| defaultargs.js:1:1:1:24 | functio ... +19) {} |
55
| exprclosures.js:1:7:1:21 | function(x) x+1 |
6-
| generators.js:1:1:5:1 | functio ... ex++;\\n} |
7-
| generators.js:7:2:7:19 | function* bar() {} |
6+
| generators.js:1:1:4:1 | functio ... i++;\\n} |
7+
| generators.js:6:2:6:19 | function* bar() {} |
88
| restparms.js:1:1:2:1 | functio ... ys) {\\n} |
99
| tst.js:1:1:1:15 | function A() {} |
1010
| tst.js:2:1:2:16 | function B(x) {} |

‎javascript/ql/test/library-tests/Functions/getVariable.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
| defaultargs.js:1:1:1:24 | functio ... +19) {} |
2-
| generators.js:1:1:5:1 | functio ... ex++;\\n} |
3-
| generators.js:7:2:7:19 | function* bar() {} |
2+
| generators.js:1:1:4:1 | functio ... i++;\\n} |
3+
| generators.js:6:2:6:19 | function* bar() {} |
44
| restparms.js:1:1:2:1 | functio ... ys) {\\n} |
55
| tst.js:1:1:1:15 | function A() {} |
66
| tst.js:2:1:2:16 | function B(x) {} |

‎javascript/ql/test/library-tests/Functions/isGenerator.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
| generators.js:1:1:5:1 | functio ... ex++;\\n} |
2-
| generators.js:7:2:7:19 | function* bar() {} |
1+
| generators.js:1:1:4:1 | functio ... i++;\\n} |
2+
| generators.js:6:2:6:19 | function* bar() {} |

‎javascript/ql/test/library-tests/frameworks/Testing/uncustomised/Tests.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
| test2.js:10:9:19:9 | functio ... } |
1+
| test2.js:8:9:12:9 | functio ... } |
22
| test.js:2:1:4:2 | QUnit.t ... " );\\n}) |
33
| test.js:8:3:10:4 | it("con ... );\\n }) |

‎javascript/ql/test/library-tests/frameworks/Testing/uncustomised/test2.js

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,14 @@
11
Function.RegisterNamespace("Test");
22

33
[Fixture]
4-
Test.SampleFixture=function(){
5-
4+
Test.test = function() {
65
[Fixture]
7-
function NestedFixture(){
8-
6+
function inner(){
97
[Fact]
10-
function NestedFact(){
11-
// Arrange
8+
function inner2() {
129
var expected="expected";
13-
14-
// Act
1510
var actual=expected;
16-
17-
// Assert
1811
Assert.Equal(expected,actual);
1912
}
20-
2113
}
22-
2314
};

‎javascript/ql/test/library-tests/frameworks/xUnit/XUnitAnnotation.expected

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
| tst.js:3:1:3:9 | [Fixture] | tst.js:4:1:23:2 | Test.Sa ... }\\n\\n}; |
2-
| tst.js:6:5:6:13 | [Fixture] | tst.js:7:5:21:5 | functio ... \\n\\n } |
3-
| tst.js:9:9:9:14 | [Fact] | tst.js:10:9:19:9 | functio ... } |
4-
| tst.js:25:1:25:43 | [Import ... t.js")] | tst.js:27:1:31:2 | Test.xU ... ..]\\n\\n}; |
5-
| tst.js:26:1:26:9 | Fixture | tst.js:27:1:31:2 | Test.xU ... ..]\\n\\n}; |
6-
| tst.js:33:1:33:9 | [Fixture] | tst.js:34:1:46:2 | Test.Re ... }\\n\\n}; |
7-
| tst.js:36:5:38:7 | [Import ... })] | tst.js:40:5:44:5 | functio ... \\n\\n } |
8-
| tst.js:39:5:39:10 | Fact | tst.js:40:5:44:5 | functio ... \\n\\n } |
1+
| tst.js:3:1:3:9 | [Fixture] | tst.js:4:1:14:2 | Test.Ex ... }\\n}; |
2+
| tst.js:5:5:5:13 | [Fixture] | tst.js:6:5:13:5 | functio ... }\\n } |
3+
| tst.js:7:9:7:14 | [Fact] | tst.js:8:9:12:9 | functio ... } |
4+
| tst.js:16:1:16:43 | [Import ... t.js")] | tst.js:18:1:22:2 | Test.xU ... ..]\\n\\n}; |
5+
| tst.js:17:1:17:9 | Fixture | tst.js:18:1:22:2 | Test.xU ... ..]\\n\\n}; |
6+
| tst.js:24:1:24:9 | [Fixture] | tst.js:25:1:34:2 | Test.Ex ... }\\n}; |
7+
| tst.js:27:5:29:7 | [Import ... })] | tst.js:31:5:33:5 | functio ... ]\\n } |
8+
| tst.js:30:5:30:10 | Fact | tst.js:31:5:33:5 | functio ... ]\\n } |

‎javascript/ql/test/library-tests/frameworks/xUnit/XUnitAttribute.expected

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
| tst.js:3:2:3:8 | Fixture | Fixture | 0 |
2-
| tst.js:6:6:6:12 | Fixture | Fixture | 0 |
3-
| tst.js:9:10:9:13 | Fact | Fact | 0 |
4-
| tst.js:25:2:25:42 | Import( ... rt.js") | Import | 1 |
5-
| tst.js:26:2:26:8 | Fixture | Fixture | 0 |
6-
| tst.js:33:2:33:8 | Fixture | Fixture | 0 |
7-
| tst.js:36:6:38:6 | ImportJ ... \\n }) | ImportJson | 2 |
8-
| tst.js:39:6:39:9 | Fact | Fact | 0 |
2+
| tst.js:5:6:5:12 | Fixture | Fixture | 0 |
3+
| tst.js:7:10:7:13 | Fact | Fact | 0 |
4+
| tst.js:16:2:16:42 | Import( ... rt.js") | Import | 1 |
5+
| tst.js:17:2:17:8 | Fixture | Fixture | 0 |
6+
| tst.js:24:2:24:8 | Fixture | Fixture | 0 |
7+
| tst.js:27:6:29:6 | ImportJ ... \\n }) | ImportJson | 2 |
8+
| tst.js:30:6:30:9 | Fact | Fact | 0 |

‎javascript/ql/test/library-tests/frameworks/xUnit/XUnitAttributeParameters.expected

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
| tst.js:25:2:25:42 | Import( ... rt.js") | 0 | tst.js:25:9:25:41 | "../../ ... ert.js" |
2-
| tst.js:36:6:38:6 | ImportJ ... \\n }) | 0 | tst.js:36:17:36:49 | "../../ ... ert.js" |
3-
| tst.js:36:6:38:6 | ImportJ ... \\n }) | 1 | tst.js:36:51:38:5 | functio ... ;\\n } |
1+
| tst.js:16:2:16:42 | Import( ... rt.js") | 0 | tst.js:16:9:16:41 | "../../ ... ert.js" |
2+
| tst.js:27:6:29:6 | ImportJ ... \\n }) | 0 | tst.js:27:17:27:49 | "../../ ... ert.js" |
3+
| tst.js:27:6:29:6 | ImportJ ... \\n }) | 1 | tst.js:27:51:29:5 | functio ... ;\\n } |

‎javascript/ql/test/library-tests/frameworks/xUnit/XUnitFixture.expected

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
| tst.js:4:20:23:1 | functio ... }\\n\\n} | tst.js:3:1:3:9 | [Fixture] |
2-
| tst.js:7:5:21:5 | functio ... \\n\\n } | tst.js:6:5:6:13 | [Fixture] |
3-
| tst.js:27:22:31:1 | functio ... ...]\\n\\n} | tst.js:25:1:25:43 | [Import ... t.js")] |
4-
| tst.js:27:22:31:1 | functio ... ...]\\n\\n} | tst.js:26:1:26:9 | Fixture |
5-
| tst.js:34:21:46:1 | functio ... }\\n\\n} | tst.js:33:1:33:9 | [Fixture] |
1+
| tst.js:4:20:14:1 | functio ... }\\n} | tst.js:3:1:3:9 | [Fixture] |
2+
| tst.js:6:5:13:5 | functio ... }\\n } | tst.js:5:5:5:13 | [Fixture] |
3+
| tst.js:18:24:22:1 | functio ... ...]\\n\\n} | tst.js:16:1:16:43 | [Import ... t.js")] |
4+
| tst.js:18:24:22:1 | functio ... ...]\\n\\n} | tst.js:17:1:17:9 | Fixture |
5+
| tst.js:25:21:34:1 | functio ... }\\n} | tst.js:24:1:24:9 | [Fixture] |

‎javascript/ql/test/library-tests/frameworks/xUnit/XUnitTarget.expected

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
| tst.js:4:1:23:2 | Test.Sa ... }\\n\\n}; |
2-
| tst.js:7:5:21:5 | functio ... \\n\\n } |
3-
| tst.js:10:9:19:9 | functio ... } |
4-
| tst.js:12:13:12:36 | var exp ... ected"; |
5-
| tst.js:15:13:15:32 | var actual=expected; |
6-
| tst.js:27:1:31:2 | Test.xU ... ..]\\n\\n}; |
7-
| tst.js:34:1:46:2 | Test.Re ... }\\n\\n}; |
8-
| tst.js:35:5:35:22 | var response=null; |
9-
| tst.js:37:9:37:24 | response=result; |
10-
| tst.js:40:5:44:5 | functio ... \\n\\n } |
1+
| tst.js:4:1:14:2 | Test.Ex ... }\\n}; |
2+
| tst.js:6:5:13:5 | functio ... }\\n } |
3+
| tst.js:8:9:12:9 | functio ... } |
4+
| tst.js:9:13:9:36 | var exp ... ected"; |
5+
| tst.js:10:13:10:32 | var actual=expected; |
6+
| tst.js:18:1:22:2 | Test.xU ... ..]\\n\\n}; |
7+
| tst.js:25:1:34:2 | Test.Ex ... }\\n}; |
8+
| tst.js:26:5:26:24 | var response = null; |
9+
| tst.js:28:9:28:24 | response=result; |
10+
| tst.js:31:5:33:5 | functio ... ]\\n } |

‎javascript/ql/test/library-tests/frameworks/xUnit/tst.js

Lines changed: 8 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,34 @@
11
Function.RegisterNamespace("Test");
22

33
[Fixture]
4-
Test.SampleFixture=function(){
5-
4+
Test.ExampleTest = function() {
65
[Fixture]
7-
function NestedFixture(){
8-
6+
function inner() {
97
[Fact]
10-
function NestedFact(){
11-
// Arrange
8+
function inner2(){
129
var expected="expected";
13-
14-
// Act
1510
var actual=expected;
16-
17-
// Assert
1811
Assert.Equal(expected,actual);
1912
}
20-
2113
}
22-
2314
};
2415

2516
[Import("../../Source/xUnit.js/Assert.js")]
2617
[Fixture]
27-
Test.xUnit.js.Assert=function(){
18+
Test.xUnit.js.Assert = function() {
2819

2920
// [...]
3021

3122
};
3223

3324
[Fixture]
34-
Test.ResponseParser=function(){
35-
var response=null;
25+
Test.ExampleTest2 = function() {
26+
var response = null;
3627
[ImportJson("../../Source/xUnit.js/Assert.js",function(path,result){
3728
response=result;
3829
})]
3930
[Fact]
40-
function ParsesKnownObjectsFromResponse(){
41-
31+
function f(){
4232
// [...]
43-
4433
}
45-
46-
};
34+
};

‎javascript/ql/test/violation-tests/DOM/Alert/Alert.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
| AlertUse.js:1:1:1:12 | alert("bla") | Avoid calling alert. |
1+
| alert.js:1:1:1:12 | alert("hi!") | Avoid calling alert. |

‎javascript/ql/test/violation-tests/DOM/Alert/AlertUse.js

Lines changed: 0 additions & 5 deletions
This file was deleted.

‎javascript/ql/test/violation-tests/DOM/Alert/alert.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
alert("hi!"); // NOT OK
2+
x.alert("hi!"); // OK
3+
new alert(); // OK
4+
function alert() { } // OK

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

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,2 @@
1-
| argumentsRedefined.js:2:2:2:10 | arguments | Redefinition of arguments. |
2-
| argumentsRedefined.js:3:20:3:28 | arguments | Redefinition of arguments. |
3-
| argumentsRedefined.js:4:17:4:25 | arguments | Redefinition of arguments. |
4-
| argumentsRedefined.js:5:11:5:19 | arguments | Redefinition of arguments. |
5-
| argumentsRedefined.js:15:12:15:20 | arguments | Redefinition of arguments. |
1+
| tst.js:3:5:3:13 | arguments | Redefinition of arguments. |
2+
| tst.js:7:7:7:15 | arguments | Redefinition of arguments. |

‎javascript/ql/test/violation-tests/Declarations/ArgumentsRedefined/argumentsRedefined.js

Lines changed: 0 additions & 23 deletions
This file was deleted.

‎javascript/ql/test/violation-tests/Declarations/ArgumentsRedefined/tst.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
function f() {
2+
if (arguments[0].isArray())
3+
arguments = arguments[0]; // NOT OK
4+
}
5+
6+
function g(x, y) {
7+
var arguments = [y, x]; // NOT OK
8+
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
| jslint.js:7:1:7:1 | z | Variable 'z' is used before its $@. | jslint.js:8:11:8:11 | z | declaration |
2-
| variableDeclarationAfterUsageCheck.js:5:7:5:7 | x | Variable 'x' is used before its $@. | variableDeclarationAfterUsageCheck.js:6:5:6:5 | x | declaration |
2+
| tst2.js:5:13:5:13 | x | Variable 'x' is used before its $@. | tst2.js:6:5:6:5 | x | declaration |

‎javascript/ql/test/violation-tests/Declarations/DeclBeforeUse/tst2.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
function f(x) {
2+
console.log(x); // OK
3+
}
4+
5+
console.log(x); // NOT OK
6+
var x = 1;
7+
8+
function g() {
9+
console.log(y); // OK (not in same function)
10+
}
11+
var y = 1;

‎javascript/ql/test/violation-tests/Declarations/DeclBeforeUse/variableDeclarationAfterUsageCheck.js

Lines changed: 0 additions & 12 deletions
This file was deleted.

‎javascript/ql/test/violation-tests/Declarations/DeclBeforeUse/variableDeclarationWithoutVar.js

Lines changed: 0 additions & 5 deletions
This file was deleted.

0 commit comments

Comments
 (0)