Commit caff930
module: replace default paths in require.resolve()
Prior to this commit, the default search paths would be included
in the require.resolve() process, even if user specified paths
were provided. This commit causes the default paths to be
omitted by using a fake parent module.
Refs: #5963
PR-URL: #17113
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 9c0c336 commit caff930Copy full SHA for caff930
File tree
Expand file treeCollapse file tree
5 files changed
+26
-1
lines changedOpen diff view settings
Filter options
- lib
- test
- fixtures/resolve-paths
- default
- node_modules/dep
- defined/node_modules/dep
- parallel
Expand file treeCollapse file tree
5 files changed
+26
-1
lines changedOpen diff view settings
Collapse file
+4-1Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
518 | 518 | |
519 | 519 | |
520 | 520 | |
| 521 | + |
| 522 | + |
521 | 523 | |
522 | 524 | |
523 | 525 | |
524 | 526 | |
525 | | - |
| 527 | + |
| 528 | + |
526 | 529 | |
527 | 530 | |
528 | 531 | |
|
Collapse file
test/fixtures/resolve-paths/default/node_modules/dep/index.js
Copy file name to clipboardExpand all lines: test/fixtures/resolve-paths/default/node_modules/dep/index.jsWhitespace-only changes.
Collapse file
test/fixtures/resolve-paths/default/verify-paths.js
Copy file name to clipboard+21Lines changed: 21 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
Collapse file
test/fixtures/resolve-paths/defined/node_modules/dep/index.js
Copy file name to clipboardExpand all lines: test/fixtures/resolve-paths/defined/node_modules/dep/index.jsWhitespace-only changes.
Collapse file
test/parallel/test-require-resolve.js
Copy file name to clipboardExpand all lines: test/parallel/test-require-resolve.js+1Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
37 | 37 | |
38 | 38 | |
39 | 39 | |
| 40 | + |
0 commit comments