Move the unenv tests from the preset to wrangler e2e

Author: vicb

.changeset/gold-beans-tie.md

@@ -0,0 +1,7 @@
+---
+"@cloudflare/vite-plugin": patch
+"@cloudflare/unenv-preset": patch
+"wrangler": patch
+---
+
+wrangler and vite-plugin now depend upon the latest version of unenv-preset

packages/unenv-preset/README.md

@@ -18,3 +18,7 @@ const { alias, inject, external, polyfill } = env;
 ```
 
 See the unenv [README](https://github.com/unjs/unenv/blob/main/README.md) for more details.
+
+## Tests
+
+This package is tested via wrangler e2e tests.

packages/unenv-preset/package.json

@@ -40,19 +40,14 @@
 	"scripts": {
 		"build": "unbuild",
 		"check:lint": "eslint",
-		"check:type": "tsc --noEmit",
-		"test:ci": "vitest run",
-		"test:watch": "vitest"
+		"check:type": "tsc --noEmit"
 	},
 	"devDependencies": {
 		"@types/debug": "4.1.12",
 		"@types/node-unenv": "npm:@types/node@^22.14.0",
 		"debug": "4.4.1",
 		"typescript": "catalog:default",
-		"unbuild": "^3.2.0",
-		"undici": "catalog:default",
-		"vitest": "catalog:default",
-		"wrangler": "workspace:*"
+		"unbuild": "^3.2.0"
 	},
 	"peerDependencies": {
 		"unenv": "2.0.0-rc.17",

packages/unenv-preset/tests/index.test.ts

@@ -42,7 +42,7 @@
 		"test:watch": "vitest"
 	},
 	"dependencies": {
-		"@cloudflare/unenv-preset": "2.3.3",
+		"@cloudflare/unenv-preset": "workspace:*",
 		"@mjackson/node-fetch-server": "^0.6.1",
 		"@rollup/plugin-replace": "^6.0.1",
 		"get-port": "^7.1.0",

packages/vite-plugin-cloudflare/package.json

@@ -6,5 +6,5 @@
 		"jsx": "preserve" // not used, but required for import of types (which transitively import jsx)
 	},
 	"include": ["vitest.config.mts", "**/*.ts", "../src/*.d.ts"],
-	"exclude": []
+	"exclude": ["unenv-preset"]
 }

packages/wrangler/e2e/tsconfig.json

@@ -0,0 +1,20 @@
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const thisPath = path.dirname(fileURLToPath(import.meta.url));
+
+/** Path to the worker script */
+export const workerScript = path.join(thisPath, "/worker/index.ts");
+
+/**
+ * @returns The current date formatted as YYYY-MM-DD
+ */
+export function getYMDDate(): string {
+	const date = new Date();
+	const year = date.getFullYear();
+	// Months are 0-indexed
+	const month = (date.getMonth() + 1).toString().padStart(2, "0");
+	const day = date.getDate().toString().padStart(2, "0");
+
+	return `${year}-${month}-${day}`;
+}

packages/wrangler/e2e/unenv-preset/helper.ts

@@ -0,0 +1,51 @@
+/**
+ * Test building and serving locally a worker in nodejs_compat mode.
+ */
+
+import dedent from "ts-dedent";
+import { fetch } from "undici";
+import { afterAll, beforeAll, describe, expect, test, vi } from "vitest";
+import { WranglerE2ETestHelper } from "../helpers/e2e-wrangler-test";
+import { generateResourceName } from "../helpers/generate-resource-name";
+import { getYMDDate, workerScript } from "./helper";
+import { TESTS } from "./worker/index";
+import type { WranglerLongLivedCommand } from "../helpers/wrangler";
+
+describe(`@cloudflare/unenv-preset local tests`, () => {
+	let url: string;
+	let wrangler: WranglerLongLivedCommand;
+
+	beforeAll(async () => {
+		const helper = new WranglerE2ETestHelper();
+		await helper.seed({
+			"wrangler.jsonc": dedent`{
+				"name": "${generateResourceName()}",
+				"main": "${workerScript}",
+				"compatibility_date": "${getYMDDate()}",
+				"compatibility_flags": ["nodejs_compat"],
+				"vars": {
+					"DEBUG": "example"
+				}
+			}`,
+		});
+
+		wrangler = helper.runLongLived(`wrangler dev`, {
+			stopOnTestFinished: false,
+		});
+		url = (await wrangler.waitForReady()).url;
+	});
+
+	afterAll(async () => {
+		await wrangler.stop();
+	});
+
+	test.for(Object.keys(TESTS))("%s", { timeout: 20_000 }, async (testName) => {
+		// Retries the callback until it succeeds or times out.
+		// Useful for the i.e. DNS tests where underlying requests might error/timeout.
+		await vi.waitFor(async () => {
+			const response = await fetch(`${url}/${testName}`);
+			const body = await response.text();
+			expect(body).toMatch("OK!");
+		});
+	});
+});

packages/wrangler/e2e/unenv-preset/local.test.ts

@@ -0,0 +1,74 @@
+/**
+ * Test building and serving a worker in nodejs_compat mode.
+ * This makes sure the prod workerd is is compatible with unenv.
+ */
+
+import assert from "node:assert";
+import dedent from "ts-dedent";
+import { fetch } from "undici";
+import { afterAll, beforeAll, describe, expect, test, vi } from "vitest";
+import { CLOUDFLARE_ACCOUNT_ID } from "../helpers/account-id";
+import { WranglerE2ETestHelper } from "../helpers/e2e-wrangler-test";
+import { generateResourceName } from "../helpers/generate-resource-name";
+import { retry } from "../helpers/retry";
+import { getYMDDate, workerScript } from "./helper";
+import { TESTS } from "./worker/index";
+
+describe.skipIf(!CLOUDFLARE_ACCOUNT_ID)(
+	`@cloudflare/unenv-preset remote tests`,
+	() => {
+		let url: string;
+		let helper: WranglerE2ETestHelper;
+
+		beforeAll(async () => {
+			helper = new WranglerE2ETestHelper();
+			await helper.seed({
+				"wrangler.jsonc": dedent`{
+				"name": "${generateResourceName()}",
+				"main": "${workerScript}",
+				"compatibility_date": "${getYMDDate()}",
+				"compatibility_flags": ["nodejs_compat"],
+				"vars": {
+					"DEBUG": "example"
+				}
+			}`,
+			});
+
+			const { stdout } = await helper.run(`wrangler deploy`);
+
+			const match = stdout.match(
+				/(?<url>https:\/\/tmp-e2e-.+?\..+?\.workers\.dev)/
+			);
+			assert(match?.groups);
+			url = match.groups.url;
+
+			const { text } = await retry(
+				(s) => s.status !== 200,
+				async () => {
+					const r = await fetch(`${url}/ping`);
+					return { text: await r.text(), status: r.status };
+				}
+			);
+			expect(text).toEqual("pong");
+		}, 30_000);
+
+		afterAll(async () => {
+			// clean up user Worker
+			await helper.run(`wrangler delete`);
+		});
+
+		test.for(Object.keys(TESTS))(
+			"%s",
+			{ timeout: 20_000 },
+			async (testName) => {
+				// Retries the callback until it succeeds or times out.
+				// Useful for the i.e. DNS tests where underlying requests might error/timeout.
+				await vi.waitFor(async () => {
+					const response = await fetch(`${url}/${testName}`);
+					const body = await response.text();
+					expect(body).toMatch("OK!");
+				});
+			}
+		);
+	}
+);

packages/wrangler/e2e/unenv-preset/remote.test.ts

@@ -0,0 +1,9 @@
+{
+	"extends": "@cloudflare/workers-tsconfig/tsconfig.json",
+	"compilerOptions": {
+		"types": ["node"],
+		"module": "nodenext",
+		"moduleResolution": "nodenext"
+	},
+	"include": ["**/*.ts"]
+}