test: Expound upon environment tests

rschristian · rschristian · commit 2a548a9e6614 · 2025-04-17T23:58:09.000-05:00
diff --git a/package.json b/package.json
@@ -37,14 +37,15 @@
     "stack-trace": "^1.0.0-pre2"
   },
   "devDependencies": {
+    "@cloudflare/vite-plugin": "^1.0.8",
     "@types/node": "^20.11.16",
     "dedent": "^1.5.3",
     "magic-string": "^0.30.6",
     "prettier": "^2.8.8",
     "prettier-config-rschristian": "^0.1.1",
     "tmp-promise": "^3.0.3",
     "uvu": "^0.5.6",
-    "vite": "^5.0.12"
+    "vite": "^6.3.1"
   },
   "prettier": "prettier-config-rschristian"
 }
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/tests/environments.test.js b/tests/environments.test.js
@@ -0,0 +1,44 @@
+import { test } from 'uvu';
+import * as assert from 'uvu/assert';
+
+import { setupTest, teardownTest, loadFixture, viteBuild, viteBuildCli } from './lib/lifecycle.js';
+
+let env;
+test.before.each(async () => {
+    env = await setupTest();
+});
+
+test.after.each(async () => {
+    await teardownTest(env);
+});
+
+test('Should skip plugin during SSR build', async () => {
+    await loadFixture('environments/ssr', env);
+
+    let message = '';
+    try {
+        await viteBuild(env.tmp.path);
+    } catch (error) {
+        message = error.message;
+    }
+
+    assert.match(message, '');
+});
+
+test('Should skip plugin during SSR build (CLI)', async () => {
+    await loadFixture('simple', env);
+    const output = await viteBuildCli(env.tmp.path, ['--ssr', 'src/index.js']);
+    await output.done;
+
+    assert.equal(output.stderr, []);
+});
+
+test('Should skip plugin during non-client environment build (Cloudflare plugin)', async () => {
+    await loadFixture('environments/cloudflare', env);
+    const output = await viteBuildCli(env.tmp.path);
+    await output.done;
+
+    assert.equal(output.stderr, []);
+});
+
+test.run();
diff --git a/tests/fixtures/environments/cloudflare/index.html b/tests/fixtures/environments/cloudflare/index.html
@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html lang="en">
+    <head>
+        <meta charset="UTF-8" />
+    </head>
+    <body>
+        <script prerender type="module" src="/src/index.js"></script>
+    </body>
+</html>
diff --git a/tests/fixtures/environments/cloudflare/src/index.js b/tests/fixtures/environments/cloudflare/src/index.js
@@ -0,0 +1,3 @@
+export async function prerender() {
+    return `<h1>Simple Test Result</h1>`;
+}
diff --git a/tests/fixtures/environments/cloudflare/src/server.js b/tests/fixtures/environments/cloudflare/src/server.js
@@ -0,0 +1 @@
+console.log("Server is running...");
diff --git a/tests/fixtures/environments/cloudflare/vite.config.js b/tests/fixtures/environments/cloudflare/vite.config.js
@@ -0,0 +1,9 @@
+import { defineConfig } from 'vite';
+import { vitePrerenderPlugin } from 'vite-prerender-plugin';
+import { cloudflare } from '@cloudflare/vite-plugin';
+
+console.log('Config is used');
+
+export default defineConfig({
+    plugins: [cloudflare(), vitePrerenderPlugin()],
+});
diff --git a/tests/fixtures/environments/cloudflare/wrangler.toml b/tests/fixtures/environments/cloudflare/wrangler.toml
@@ -0,0 +1,4 @@
+name = "my-worker"
+compatibility_date = "2023-03-01"
+
+main = "src/server.js"
diff --git a/tests/fixtures/environments/ssr/index.html b/tests/fixtures/environments/ssr/index.html
@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html lang="en">
+    <head>
+        <meta charset="UTF-8" />
+    </head>
+    <body>
+        <script prerender type="module" src="/src/index.js"></script>
+    </body>
+</html>
diff --git a/tests/fixtures/environments/ssr/src/index.js b/tests/fixtures/environments/ssr/src/index.js
@@ -0,0 +1,3 @@
+export async function prerender() {
+    return `<h1>Simple Test Result</h1>`;
+}
diff --git a/tests/fixtures/environments/ssr/vite.config.js b/tests/fixtures/environments/ssr/vite.config.js
@@ -0,0 +1,12 @@
+import { defineConfig } from 'vite';
+import { vitePrerenderPlugin } from 'vite-prerender-plugin';
+
+export default defineConfig({
+    build: {
+        ssr: true,
+        rollupOptions: {
+            input: 'src/index.js'
+        }
+    },
+    plugins: [vitePrerenderPlugin()],
+});
diff --git a/tests/index.test.js b/tests/index.test.js
@@ -90,35 +90,6 @@ test('Should support custom output filenames', async () => {
             plugins: [vitePrerenderPlugin()],
         });
     `);
-    await viteBuild(env.tmp.path);
-
-    let message = '';
-    try {
-        await viteBuild(env.tmp.path);
-    } catch (error) {
-        message = error.message;
-    }
-
-    assert.match(message, '');
-});
-
-test('Should skip plugin during SSR build', async () => {
-    await loadFixture('simple', env);
-    await writeConfig(env.tmp.path, `
-        import { defineConfig } from 'vite';
-        import { vitePrerenderPlugin } from 'vite-prerender-plugin';
-
-        export default defineConfig({
-            build: {
-                ssr: true,
-                rollupOptions: {
-                    input: 'src/index.js'
-                }
-            },
-            plugins: [vitePrerenderPlugin()],
-        });
-    `);
-    await viteBuild(env.tmp.path);
 
     let message = '';
     try {
diff --git a/tests/lib/lifecycle.js b/tests/lib/lifecycle.js
@@ -50,9 +50,10 @@ export async function viteBuild(cwd) {
 
 /**
  * @param {string} cwd
+ * @param {string[]} flags
  */
-export async function viteBuildCli(cwd) {
-    const child = childProcess.spawn('node', ['node_modules/vite/bin/vite.js', 'build'], {
+export async function viteBuildCli(cwd, flags = []) {
+    const child = childProcess.spawn('node', ['node_modules/vite/bin/vite.js', 'build', ...flags], {
         cwd,
     });
 
diff --git a/tests/lib/utils.js b/tests/lib/utils.js
@@ -35,6 +35,9 @@ export async function copyDependencies(cwd) {
     await copyNodeModule('source-map');
     await copyNodeModule('stack-trace');
     await copyNodeModule('kolorist');
+
+    // Dependencies used by tests
+    await copyNodeModule('@cloudflare/vite-plugin');
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+export async function prerender() {`
	`2`	+ return `<h1>Simple Test Result</h1>`;
	`3`	`+}`