update react and vitest eslint plugins (#4006)

Author: nstepien

eslint.config.js

@@ -917,6 +917,24 @@ copy(
       '@eslint-react/no-create-ref': 0,
 
       // https://github.com/vitest-dev/eslint-plugin-vitest#rules
+      /*
+// copy all the rules from the rules table for easy pasting
+copy(
+  Iterator.from(
+    document
+      // select rules table
+      .querySelector('.markdown-heading:has(> a[href="#rules"]) ~ markdown-accessiblity-table')
+      // select all rows with a rule
+      .querySelectorAll('tr:has(a)')
+  )
+    // filter out deprecated rules
+    .filter((row) => row.lastElementChild.textContent === '')
+    // map row to rule declaration
+    .map((row) => `'vitest/${row.firstElementChild.textContent}': 1,`)
+    .toArray()
+    .join('\n')
+);
+      */
       'vitest/consistent-each-for': 1,
       'vitest/consistent-test-filename': 0,
       'vitest/consistent-test-it': 1,
@@ -931,7 +949,6 @@ copy(
       'vitest/no-conditional-in-test': 0,
       'vitest/no-conditional-tests': 1,
       'vitest/no-disabled-tests': 0,
-      'vitest/no-done-callback': 1,
       'vitest/no-duplicate-hooks': 1,
       'vitest/no-focused-tests': [1, { fixable: false }],
       'vitest/no-hooks': 0,
@@ -979,7 +996,7 @@ copy(
       'vitest/prefer-strict-equal': 1,
       'vitest/prefer-to-be': 1,
       'vitest/prefer-to-be-falsy': 0,
-      'vitest/prefer-to-be-object': 0,
+      'vitest/prefer-to-be-object': 1,
       'vitest/prefer-to-be-truthy': 0,
       'vitest/prefer-to-contain': 1,
       'vitest/prefer-to-have-been-called-times': 1,
@@ -990,7 +1007,8 @@ copy(
       'vitest/require-hook': 0,
       'vitest/require-local-test-context-for-concurrent-snapshots': 0,
       'vitest/require-mock-type-parameters': 0,
-      'vitest/require-to-throw-message': 0,
+      'vitest/require-test-timeout': 0,
+      'vitest/require-to-throw-message': 1,
       'vitest/require-top-level-describe': 0,
       'vitest/valid-describe-callback': 1,
       'vitest/valid-expect': [1, { alwaysAwait: true }],

package.json

@@ -47,7 +47,7 @@
     "typecheck": "tsc --build"
   },
   "devDependencies": {
-    "@eslint-react/eslint-plugin": "3.0.0-rc.0",
+    "@eslint-react/eslint-plugin": "^3.0.0",
     "@eslint/markdown": "^7.5.1",
     "@faker-js/faker": "^10.3.0",
     "@tanstack/react-router": "^1.166.7",
@@ -58,7 +58,7 @@
     "@vitejs/plugin-react": "^6.0.1",
     "@vitest/browser-playwright": "^4.1.0",
     "@vitest/coverage-istanbul": "^4.1.0",
-    "@vitest/eslint-plugin": "^1.6.11",
+    "@vitest/eslint-plugin": "^1.6.12",
     "clsx": "^2.1.1",
     "ecij": "^0.4.1",
     "eslint": "^10.0.3",

test/browser/column/key.test.ts

@@ -9,6 +9,5 @@ test('key is escaped in query selectors', async () => {
     }
   ];
 
-  // eslint-disable-next-line vitest/no-alias-methods
   await expect(setup({ columns, rows: [] })).resolves.not.toThrow();
 });