hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-30 19:26:02 +02:00
Code Issues Packages Projects Releases Wiki Activity

add support for the pinkie-promise polyfill

Browse Source
This commit is contained in:
Erik Krogh Kristensen
2021-06-20 19:16:14 +02:00
parent bb1c971348
commit b574292dab
5 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
javascript/ql/src/semmle/javascript/Promises.qll
View File

@@ -83,6 +83,8 @@ private DataFlow::SourceNode getAPromiseObject() {
result = DataFlow::moduleImport("native-promise-only")
or
result = DataFlow::moduleImport("when")
or
result = DataFlow::moduleImport("pinkie-promise")
}
/**

1
javascript/ql/test/library-tests/Promises/AdditionalPromises.expected
View File

@@ -82,3 +82,4 @@
| promises.js:88:17:90:4 | Q.Promi ... );\\n }) |
| promises.js:112:17:112:62 | new RSV ... ct) {}) |
| promises.js:124:19:124:30 | when(source) |
| promises.js:130:14:130:69 | new Pro ... s'); }) |

5
javascript/ql/test/library-tests/Promises/promises.js
View File

@@ -124,3 +124,8 @@
const promise = when(source);
const promise2 = when.resolve(source);
})();
(function() {
var Promise = require('pinkie-promise');
var prom = new Promise(function (resolve) { resolve('unicorns'); });
})();

3
javascript/ql/test/library-tests/Promises/tests.expected
View File

@@ -91,6 +91,7 @@ test_PromiseDefinition_getExecutor
| promises.js:43:19:45:6 | Q.Promi ... \\n }) | promises.js:43:29:45:5 | functio ... ;\\n } |
| promises.js:88:17:90:4 | Q.Promi ... );\\n }) | promises.js:88:27:90:3 | functio ... e);\\n } |
| promises.js:112:17:112:62 | new RSV ... ct) {}) | promises.js:112:34:112:61 | functio ... ect) {} |
| promises.js:130:14:130:69 | new Pro ... s'); }) | promises.js:130:26:130:68 | functio ... ns'); } |
test_PromiseDefinition_getAFinallyHandler
| flow.js:105:2:105:48 | new Pro ... "BLA")) | flow.js:105:58:105:76 | x => {throw source} |
| flow.js:109:2:109:48 | new Pro ... "BLA")) | flow.js:109:58:109:70 | x => rejected |
@@ -129,6 +130,7 @@ test_PromiseDefinition
| promises.js:88:17:90:4 | Q.Promi ... );\\n }) |
| promises.js:112:17:112:62 | new RSV ... ct) {}) |
| promises.js:124:19:124:30 | when(source) |
| promises.js:130:14:130:69 | new Pro ... s'); }) |
test_PromiseDefinition_getAResolveHandler
| flow.js:24:2:24:49 | new Pro ... ource)) | flow.js:24:56:24:67 | x => sink(x) |
| flow.js:26:2:26:49 | new Pro ... ource)) | flow.js:26:56:26:66 | x => foo(x) |
@@ -211,6 +213,7 @@ test_PromiseDefinition_getResolveParameter
| promises.js:43:19:45:6 | Q.Promi ... \\n }) | promises.js:43:39:43:45 | resolve |
| promises.js:88:17:90:4 | Q.Promi ... );\\n }) | promises.js:88:37:88:43 | resolve |
| promises.js:112:17:112:62 | new RSV ... ct) {}) | promises.js:112:43:112:49 | resolve |
| promises.js:130:14:130:69 | new Pro ... s'); }) | promises.js:130:36:130:42 | resolve |
test_PromiseDefinition_getACatchHandler
| flow.js:32:2:32:49 | new Pro ... ource)) | flow.js:32:57:32:68 | x => sink(x) |
| flow.js:48:2:48:36 | new Pro ... urce }) | flow.js:48:44:48:55 | x => sink(x) |