Skip to content

Commit be5de9c

Browse files
committed
JS: Update test output
path.resolve() and template expressions are now working. Previously they could not be resolved because Import.getImportedPath() returned a PathExpr, and these were not instances of PathExpr.
1 parent ed2a832 commit be5de9c

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

javascript/ql/test/library-tests/PathResolution/DirnameImports/main.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ require(__dirname + '/../import-packages.ts'); // $ importTarget=import-packages
77
require(__dirname + '/' + 'target.js'); // $ importTarget=DirnameImports/target.js
88

99
require(path.join(__dirname, 'target.js')); // $ importTarget=DirnameImports/target.js
10-
require(path.resolve(__dirname, 'target.js')); // $ MISSING: importTarget=DirnameImports/target.js
10+
require(path.resolve(__dirname, 'target.js')); // $ importTarget=DirnameImports/target.js
1111

1212
const subdir = 'nested';
1313
require(__dirname + '/' + subdir + '/target.js'); // $ importTarget=DirnameImports/nested/target.js
1414

15-
require(`${__dirname}/target.js`); // $ MISSING: importTarget=DirnameImports/target.js
15+
require(`${__dirname}/target.js`); // $ importTarget=DirnameImports/target.js

javascript/ql/test/library-tests/PathResolution/test.expected

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@
5252
| DirnameImports/main.js:6:1:6:45 | require ... es.ts') | import-packages.ts |
5353
| DirnameImports/main.js:7:1:7:38 | require ... et.js') | DirnameImports/target.js |
5454
| DirnameImports/main.js:9:1:9:42 | require ... t.js')) | DirnameImports/target.js |
55+
| DirnameImports/main.js:10:1:10:45 | require ... t.js')) | DirnameImports/target.js |
5556
| DirnameImports/main.js:13:1:13:48 | require ... et.js') | DirnameImports/nested/target.js |
57+
| DirnameImports/main.js:15:1:15:33 | require ... et.js`) | DirnameImports/target.js |
5658
| Extended/src/main.ts:2:1:2:21 | import ... /file"; | Extended/lib/file.ts |
5759
| Extended/src/main.ts:3:1:3:24 | import ... le.ts"; | Extended/lib/file.ts |
5860
| Extended/src/main.ts:4:1:4:24 | import ... le.js"; | Extended/lib/file.ts |

0 commit comments

Comments
 (0)