From ebcb032c6bc60ae94039bc047509acffd9713379 Mon Sep 17 00:00:00 2001 From: Luca Casonato Date: Sun, 27 Sep 2020 11:16:18 -0700 Subject: fix: no check recognizes require (#7720) --- cli/graph.rs | 5 ++++- cli/tests/ignore_require.js | 1 + cli/tests/integration_tests.rs | 6 ++++++ 3 files changed, 11 insertions(+), 1 deletion(-) create mode 100644 cli/tests/ignore_require.js (limited to 'cli') diff --git a/cli/graph.rs b/cli/graph.rs index b632bb626..ccc7751e4 100644 --- a/cli/graph.rs +++ b/cli/graph.rs @@ -270,7 +270,10 @@ impl Module { // Parse out all the syntactical dependencies for a module let dependencies = parsed_module.analyze_dependencies(); - for desc in dependencies.iter() { + for desc in dependencies + .iter() + .filter(|desc| desc.kind != DependencyKind::Require) + { let location = Location { filename: self.specifier.to_string(), col: desc.col, diff --git a/cli/tests/ignore_require.js b/cli/tests/ignore_require.js new file mode 100644 index 000000000..118fe14e5 --- /dev/null +++ b/cli/tests/ignore_require.js @@ -0,0 +1 @@ +require("invalid module specifier"); diff --git a/cli/tests/integration_tests.rs b/cli/tests/integration_tests.rs index 4ec816c4f..c062d3371 100644 --- a/cli/tests/integration_tests.rs +++ b/cli/tests/integration_tests.rs @@ -2516,6 +2516,12 @@ itest!(info_type_import { output: "info_type_import.out", }); +itest!(ignore_require { + args: "cache --reload --no-check ignore_require.js", + output_str: Some(""), + exit_code: 0, +}); + #[test] fn cafile_env_fetch() { use deno_core::url::Url; -- cgit v1.2.3