From 6431622a6debc0443f9269fe0157571ec54701c0 Mon Sep 17 00:00:00 2001 From: Kitson Kelly Date: Thu, 20 Feb 2020 14:58:05 +1100 Subject: fix: mis-detecting imports on JavaScript when there is no checkJs (#4040) This PR fixes an issue where we recursively analysed imports on plain JS files in the compiler irrespective of "checkJs" being true. This caused problems where when analysing the imports of those files, we would mistake some import like structures (AMD/CommonJS) as dependencies and try to resolve the "modules" even though the compiler would not actually look at those files. --- cli/tests/integration_tests.rs | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'cli/tests/integration_tests.rs') diff --git a/cli/tests/integration_tests.rs b/cli/tests/integration_tests.rs index 251c6da3c..5429170c7 100644 --- a/cli/tests/integration_tests.rs +++ b/cli/tests/integration_tests.rs @@ -1004,6 +1004,11 @@ itest!(cafile_info { http_server: true, }); +itest!(fix_js_imports { + args: "run --reload fix_js_imports.ts", + output: "fix_js_imports.ts.out", +}); + #[test] fn cafile_fetch() { use deno::http_cache::url_to_filename; -- cgit v1.2.3