diff options
| author | Bartek Iwańczuk <biwanczuk@gmail.com> | 2023-05-02 02:35:33 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-05-02 02:35:33 +0200 |
| commit | 2f651b2d64523bdd377d22b8b7213a04ad82f459 (patch) | |
| tree | f5776da1ee7e25b70bd736bf2428c3fbb06b35a0 /cli/tests/integration/npm_tests.rs | |
| parent | 000315e75a20e82616a227702c98346f2b5e8b59 (diff) | |
fix(npm): canonicalize filename before returning (#18948)
This commit changes how paths for npm packages are handled,
by canonicalizing them when resolving. This is done so that instead
of returning
"node_modules/<package_name>@<version>/node_modules/<dep>/index.js"
(which is a symlink) we "node_modules/<dep>@<dep_version>/index.js.
Fixes https://github.com/denoland/deno/issues/18924
Fixes https://github.com/bluwy/create-vite-extra/issues/31
---------
Co-authored-by: David Sherret <dsherret@gmail.com>
Diffstat (limited to 'cli/tests/integration/npm_tests.rs')
| -rw-r--r-- | cli/tests/integration/npm_tests.rs | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/cli/tests/integration/npm_tests.rs b/cli/tests/integration/npm_tests.rs index 8f6ac7528..d4f2d3e45 100644 --- a/cli/tests/integration/npm_tests.rs +++ b/cli/tests/integration/npm_tests.rs @@ -156,6 +156,16 @@ itest!(mixed_case_package_name_local_dir { temp_cwd: true, }); +itest!(local_dir_resolves_symlinks { + args: "run -A index.js", + output: "npm/local_dir_resolves_symlinks/index.out", + exit_code: 0, + envs: env_vars_for_npm_tests(), + cwd: Some("npm/local_dir_resolves_symlinks/"), + copy_temp_dir: Some("npm/local_dir_resolves_symlinks/"), + http_server: true, +}); + // FIXME(bartlomieju): npm: specifiers are not handled in dynamic imports // at the moment // itest!(dynamic_import { |