test: Update Vitest (#16682)

This PR updates Vitest to the latest version which has much better
(non-flickering) output and also adds a custom config for the Node
integration tests. This custom config was mainly to override the `pool:
'threads'` option which improves performance significantly when you
don't need process isolation.

Author: timfish

dev-packages/e2e-tests/test-applications/solid-solidrouter/package.json

@@ -22,7 +22,7 @@
     "solid-devtools": "^0.29.2",
     "tailwindcss": "^3.4.1",
     "vite": "^5.4.11",
-    "vite-plugin-solid": "^2.8.2"
+    "vite-plugin-solid": "^2.11.6"
   },
   "dependencies": {
     "@solidjs/router": "^0.13.5",

dev-packages/e2e-tests/test-applications/solid/package.json

@@ -22,7 +22,7 @@
     "solid-devtools": "^0.29.2",
     "tailwindcss": "^3.4.1",
     "vite": "^5.4.11",
-    "vite-plugin-solid": "^2.8.2"
+    "vite-plugin-solid": "^2.11.6"
   },
   "dependencies": {
     "solid-js": "^1.8.18",

dev-packages/e2e-tests/test-applications/solidstart-dynamic-import/package.json

@@ -28,7 +28,7 @@
     "typescript": "^5.4.5",
     "vinxi": "^0.4.0",
     "vite": "^5.4.10",
-    "vite-plugin-solid": "^2.10.2",
+    "vite-plugin-solid": "^2.11.6",
     "vitest": "^1.5.0"
   },
   "overrides": {

dev-packages/e2e-tests/test-applications/solidstart-spa/package.json

@@ -28,7 +28,7 @@
     "typescript": "^5.4.5",
     "vinxi": "^0.4.0",
     "vite": "^5.4.11",
-    "vite-plugin-solid": "^2.10.2",
+    "vite-plugin-solid": "^2.11.6",
     "vitest": "^1.5.0"
   },
   "overrides": {

dev-packages/e2e-tests/test-applications/solidstart-top-level-import/package.json

@@ -28,7 +28,7 @@
     "typescript": "^5.4.5",
     "vinxi": "^0.4.0",
     "vite": "^5.4.11",
-    "vite-plugin-solid": "^2.10.2",
+    "vite-plugin-solid": "^2.11.6",
     "vitest": "^1.5.0"
   },
   "overrides": {

dev-packages/e2e-tests/test-applications/solidstart/package.json

@@ -28,7 +28,7 @@
     "typescript": "^5.4.5",
     "vinxi": "^0.4.0",
     "vite": "^5.4.11",
-    "vite-plugin-solid": "^2.10.2",
+    "vite-plugin-solid": "^2.11.6",
     "vitest": "^1.5.0"
   },
   "overrides": {

dev-packages/node-integration-tests/vite.config.ts

@@ -11,5 +11,21 @@ export default defineConfig({
     },
     include: ['./**/test.ts'],
     testTimeout: 15000,
+    // Ensure we can see debug output when DEBUG=true
+    ...(process.env.DEBUG
+      ? {
+          disableConsoleIntercept: true,
+          silent: false,
+        }
+      : {}),
+    // By default Vitest uses child processes to run tests but all our tests
+    // already run in their own processes. We use threads instead because the
+    // overhead is significantly less.
+    pool: 'threads',
+    reporters: process.env.DEBUG
+      ? ['default', { summary: false }]
+      : process.env.GITHUB_ACTIONS
+        ? ['dot', 'github-actions']
+        : ['verbose'],
   },
 });

package.json

@@ -115,7 +115,7 @@
     "@size-limit/webpack": "~11.1.6",
     "@types/jsdom": "^21.1.6",
     "@types/node": "^18.19.1",
-    "@vitest/coverage-v8": "^2.1.8",
+    "@vitest/coverage-v8": "^3.2.4",
     "deepmerge": "^4.2.2",
     "downlevel-dts": "~0.11.0",
     "es-check": "^7.2.1",
@@ -134,7 +134,7 @@
     "sucrase": "^3.35.0",
     "ts-node": "10.9.1",
     "typescript": "~5.0.0",
-    "vitest": "^2.1.8",
+    "vitest": "^3.2.4",
     "yalc": "^1.0.0-pre.53",
     "yarn-deduplicate": "6.0.2"
   },

packages/browser/vite.config.ts

@@ -5,5 +5,9 @@ export default defineConfig({
   ...baseConfig,
   test: {
     ...baseConfig.test,
+    // Vitest 3 mocks all timers which broke some tests
+    fakeTimers: {
+      toFake: ['setTimeout', 'clearTimeout', 'setInterval', 'clearInterval', 'setImmediate', 'clearImmediate', 'Date'],
+    },
   },
 });

packages/react-router/test/server/createSentryHandleRequest.test.ts

@@ -242,6 +242,7 @@ describe('createSentryHandleRequest', () => {
 
     await handleRequest(mockRequest, 200, mockResponseHeaders, mockRouterContext, mockLoadContext);
 
+    // eslint-disable-next-line @typescript-eslint/unbound-method
     expect(mockResponseHeaders.set).toHaveBeenCalledWith('Content-Type', 'text/html');
   });