Skip to content

Commit ad57887

Browse files
committed
Update test
1 parent 1671dd3 commit ad57887

2 files changed

Lines changed: 13 additions & 21 deletions

File tree

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,15 @@
11
// Test for imports using __dirname
22
const path = require('path');
33

4-
// Using __dirname directly
5-
const direct = require(__dirname + '/target.js'); // $ importTarget=DirnameImports/target.js
4+
require(__dirname + '/target.js'); // $ importTarget=DirnameImports/target.js
5+
require(__dirname + '/nested/target.js'); // $ importTarget=DirnameImports/nested/target.js
6+
require(__dirname + '/../import-packages.ts'); // $ importTarget=import-packages.ts
7+
require(__dirname + '/' + 'target.js'); // $ importTarget=DirnameImports/target.js
68

7-
// Using __dirname with path.join
8-
const withPathJoin = require(path.join(__dirname, 'target.js')); // $ importTarget=DirnameImports/target.js
9+
require(path.join(__dirname, 'target.js')); // $ importTarget=DirnameImports/target.js
10+
require(path.resolve(__dirname, 'target.js')); // $ MISSING: importTarget=DirnameImports/target.js // path.resolve is not a PathExpr
911

10-
// Using __dirname with nested path
11-
const nested = require(__dirname + '/nested/target.js'); // $ importTarget=DirnameImports/nested/target.js
12-
13-
// Using __dirname with parent directory
14-
const parent = require(__dirname + '/../import-packages.ts'); // $ importTarget=import-packages.ts
15-
16-
// Using __dirname with path concat and variable
1712
const subdir = 'nested';
18-
const dynamic = require(__dirname + '/' + subdir + '/target.js'); // $ importTarget=DirnameImports/nested/target.js
19-
20-
// Using __dirname in an AddExpr chain
21-
const chainedAdd = require(__dirname + '/' + 'target.js'); // $ importTarget=DirnameImports/target.js
13+
require(__dirname + '/' + subdir + '/target.js'); // $ importTarget=DirnameImports/nested/target.js
2214

2315
require(`${__dirname}/target.js`); // $ MISSING: importTarget=DirnameImports/target.js // TemplateLiteral is not a PathExpr

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@
4040
| DeclarationFiles/src/main.ts:5:1:5:27 | import ... cript"; | DeclarationFiles/lib/typescript.ts |
4141
| DeclarationFiles/src/main.ts:6:1:6:30 | import ... pt.js"; | DeclarationFiles/lib/typescript.ts |
4242
| DeclarationFiles/src/main.ts:7:1:7:32 | import ... .d.ts"; | DeclarationFiles/lib/typescript.d.ts |
43-
| DirnameImports/main.js:5:16:5:48 | require ... et.js') | DirnameImports/target.js |
44-
| DirnameImports/main.js:8:22:8:63 | require ... t.js')) | DirnameImports/target.js |
45-
| DirnameImports/main.js:11:16:11:55 | require ... et.js') | DirnameImports/nested/target.js |
46-
| DirnameImports/main.js:14:16:14:60 | require ... es.ts') | import-packages.ts |
47-
| DirnameImports/main.js:18:17:18:64 | require ... et.js') | DirnameImports/nested/target.js |
48-
| DirnameImports/main.js:21:20:21:57 | require ... et.js') | DirnameImports/target.js |
43+
| DirnameImports/main.js:4:1:4:33 | require ... et.js') | DirnameImports/target.js |
44+
| DirnameImports/main.js:5:1:5:40 | require ... et.js') | DirnameImports/nested/target.js |
45+
| DirnameImports/main.js:6:1:6:45 | require ... es.ts') | import-packages.ts |
46+
| DirnameImports/main.js:7:1:7:38 | require ... et.js') | DirnameImports/target.js |
47+
| DirnameImports/main.js:9:1:9:42 | require ... t.js')) | DirnameImports/target.js |
48+
| DirnameImports/main.js:13:1:13:48 | require ... et.js') | DirnameImports/nested/target.js |
4949
| Extended/src/main.ts:2:1:2:21 | import ... /file"; | Extended/lib/file.ts |
5050
| Extended/src/main.ts:3:1:3:24 | import ... le.ts"; | Extended/lib/file.ts |
5151
| Extended/src/main.ts:4:1:4:24 | import ... le.js"; | Extended/lib/file.ts |

0 commit comments

Comments
 (0)