feat(tests): update test plugins rules

Author: amir-rahmanii

src/modules/jest.mjs

@@ -1,4 +1,5 @@
 import plugin from 'eslint-plugin-jest';
+import globals from 'globals';
 
 import { globs } from '../utils/globs.mjs';
 
@@ -9,6 +10,11 @@ function jest() {
   return {
     files: globs.test,
     plugins: { jest: plugin },
+    languageOptions: {
+      globals: {
+        ...globals.jest,
+      },
+    },
     rules: {
       'jest/consistent-test-it': 'off',
       'jest/expect-expect': 'off',

src/modules/playwright.mjs

@@ -8,19 +8,6 @@ function playwright() {
     plugins: { playwright: plugin },
     files: globs.e2e,
     rules: {
-      'jest/no-standalone-expect': [
-        'error',
-        {
-          additionalTestBlockFunctions: [
-            'test.jestPlaywrightDebug',
-            'it.jestPlaywrightDebug',
-            'test.jestPlaywrightSkip',
-            'it.jestPlaywrightSkip',
-            'test.jestPlaywrightConfig',
-            'it.jestPlaywrightConfig',
-          ],
-        },
-      ],
       'playwright/expect-expect': 'error',
       'playwright/missing-playwright-await': 'error',
       'playwright/max-nested-describe': ['warn', { max: 5 }],
@@ -40,6 +27,7 @@ function playwright() {
       'playwright/no-useless-await': 'error',
       'playwright/no-unsafe-references': 'warn',
       'playwright/no-useless-not': 'warn',
+      'playwright/no-unused-locators': 'warn',
       'playwright/no-wait-for-navigation': 'warn',
       'playwright/no-wait-for-selector': 'warn',
       'playwright/no-wait-for-timeout': 'warn',

src/modules/tests.mjs

@@ -1,4 +1,3 @@
-import plugin from '@vitest/eslint-plugin';
 import globals from 'globals';
 
 import { predicate } from '../utils/conditions.mjs';
@@ -8,28 +7,20 @@ import { globs } from '../utils/globs.mjs';
  * @param { import('..').Options } options
  * @return { import('eslint').Linter.Config }
  */
+
 function tests(options = {}) {
   return {
-    plugins: { vitest: plugin },
     files: [...globs.test, ...globs.e2e],
     languageOptions: {
-      globals: { ...globals['shared-node-browser'], ...globals.jest },
+      globals: { ...globals['shared-node-browser'] },
     },
     rules: {
-      'vitest/padding-around-after-all-blocks': 'warn',
-      'vitest/padding-around-after-each-blocks': 'warn',
-      'vitest/padding-around-before-all-blocks': 'warn',
-      'vitest/padding-around-before-each-blocks': 'warn',
-      'vitest/padding-around-describe-blocks': 'warn',
-      'vitest/padding-around-expect-groups': 'warn',
-      'vitest/padding-around-test-blocks': 'warn',
-
       'max-lines-per-function': 'off',
       'no-sparse-arrays': 'off',
       'no-empty-function': 'off',
 
       ...predicate(options.react, {
-        'react/jsx-no-constructed-context-values': 'off',
+        '@eslint-react/no-unstable-context-value': 'off',
       }),
 
       ...predicate(options.typescript, {

src/modules/vitest.mjs

@@ -58,6 +58,15 @@ function vitest() {
       'vitest/no-standalone-expect': 'warn',
       'vitest/no-test-prefixes': 'warn',
       'vitest/no-test-return-statement': 'warn',
+
+      'vitest/padding-around-after-all-blocks': 'warn',
+      'vitest/padding-around-after-each-blocks': 'warn',
+      'vitest/padding-around-before-all-blocks': 'warn',
+      'vitest/padding-around-before-each-blocks': 'warn',
+      'vitest/padding-around-describe-blocks': 'warn',
+      'vitest/padding-around-expect-groups': 'warn',
+      'vitest/padding-around-test-blocks': 'warn',
+
       'vitest/prefer-called-once': 'warn',
       'vitest/prefer-called-times': 'off',
       'vitest/prefer-called-with': 'warn',
@@ -88,11 +97,11 @@ function vitest() {
       'vitest/require-local-test-context-for-concurrent-snapshots': 'warn',
       'vitest/require-mock-type-parameters': 'off',
       'vitest/require-to-throw-message': 'warn',
-      'vitest/require-top-level-describe': 'warn',
+      'vitest/require-top-level-describe': 'off',
       'vitest/valid-describe-callback': 'warn',
       'vitest/valid-expect': 'warn',
       'vitest/valid-expect-in-promise': 'warn',
-      'vitest/valid-title': 'warn',
+      'vitest/valid-title': ['warn', { allowArguments: true }],
       'vitest/warn-todo': 'warn',
     },
   };