Fix Vite version checks

Author: timfish

packages/integration-tests-next/fixtures/vite4/utils.ts

@@ -6,7 +6,6 @@ import { TestContext, test as vitestTest } from "vitest";
 import { execSync } from "node:child_process";
 
 const cwd = dirname(fileURLToPath(import.meta.url));
-const NODE_MAJOR_VERSION = parseInt(process.versions.node.split(".")[0] || "0", 10);
 
 type TestCallback = (props: {
   outDir: string;
@@ -26,37 +25,31 @@ export function test(url: string, callback: TestCallback) {
   // Clear the output directory before running the test
   rmSync(outDir, { recursive: true, force: true });
 
-  // Vite v8 requires Node 20+
-  if (NODE_MAJOR_VERSION < 20) {
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
-    vitestTest.skip(testName);
-  } else {
-    vitestTest(`Vite v4 > ${testName}`, (ctx) =>
-      callback({
-        outDir,
-        runBundler: (env) =>
-          runBundler(
-            `vite build --config ${testName}.config.ts`,
-            {
-              cwd,
-              env: {
-                ...process.env,
-                ...env,
-              },
-            },
-            outDir
-          ),
-        readOutputFiles: () => readAllFiles(outDir),
-        runFileInNode: (file) => {
-          const fullPath = join(outDir, file);
-          return execSync(`node ${fullPath}`, {
+  vitestTest(`Vite v4 > ${testName}`, (ctx) =>
+    callback({
+      outDir,
+      runBundler: (env) =>
+        runBundler(
+          `vite build --config ${testName}.config.ts`,
+          {
             cwd,
-            env: { ...process.env, NO_COLOR: "1", FORCE_COLOR: "0" },
-          }).toString();
-        },
-        createTempDir: () => createTempDir(),
-        ctx,
-      })
-    );
-  }
+            env: {
+              ...process.env,
+              ...env,
+            },
+          },
+          outDir
+        ),
+      readOutputFiles: () => readAllFiles(outDir),
+      runFileInNode: (file) => {
+        const fullPath = join(outDir, file);
+        return execSync(`node ${fullPath}`, {
+          cwd,
+          env: { ...process.env, NO_COLOR: "1", FORCE_COLOR: "0" },
+        }).toString();
+      },
+      createTempDir: () => createTempDir(),
+      ctx,
+    })
+  );
 }

packages/integration-tests-next/fixtures/vite7/utils.ts

@@ -26,7 +26,7 @@ export function test(url: string, callback: TestCallback) {
   // Clear the output directory before running the test
   rmSync(outDir, { recursive: true, force: true });
 
-  // Vite v8 requires Node 20+
+  // Vite v7 requires Node 20+
   if (NODE_MAJOR_VERSION < 20) {
     // eslint-disable-next-line @typescript-eslint/no-empty-function
     vitestTest.skip(testName);