feat(linter): mark jest/vitest rules as recommended (#27672)

Copying recommended state from upstream Jest/Vitest plugin.

crates/oxc_linter/src/rules/jest/expect_expect.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ExpectExpect,
     jest,
     correctness,
+    recommended,
     config = ExpectExpectConfig,
     docs = DOCUMENTATION,
     version = "0.0.12",

crates/oxc_linter/src/rules/jest/no_alias_methods.rs

@@ -54,6 +54,7 @@ declare_oxc_lint!(
     NoAliasMethods,
     jest,
     style,
+    recommended,
     fix,
     version = "0.0.12",
     short_description = "Enforces Jest's canonical matcher names instead of aliases.",

crates/oxc_linter/src/rules/jest/no_commented_out_tests.rs

@@ -9,7 +9,14 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoCommentedOutTests;
 
-declare_oxc_lint!(NoCommentedOutTests, jest, suspicious, docs = DOCUMENTATION, version = "0.0.8",);
+declare_oxc_lint!(
+    NoCommentedOutTests,
+    jest,
+    suspicious,
+    recommended,
+    docs = DOCUMENTATION,
+    version = "0.0.8",
+);
 
 impl Rule for NoCommentedOutTests {
     fn run_once(&self, ctx: &LintContext) {

crates/oxc_linter/src/rules/jest/no_conditional_expect.rs

@@ -10,7 +10,14 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoConditionalExpect;
 
-declare_oxc_lint!(NoConditionalExpect, jest, correctness, docs = DOCUMENTATION, version = "0.0.12",);
+declare_oxc_lint!(
+    NoConditionalExpect,
+    jest,
+    correctness,
+    recommended,
+    docs = DOCUMENTATION,
+    version = "0.0.12",
+);
 
 impl Rule for NoConditionalExpect {
     fn run_on_jest_node<'a, 'c>(

crates/oxc_linter/src/rules/jest/no_deprecated_functions.rs

@@ -111,6 +111,7 @@ declare_oxc_lint!(
     NoDeprecatedFunctions,
     jest,
     style,
+    recommended,
     fix,
     config = NoDeprecatedFunctionsConfig,
     version = "0.0.18",

crates/oxc_linter/src/rules/jest/no_disabled_tests.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoDisabledTests,
     jest,
     correctness,
+    recommended,
     docs = DOCUMENTATION,
     version = "0.0.7",
     short_description = SHORT_DESCRIPTION,

crates/oxc_linter/src/rules/jest/no_done_callback.rs

@@ -76,6 +76,7 @@ declare_oxc_lint!(
     jest,
     // TODO: add suggestion (see jest-community/eslint-plugin-jest#586)
     style,
+    recommended,
     pending,
     version = "0.0.13",
     short_description = "This rule checks the function parameter of hooks & tests for use of the done argument, suggesting you return a promise instead.",

crates/oxc_linter/src/rules/jest/no_export.rs

@@ -42,6 +42,7 @@ declare_oxc_lint!(
     NoExport,
     jest,
     correctness,
+    recommended,
     version = "0.0.13",
     short_description = "Prevents using exports if a file has one or more tests in it.",
 );

crates/oxc_linter/src/rules/jest/no_focused_tests.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoFocusedTests,
     jest,
     correctness,
+    recommended,
     suggestion,
     docs = DOCUMENTATION,
     version = "0.0.8",

crates/oxc_linter/src/rules/jest/no_identical_title.rs

@@ -9,7 +9,14 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoIdenticalTitle;
 
-declare_oxc_lint!(NoIdenticalTitle, jest, style, docs = DOCUMENTATION, version = "0.0.14",);
+declare_oxc_lint!(
+    NoIdenticalTitle,
+    jest,
+    style,
+    recommended,
+    docs = DOCUMENTATION,
+    version = "0.0.14",
+);
 
 impl Rule for NoIdenticalTitle {
     fn run_once(&self, ctx: &LintContext) {

crates/oxc_linter/src/rules/jest/no_interpolation_in_snapshots.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoInterpolationInSnapshots,
     jest,
     style,
+    recommended,
     docs = DOCUMENTATION,
     version = "0.0.13",
 );

crates/oxc_linter/src/rules/jest/no_jasmine_globals.rs

@@ -65,6 +65,7 @@ declare_oxc_lint!(
     NoJasmineGlobals,
     jest,
     style,
+    recommended,
     conditional_fix,
     version = "0.0.13",
     short_description = "This rule reports on any usage of Jasmine globals, which are not ported to Jest, and suggests alternatives from Jest's own API.",

crates/oxc_linter/src/rules/jest/no_mocks_import.rs

@@ -12,6 +12,7 @@ declare_oxc_lint!(
     NoMocksImport,
     jest,
     style,
+    recommended,
     docs = SharedNoMocksImport::DOCUMENTATION,
     version = "0.0.13",
 );

crates/oxc_linter/src/rules/jest/no_standalone_expect.rs

@@ -13,6 +13,7 @@ declare_oxc_lint!(
     NoStandaloneExpect,
     jest,
     correctness,
+    recommended,
     config = SharedNoStandaloneExpect::NoStandaloneExpectConfig,
     docs = SharedNoStandaloneExpect::DOCUMENTATION,
     version = "0.0.13",

crates/oxc_linter/src/rules/jest/no_test_prefixes.rs

@@ -12,7 +12,15 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoTestPrefixes;
 
-declare_oxc_lint!(NoTestPrefixes, jest, style, fix, docs = DOCUMENTATION, version = "0.0.7",);
+declare_oxc_lint!(
+    NoTestPrefixes,
+    jest,
+    style,
+    recommended,
+    fix,
+    docs = DOCUMENTATION,
+    version = "0.0.7",
+);
 
 impl Rule for NoTestPrefixes {
     fn run_on_jest_node<'a, 'c>(

crates/oxc_linter/src/rules/jest/valid_describe_callback.rs

@@ -48,6 +48,7 @@ declare_oxc_lint!(
     ValidDescribeCallback,
     jest,
     correctness,
+    recommended,
     version = "0.0.8",
     short_description = "Enforce that the second argument to `describe()` is a valid callback function.",
 );

crates/oxc_linter/src/rules/jest/valid_expect.rs

@@ -13,6 +13,7 @@ declare_oxc_lint!(
     ValidExpect,
     jest,
     correctness,
+    recommended,
     suggestion,
     config = ValidExpectConfig,
     docs = DOCUMENTATION,

crates/oxc_linter/src/rules/jest/valid_expect_in_promise.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ValidExpectInPromise,
     jest,
     correctness,
+    recommended,
     docs = DOCUMENTATION,
     version = "1.60.0",
 );

crates/oxc_linter/src/rules/jest/valid_title.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ValidTitle,
     jest,
     correctness,
+    recommended,
     conditional_fix,
     // TODO: Replace this with an actual config struct. This is a dummy value to
     // indicate that this rule has configuration and avoid errors.

crates/oxc_linter/src/rules/vitest/expect_expect.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ExpectExpect,
     vitest,
     correctness,
+    recommended,
     config = ExpectExpectConfig,
     docs = DOCUMENTATION,
     version = "0.0.12",

crates/oxc_linter/src/rules/vitest/no_commented_out_tests.rs

@@ -9,7 +9,14 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoCommentedOutTests;
 
-declare_oxc_lint!(NoCommentedOutTests, vitest, suspicious, docs = DOCUMENTATION, version = "0.0.8",);
+declare_oxc_lint!(
+    NoCommentedOutTests,
+    vitest,
+    suspicious,
+    recommended,
+    docs = DOCUMENTATION,
+    version = "0.0.8",
+);
 
 impl Rule for NoCommentedOutTests {
     fn run_once(&self, ctx: &LintContext) {

crates/oxc_linter/src/rules/vitest/no_conditional_expect.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoConditionalExpect,
     vitest,
     correctness,
+    recommended,
     docs = DOCUMENTATION,
     version = "0.0.12",
 );

crates/oxc_linter/src/rules/vitest/no_disabled_tests.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoDisabledTests,
     vitest,
     correctness,
+    recommended,
     docs = DOCUMENTATION,
     version = "0.0.7",
     short_description = SHORT_DESCRIPTION,

crates/oxc_linter/src/rules/vitest/no_focused_tests.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoFocusedTests,
     vitest,
     correctness,
+    recommended,
     suggestion,
     docs = DOCUMENTATION,
     version = "0.0.8",

crates/oxc_linter/src/rules/vitest/no_identical_title.rs

@@ -9,7 +9,14 @@ use crate::{
 #[derive(Debug, Default, Clone)]
 pub struct NoIdenticalTitle;
 
-declare_oxc_lint!(NoIdenticalTitle, vitest, style, docs = DOCUMENTATION, version = "0.0.14",);
+declare_oxc_lint!(
+    NoIdenticalTitle,
+    vitest,
+    style,
+    recommended,
+    docs = DOCUMENTATION,
+    version = "0.0.14",
+);
 
 impl Rule for NoIdenticalTitle {
     fn run_once(&self, ctx: &LintContext) {

crates/oxc_linter/src/rules/vitest/no_import_node_test.rs

@@ -48,6 +48,7 @@ declare_oxc_lint!(
     NoImportNodeTest,
     vitest,
     style,
+    recommended,
     suggestion,
     version = "0.7.0",
     short_description = "Disallow importing from `node:test`, preferring `vitest` instead.",

crates/oxc_linter/src/rules/vitest/no_interpolation_in_snapshots.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoInterpolationInSnapshots,
     vitest,
     style,
+    recommended,
     docs = DOCUMENTATION,
     version = "0.0.13",
 );

crates/oxc_linter/src/rules/vitest/no_mocks_import.rs

@@ -11,6 +11,7 @@ declare_oxc_lint!(
     NoMocksImport,
     vitest,
     style,
+    recommended,
     docs = SharedNoMocksImport::DOCUMENTATION,
     version = "0.0.13",
 );

crates/oxc_linter/src/rules/vitest/no_standalone_expect.rs

@@ -12,6 +12,7 @@ declare_oxc_lint!(
     NoStandaloneExpect,
     vitest,
     correctness,
+    recommended,
     config = SharedNoStandaloneExpect::NoStandaloneExpectConfig,
     docs = SharedNoStandaloneExpect::DOCUMENTATION,
     version = "0.0.13",

crates/oxc_linter/src/rules/vitest/no_unneeded_async_expect_function.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     NoUnneededAsyncExpectFunction,
     vitest,
     style,
+    recommended,
     fix,
     docs = DOCUMENTATION,
     version = "1.39.0",

crates/oxc_linter/src/rules/vitest/prefer_called_exactly_once_with.rs

@@ -187,6 +187,7 @@ declare_oxc_lint!(
     PreferCalledExactlyOnceWith,
     vitest,
     style,
+    recommended,
     dangerous_fix,
     version = "1.58.0",
     short_description = "It checks when a target is asserted with both `toHaveBeenCalledOnce` and `toHaveBeenCalledWith` instead of `toHaveBeenCalledExactlyOnceWith`.",

crates/oxc_linter/src/rules/vitest/require_local_test_context_for_concurrent_snapshots.rs

@@ -86,6 +86,7 @@ declare_oxc_lint!(
     RequireLocalTestContextForConcurrentSnapshots,
     vitest,
     correctness,
+    recommended,
     version = "0.8.0",
     short_description = "The rule is intended to ensure that concurrent snapshot tests are executed within a properly configured local test context.",
 );

crates/oxc_linter/src/rules/vitest/valid_describe_callback.rs

@@ -43,6 +43,7 @@ declare_oxc_lint!(
     ValidDescribeCallback,
     vitest,
     correctness,
+    recommended,
     version = "0.0.8",
     short_description = "Enforce that the second argument to `describe()` is a valid callback function.",
 );

crates/oxc_linter/src/rules/vitest/valid_expect.rs

@@ -19,6 +19,7 @@ declare_oxc_lint!(
     ValidExpect,
     vitest,
     correctness,
+    recommended,
     suggestion,
     config = ValidExpectConfig,
     docs = DOCUMENTATION,

crates/oxc_linter/src/rules/vitest/valid_expect_in_promise.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ValidExpectInPromise,
     vitest,
     correctness,
+    recommended,
     docs = DOCUMENTATION,
     version = "1.60.0",
 );

crates/oxc_linter/src/rules/vitest/valid_title.rs

@@ -14,6 +14,7 @@ declare_oxc_lint!(
     ValidTitle,
     vitest,
     correctness,
+    recommended,
     conditional_fix,
     // TODO: Replace this with an actual config struct. This is a dummy value to
     // indicate that this rule has configuration and avoid errors.