diff options
author | Matt Mastracci <matthew@mastracci.com> | 2024-02-10 13:22:13 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-10 20:22:13 +0000 |
commit | f5e46c9bf2f50d66a953fa133161fc829cecff06 (patch) | |
tree | 8faf2f5831c1c7b11d842cd9908d141082c869a5 /tests/integration/lint_tests.rs | |
parent | d2477f780630a812bfd65e3987b70c0d309385bb (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.rs | 211 |
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"); +} |