summaryrefslogtreecommitdiff
path: root/tests/integration/lint_tests.rs
diff options
context:
space:
mode:
authorMatt Mastracci <matthew@mastracci.com>2024-02-10 13:22:13 -0700
committerGitHub <noreply@github.com>2024-02-10 20:22:13 +0000
commitf5e46c9bf2f50d66a953fa133161fc829cecff06 (patch)
tree8faf2f5831c1c7b11d842cd9908d141082c869a5 /tests/integration/lint_tests.rs
parentd2477f780630a812bfd65e3987b70c0d309385bb (diff)
chore: move cli/tests/ -> tests/ (#22369)
This looks like a massive PR, but it's only a move from cli/tests -> tests, and updates of relative paths for files. This is the first step towards aggregate all of the integration test files under tests/, which will lead to a set of integration tests that can run without the CLI binary being built. While we could leave these tests under `cli`, it would require us to keep a more complex directory structure for the various test runners. In addition, we have a lot of complexity to ignore various test files in the `cli` project itself (cargo publish exclusion rules, autotests = false, etc). And finally, the `tests/` folder will eventually house the `test_ffi`, `test_napi` and other testing code, reducing the size of the root repo directory. For easier review, the extremely large and noisy "move" is in the first commit (with no changes -- just a move), while the remainder of the changes to actual files is in the second commit.
Diffstat (limited to 'tests/integration/lint_tests.rs')
-rw-r--r--tests/integration/lint_tests.rs211
1 files changed, 211 insertions, 0 deletions
diff --git a/tests/integration/lint_tests.rs b/tests/integration/lint_tests.rs
new file mode 100644
index 000000000..b266fb5b7
--- /dev/null
+++ b/tests/integration/lint_tests.rs
@@ -0,0 +1,211 @@
+// Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
+
+use test_util::assert_contains;
+use test_util::TestContextBuilder;
+
+itest!(ignore_unexplicit_files {
+ args: "lint --ignore=./",
+ output_str: Some("error: No target files found.\n"),
+ exit_code: 1,
+});
+
+itest!(all {
+ args: "lint lint/without_config/file1.js lint/without_config/file2.ts lint/without_config/ignored_file.ts",
+ output: "lint/expected.out",
+ exit_code: 1,
+});
+
+itest!(quiet {
+ args: "lint --quiet lint/without_config/file1.js",
+ output: "lint/expected_quiet.out",
+ exit_code: 1,
+});
+
+itest!(json {
+ args:
+ "lint --json lint/without_config/file1.js lint/without_config/file2.ts lint/without_config/ignored_file.ts lint/without_config/malformed.js",
+ output: "lint/expected_json.out",
+ exit_code: 1,
+});
+
+itest!(compact {
+ args:
+ "lint --compact lint/without_config/file1.js lint/without_config/ignored_file.tss",
+ output: "lint/expected_compact.out",
+ exit_code: 1,
+});
+
+itest!(ignore {
+ args:
+ "lint --ignore=lint/without_config/file1.js,lint/without_config/malformed.js,lint/without_config/lint_with_config/ lint/without_config/",
+ output: "lint/expected_ignore.out",
+ exit_code: 1,
+});
+
+itest!(glob {
+ args: "lint --ignore=lint/without_config/malformed.js,lint/with_config/ lint/without_config/",
+ output: "lint/expected_glob.out",
+ exit_code: 1,
+});
+
+itest!(stdin {
+ args: "lint -",
+ input: Some("let _a: any;"),
+ output: "lint/expected_from_stdin.out",
+ exit_code: 1,
+});
+
+itest!(stdin_json {
+ args: "lint --json -",
+ input: Some("let _a: any;"),
+ output: "lint/expected_from_stdin_json.out",
+ exit_code: 1,
+});
+
+itest!(rules {
+ args: "lint --rules",
+ output: "lint/expected_rules.out",
+ exit_code: 0,
+});
+
+// Make sure that the rules are printed if quiet option is enabled.
+itest!(rules_quiet {
+ args: "lint --rules -q",
+ output: "lint/expected_rules.out",
+ exit_code: 0,
+});
+
+itest!(lint_with_config {
+ args: "lint --config lint/Deno.jsonc lint/with_config/",
+ output: "lint/with_config.out",
+ exit_code: 1,
+});
+
+itest!(lint_with_report_config {
+ args: "lint --config lint/Deno.compact.format.jsonc lint/with_config/",
+ output: "lint/with_report_config_compact.out",
+ exit_code: 1,
+});
+
+// Check if CLI flags take precedence
+itest!(lint_with_report_config_override {
+ args: "lint --config lint/Deno.compact.format.jsonc lint/with_config/ --json",
+ output: "lint/with_report_config_override.out",
+ exit_code: 1,
+});
+
+itest!(lint_with_config_and_flags {
+ args: "lint --config lint/Deno.jsonc --ignore=lint/with_config/a.ts",
+ output: "lint/with_config_and_flags.out",
+ exit_code: 1,
+});
+
+itest!(lint_with_config_without_tags {
+ args: "lint --config lint/Deno.no_tags.jsonc lint/with_config/",
+ output: "lint/with_config_without_tags.out",
+ exit_code: 1,
+});
+
+itest!(lint_with_malformed_config {
+ args: "lint --config lint/Deno.malformed.jsonc",
+ output: "lint/with_malformed_config.out",
+ exit_code: 1,
+});
+
+itest!(lint_with_malformed_config2 {
+ args: "lint --config lint/Deno.malformed2.jsonc",
+ output: "lint/with_malformed_config2.out",
+ exit_code: 1,
+});
+
+#[test]
+fn lint_with_glob_config() {
+ let context = TestContextBuilder::new().cwd("lint").build();
+
+ let cmd_output = context
+ .new_command()
+ .args("lint --config deno.glob.json")
+ .run();
+
+ cmd_output.assert_exit_code(1);
+
+ let output = cmd_output.combined_output();
+ if cfg!(windows) {
+ assert_contains!(output, r"glob\nested\fizz\fizz.ts:1:10");
+ assert_contains!(output, r"glob\pages\[id].ts:1:10");
+ assert_contains!(output, r"glob\nested\fizz\bar.ts:1:10");
+ assert_contains!(output, r"glob\nested\foo\foo.ts:1:10");
+ assert_contains!(output, r"glob\data\test1.js:1:10");
+ assert_contains!(output, r"glob\nested\foo\bar.ts:1:10");
+ assert_contains!(output, r"glob\nested\foo\fizz.ts:1:10");
+ assert_contains!(output, r"glob\nested\fizz\foo.ts:1:10");
+ assert_contains!(output, r"glob\data\test1.ts:1:10");
+ } else {
+ assert_contains!(output, "glob/nested/fizz/fizz.ts:1:10");
+ assert_contains!(output, "glob/pages/[id].ts:1:10");
+ assert_contains!(output, "glob/nested/fizz/bar.ts:1:10");
+ assert_contains!(output, "glob/nested/foo/foo.ts:1:10");
+ assert_contains!(output, "glob/data/test1.js:1:10");
+ assert_contains!(output, "glob/nested/foo/bar.ts:1:10");
+ assert_contains!(output, "glob/nested/foo/fizz.ts:1:10");
+ assert_contains!(output, "glob/nested/fizz/foo.ts:1:10");
+ assert_contains!(output, "glob/data/test1.ts:1:10");
+ }
+ assert_contains!(output, "Found 9 problems");
+ assert_contains!(output, "Checked 9 files");
+}
+
+#[test]
+fn lint_with_glob_config_and_flags() {
+ let context = TestContextBuilder::new().cwd("lint").build();
+
+ let cmd_output = context
+ .new_command()
+ .args("lint --config deno.glob.json --ignore=glob/nested/**/bar.ts")
+ .run();
+
+ cmd_output.assert_exit_code(1);
+
+ let output = cmd_output.combined_output();
+ if cfg!(windows) {
+ assert_contains!(output, r"glob\nested\fizz\fizz.ts:1:10");
+ assert_contains!(output, r"glob\pages\[id].ts:1:10");
+ assert_contains!(output, r"glob\nested\fizz\bazz.ts:1:10");
+ assert_contains!(output, r"glob\nested\foo\foo.ts:1:10");
+ assert_contains!(output, r"glob\data\test1.js:1:10");
+ assert_contains!(output, r"glob\nested\foo\bazz.ts:1:10");
+ assert_contains!(output, r"glob\nested\foo\fizz.ts:1:10");
+ assert_contains!(output, r"glob\nested\fizz\foo.ts:1:10");
+ assert_contains!(output, r"glob\data\test1.ts:1:10");
+ } else {
+ assert_contains!(output, "glob/nested/fizz/fizz.ts:1:10");
+ assert_contains!(output, "glob/pages/[id].ts:1:10");
+ assert_contains!(output, "glob/nested/fizz/bazz.ts:1:10");
+ assert_contains!(output, "glob/nested/foo/foo.ts:1:10");
+ assert_contains!(output, "glob/data/test1.js:1:10");
+ assert_contains!(output, "glob/nested/foo/bazz.ts:1:10");
+ assert_contains!(output, "glob/nested/foo/fizz.ts:1:10");
+ assert_contains!(output, "glob/nested/fizz/foo.ts:1:10");
+ assert_contains!(output, "glob/data/test1.ts:1:10");
+ }
+ assert_contains!(output, "Found 9 problems");
+ assert_contains!(output, "Checked 9 files");
+
+ let cmd_output = context
+ .new_command()
+ .args("lint --config deno.glob.json glob/data/test1.?s")
+ .run();
+
+ cmd_output.assert_exit_code(1);
+
+ let output = cmd_output.combined_output();
+ if cfg!(windows) {
+ assert_contains!(output, r"glob\data\test1.js:1:10");
+ assert_contains!(output, r"glob\data\test1.ts:1:10");
+ } else {
+ assert_contains!(output, "glob/data/test1.js:1:10");
+ assert_contains!(output, "glob/data/test1.ts:1:10");
+ }
+ assert_contains!(output, "Found 2 problems");
+ assert_contains!(output, "Checked 2 files");
+}