Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 55fd948

Browse files
author
Esben Sparre Andreasen
committed
JS: add test for js/trivial-conditional
1 parent f956e57 commit 55fd948

2 files changed

Lines changed: 29 additions & 0 deletions

File tree

javascript/ql/test/query-tests/Statements/UselessConditional/UselessConditional.expected

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,3 +33,5 @@
3333
| UselessConditionalGood.js:69:12:69:13 | xy | This use of variable 'xy' always evaluates to false. |
3434
| UselessConditionalGood.js:85:12:85:13 | xy | This use of variable 'xy' always evaluates to false. |
3535
| UselessConditionalGood.js:97:12:97:13 | xy | This use of variable 'xy' always evaluates to false. |
36+
| UselessConditionalGood.js:114:8:114:8 | p | This use of variable 'p' always evaluates to false. |
37+
| UselessConditionalGood.js:122:8:122:8 | p | This use of variable 'p' always evaluates to false. |

javascript/ql/test/query-tests/Statements/UselessConditional/UselessConditionalGood.js

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,3 +99,30 @@ function getLastLine(input) {
9999
f10(undefined, undefined);
100100

101101
})();
102+
103+
(function(){
104+
function g(p) {
105+
return function() {
106+
if (p) { // OK, whitelisted
107+
g(p);
108+
}
109+
};
110+
}
111+
112+
function f(p = false) {
113+
return function() {
114+
if (p) { // OK, whitelisted
115+
f(p);
116+
}
117+
};
118+
}
119+
120+
function h(p = false) {
121+
(function() {
122+
if (p) { // OK, whitelisted
123+
124+
}
125+
});
126+
}
127+
h();
128+
});

0 commit comments

Comments
 (0)