Skip to content

Commit 51d6f75

Browse files
committed
JS: Add partially failing test
1 parent d5d1365 commit 51d6f75

File tree

3 files changed

+22
-0
lines changed

3 files changed

+22
-0
lines changed

javascript/ql/test/library-tests/frameworks/data/test.expected

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
consistencyIssue
2+
| library-tests/frameworks/data/test.js:231 | expected an alert, but found none | NOT OK | |
3+
| library-tests/frameworks/data/test.js:232 | expected an alert, but found none | NOT OK | |
24
taintFlow
35
| paramDecorator.ts:6:54:6:54 | x | paramDecorator.ts:7:10:7:10 | x |
46
| test.js:5:30:5:37 | source() | test.js:5:8:5:38 | testlib ... urce()) |
@@ -62,6 +64,8 @@ taintFlow
6264
| test.js:223:45:223:52 | source() | test.js:223:45:223:52 | source() |
6365
| test.js:225:39:225:46 | source() | test.js:225:39:225:46 | source() |
6466
| test.js:226:50:226:57 | source() | test.js:226:50:226:57 | source() |
67+
| test.js:230:59:230:66 | source() | test.js:230:59:230:66 | source() |
68+
| test.js:233:59:233:66 | source() | test.js:233:59:233:66 | source() |
6569
isSink
6670
| test.js:54:18:54:25 | source() | test-sink |
6771
| test.js:55:22:55:29 | source() | test-sink |
@@ -128,6 +132,8 @@ isSink
128132
| test.js:223:45:223:52 | source() | test-sink |
129133
| test.js:225:39:225:46 | source() | test-sink |
130134
| test.js:226:50:226:57 | source() | test-sink |
135+
| test.js:230:59:230:66 | source() | test-sink |
136+
| test.js:233:59:233:66 | source() | test-sink |
131137
syntaxErrors
132138
| Member[foo |
133139
| Member[foo] .Member[bar] |

javascript/ql/test/library-tests/frameworks/data/test.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,4 +226,9 @@ function typeVars() {
226226
testlib.typevar.left.left.x.right.right.mySink(source()); // NOT OK
227227
testlib.typevar.left.x.right.right.mySink(source()); // OK - mismatched left and right
228228
testlib.typevar.left.left.x.right.mySink(source()); // OK - mismatched left and right
229+
230+
testlib.typevar.getThis().getThis().left.x.right.mySink(source()); // NOT OK
231+
testlib.typevar.left.getThis().getThis().x.right.mySink(source()); // NOT OK
232+
testlib.typevar.left.x.getThis().getThis().right.mySink(source()); // NOT OK
233+
testlib.typevar.left.x.right.getThis().getThis().mySink(source()); // NOT OK
229234
}

javascript/ql/test/library-tests/frameworks/data/test.ql

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,17 @@ class Steps extends ModelInput::SummaryModelCsv {
1414
"testlib;;Member[preserveAllButFirstArgument];Argument[1..];ReturnValue;taint",
1515
"testlib;;Member[preserveAllIfCall].Call;Argument[0..];ReturnValue;taint",
1616
"testlib;;Member[getSource].ReturnValue.Member[continue];Argument[this];ReturnValue;taint",
17+
"testlib;~HasThisFlow;;;Member[getThis].ReturnValue;type",
18+
]
19+
}
20+
}
21+
22+
class TypeDefs extends ModelInput::TypeModelCsv {
23+
override predicate row(string row) {
24+
row =
25+
[
26+
"testlib;~HasThisFlow;testlib;;Member[typevar]",
27+
"testlib;~HasThisFlow;testlib;~HasThisFlow;Member[left,right,x]",
1728
]
1829
}
1930
}

0 commit comments

Comments
 (0)