test(oxlint): add CliRunResult to snapshot (#8780)

oxc-project · Jan 30, 2025 · 55c2025 · 55c2025
1 parent 9731c56
commit 55c2025
Show file tree

Hide file tree

Showing 48 changed files with 159 additions and 1 deletion.
diff --git a/...es__issue_7566__tests__main.js [email protected] b/...es__issue_7566__tests__main.js [email protected]
@@ -5,3 +5,6 @@ source: apps/oxlint/src/tester.rs
 arguments: --ignore-path fixtures/issue_7566/.oxlintignore fixtures/issue_7566/tests/main.js fixtures/issue_7566/tests/function/main.js
 working directory: 
 ----------
+----------
+CLI result: LintNoFilesFound
+----------
diff --git a/...ore-path fixtures__linter__.customignore --no-ignore [email protected] b/...ore-path fixtures__linter__.customignore --no-ignore [email protected]
@@ -15,3 +15,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...shots/_--ignore-path fixtures__linter__.customignore [email protected] b/...shots/_--ignore-path fixtures__linter__.customignore [email protected]
@@ -5,3 +5,6 @@ source: apps/oxlint/src/tester.rs
 arguments: --ignore-path fixtures/linter/.customignore fixtures/linter/nan.js
 working directory: 
 ----------
+----------
+CLI result: LintNoFilesFound
+----------
diff --git a/...pshots/_--ignore-pattern _____.js --ignore-pattern _____.vue [email protected] b/...pshots/_--ignore-pattern _____.js --ignore-pattern _____.vue [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 0 files with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/_--import-plugin -A all -D no-cycle [email protected] b/apps/oxlint/src/snapshots/_--import-plugin -A all -D no-cycle [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 2 files with 99 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/_--import-plugin [email protected] b/apps/oxlint/src/snapshots/_--import-plugin [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 99 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -141,3 +141,6 @@ working directory:
   "globals": {},
   "ignorePatterns": []
 }
+----------
+CLI result: PrintConfigResult
+----------
diff --git a/apps/oxlint/src/snapshots/_--tsconfig [email protected] b/apps/oxlint/src/snapshots/_--tsconfig [email protected]
@@ -6,3 +6,6 @@ arguments: --tsconfig oxc/tsconfig.json
 working directory: 
 ----------
 The tsconfig file "<cwd>/oxc/tsconfig.json" does not exist, Please provide a valid tsconfig file.
+----------
+CLI result: InvalidOptionTsConfig
+----------
diff --git a/..._vitest_replace__eslintrc.json [email protected] b/..._vitest_replace__eslintrc.json [email protected]
@@ -24,3 +24,6 @@ working directory:
 
 Found 1 warning and 1 error.
 Finished in <variable>ms on 1 file with 109 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/apps/oxlint/src/snapshots/_-A all [email protected] b/apps/oxlint/src/snapshots/_-A all [email protected]
@@ -43,3 +43,6 @@ working directory:
   "globals": {},
   "ignorePatterns": []
 }
+----------
+CLI result: PrintConfigResult
+----------
diff --git a/apps/oxlint/src/snapshots/_-A all [email protected] b/apps/oxlint/src/snapshots/_-A all [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 3 files with 0 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/_-D correctness [email protected] b/apps/oxlint/src/snapshots/_-D correctness [email protected]
@@ -15,3 +15,6 @@ working directory:
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...nt/src/snapshots/_-W correctness -A no-debugger [email protected] b/...nt/src/snapshots/_-W correctness -A no-debugger [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 96 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/... fixtures__eslintrc_env__eslintrc_no_env.json [email protected] b/... fixtures__eslintrc_env__eslintrc_no_env.json [email protected]
@@ -14,3 +14,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 98 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/..._-W no-undef -c fixtures__no_undef__eslintrc.json [email protected] b/..._-W no-undef -c fixtures__no_undef__eslintrc.json [email protected]
@@ -14,3 +14,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 98 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...e_extension__eslintrc.json [email protected] b/...e_extension__eslintrc.json [email protected]
@@ -13,3 +13,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...ures__eslintrc_env__eslintrc_env_browser.json [email protected] b/...ures__eslintrc_env__eslintrc_env_browser.json [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 98 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...ots/_-c fixtures__eslintrc_off__eslintrc.json [email protected] b/...ots/_-c fixtures__eslintrc_off__eslintrc.json [email protected]
@@ -13,3 +13,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/..._vitest_replace__eslintrc.json [email protected] b/..._vitest_replace__eslintrc.json [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/.../src/snapshots/_-c fixtures__import__.oxlintrc.json [email protected] b/.../src/snapshots/_-c fixtures__import__.oxlintrc.json [email protected]
@@ -16,3 +16,6 @@ working directory:
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 50 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...c/snapshots/_-c fixtures__linter__eslintrc.json [email protected] b/...c/snapshots/_-c fixtures__linter__eslintrc.json [email protected]
@@ -15,3 +15,6 @@ working directory:
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/..._-c fixtures__no_console_off__eslintrc.json [email protected] b/..._-c fixtures__no_console_off__eslintrc.json [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...atch__eslintrc.json -W no-empty [email protected] b/...atch__eslintrc.json -W no-empty [email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 98 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...h__eslintrc.json -W no-empty [email protected] b/...h__eslintrc.json -W no-empty [email protected]
@@ -15,3 +15,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 98 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...test.ts -c fixtures__overrides__.oxlintrc.json [email protected] b/...test.ts -c fixtures__overrides__.oxlintrc.json [email protected]
@@ -16,6 +16,9 @@ working directory:
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 99 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
 
 ########## 
 arguments: -c fixtures/overrides/.oxlintrc.json fixtures/overrides/test.ts
@@ -40,6 +43,9 @@ working directory:
 
 Found 1 warning and 1 error.
 Finished in <variable>ms on 1 file with 99 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
 
 ########## 
 arguments: -c fixtures/overrides/.oxlintrc.json fixtures/overrides/other.jsx
@@ -56,3 +62,6 @@ working directory:
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 99 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...napshots/_-c fixtures__overrides__directories-config.json [email protected] b/...napshots/_-c fixtures__overrides__directories-config.json [email protected]
@@ -36,3 +36,6 @@ working directory:
 
 Found 2 warnings and 2 errors.
 Finished in <variable>ms on 7 files with 96 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...tures__print_config__ban_rules__eslintrc.json -A all -D eqeqeq [email protected] b/...tures__print_config__ban_rules__eslintrc.json -A all -D eqeqeq [email protected]
@@ -50,3 +50,6 @@ working directory:
   "globals": {},
   "ignorePatterns": []
 }
+----------
+CLI result: PrintConfigResult
+----------
diff --git a/...slintrc.json --disable-typescript-plugin [email protected] b/...slintrc.json --disable-typescript-plugin [email protected]
@@ -24,3 +24,6 @@ working directory:
 
 Found 2 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 48 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...xtures__typescript_eslint__eslintrc.json [email protected] b/...xtures__typescript_eslint__eslintrc.json [email protected]
@@ -32,3 +32,6 @@ working directory:
 
 Found 3 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 59 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -40,3 +40,6 @@ working directory:
 
 Found 4 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -29,3 +29,6 @@ working directory:
 
 Found 3 warnings and 0 errors.
 Finished in <variable>ms on 3 files with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/.../oxlint/src/snapshots/_fixtures__linter__debugger.js [email protected] b/.../oxlint/src/snapshots/_fixtures__linter__debugger.js [email protected]
@@ -22,3 +22,6 @@ working directory:
 
 Found 2 warnings and 0 errors.
 Finished in <variable>ms on 2 files with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -15,3 +15,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -17,3 +17,6 @@ working directory:
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -24,3 +24,6 @@ working directory:
 
 Found 2 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -7,3 +7,6 @@ working directory:
 ----------
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 0 files with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -15,3 +15,6 @@ working directory: fixtures/auto_config_detection
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...snapshots/fixtures__config_ignore_patterns__ignore_directory_-c [email protected] b/...snapshots/fixtures__config_ignore_patterns__ignore_directory_-c [email protected]
@@ -15,3 +15,6 @@ working directory: fixtures/config_ignore_patterns/ignore_directory
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...t_alias_rules_-c oxlint-eslint.json test.js -c oxlint-typescript.json [email protected] b/...t_alias_rules_-c oxlint-eslint.json test.js -c oxlint-typescript.json [email protected]
@@ -16,6 +16,9 @@ working directory: fixtures/eslint_and_typescript_alias_rules
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 1 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
 
 ########## 
 arguments: -c oxlint-typescript.json test.js
@@ -32,3 +35,6 @@ working directory: fixtures/eslint_and_typescript_alias_rules
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 1 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/..._vitest_alias_rules_-c oxlint-jest.json test.js -c oxlint-vitest.json [email protected] b/..._vitest_alias_rules_-c oxlint-jest.json test.js -c oxlint-vitest.json [email protected]
@@ -17,6 +17,9 @@ working directory: fixtures/jest_and_vitest_alias_rules
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 1 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
 
 ########## 
 arguments: -c oxlint-vitest.json test.js
@@ -34,3 +37,6 @@ working directory: fixtures/jest_and_vitest_alias_rules
 
 Found 0 warnings and 1 error.
 Finished in <variable>ms on 1 file with 1 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/apps/oxlint/src/snapshots/[email protected] b/apps/oxlint/src/snapshots/[email protected]
@@ -15,3 +15,6 @@ working directory: fixtures/linter
 
 Found 1 warning and 0 errors.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintSucceeded
+----------
diff --git a/...c/snapshots/fixtures__output_formatter_diagnostic_--format=checkstyle [email protected] b/...c/snapshots/fixtures__output_formatter_diagnostic_--format=checkstyle [email protected]
@@ -6,3 +6,6 @@ arguments: --format=checkstyle test.js
 working directory: fixtures/output_formatter_diagnostic
 ----------
 <?xml version="1.0" encoding="utf-8"?><checkstyle version="4.3"><file name="test.js"><error line="5" column="1" severity="error" message="`debugger` statement is not allowed" source="" /><error line="1" column="10" severity="warning" message="Function &apos;foo&apos; is declared but never used." source="" /><error line="1" column="17" severity="warning" message="Parameter &apos;b&apos; is declared but never used. Unused parameters should start with a &apos;_&apos;." source="" /></file></checkstyle>
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/.../src/snapshots/fixtures__output_formatter_diagnostic_--format=default [email protected] b/.../src/snapshots/fixtures__output_formatter_diagnostic_--format=default [email protected]
@@ -34,3 +34,6 @@ working directory: fixtures/output_formatter_diagnostic
 
 Found 2 warnings and 1 error.
 Finished in <variable>ms on 1 file with 97 rules using 1 threads.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...t/src/snapshots/fixtures__output_formatter_diagnostic_--format=github [email protected] b/...t/src/snapshots/fixtures__output_formatter_diagnostic_--format=github [email protected]
@@ -8,3 +8,6 @@ working directory: fixtures/output_formatter_diagnostic
 ::error file=test.js,line=5,endLine=5,col=1,endColumn=10,title=oxlint::`debugger` statement is not allowed
 ::warning file=test.js,line=1,endLine=1,col=10,endColumn=13,title=oxlint::Function 'foo' is declared but never used.
 ::warning file=test.js,line=1,endLine=1,col=17,endColumn=18,title=oxlint::Parameter 'b' is declared but never used. Unused parameters should start with a '_'.
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/...int/src/snapshots/fixtures__output_formatter_diagnostic_--format=json [email protected] b/...int/src/snapshots/fixtures__output_formatter_diagnostic_--format=json [email protected]
@@ -10,3 +10,6 @@ working directory: fixtures/output_formatter_diagnostic
 	{"message": "Function 'foo' is declared but never used.","code": "eslint(no-unused-vars)","severity": "warning","causes": [],"url": "https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-vars.html","help": "Consider removing this declaration.","filename": "test.js","labels": [{"label": "'foo' is declared here","span": {"offset": 9,"length": 3}}],"related": []},
 	{"message": "Parameter 'b' is declared but never used. Unused parameters should start with a '_'.","code": "eslint(no-unused-vars)","severity": "warning","causes": [],"url": "https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-vars.html","help": "Consider removing this parameter.","filename": "test.js","labels": [{"label": "'b' is declared here","span": {"offset": 16,"length": 1}}],"related": []}
 ]
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/.../src/snapshots/fixtures__output_formatter_diagnostic_--format=stylish [email protected] b/.../src/snapshots/fixtures__output_formatter_diagnostic_--format=stylish [email protected]
@@ -12,3 +12,6 @@ working directory: fixtures/output_formatter_diagnostic
   [2m38:9[0m  [31merror[0m  `debugger` statement is not allowed  [2meslint(no-debugger)[0m
 
 [31m✖ 3 problems (1 error, 2 warnings)[0m
+----------
+CLI result: LintFoundErrors
+----------
diff --git a/apps/oxlint/src/tester.rs b/apps/oxlint/src/tester.rs
@@ -57,7 +57,12 @@ impl Tester {
                 format!("working directory: {}\n", relative_dir.to_str().unwrap()).as_bytes(),
             );
             output.extend_from_slice("----------\n".as_bytes());
-            let _ = LintRunner::new(options).with_cwd(self.cwd.clone()).run(&mut output);
+            let result = LintRunner::new(options).with_cwd(self.cwd.clone()).run(&mut output);
+
+            output.extend_from_slice("----------\n".as_bytes());
+            output.extend_from_slice(format!("CLI result: {result:?}\n").as_bytes());
+            output.extend_from_slice("----------\n".as_bytes());
+
             output.push(b'\n');
         }