[scramjet/runway] add link header tests

Author: velzie

packages/scramjet/packages/runway/src/index.ts

@@ -148,8 +148,18 @@ async function runTest(
 		}
 	}
 
+	let serverResultResolve: (result: TestResult) => void;
+	let serverResult: Promise<TestResult> = new Promise((resolve) => {
+		serverResultResolve = resolve;
+	});
+
 	// Handle basic tests (tests that serve a page and call pass/fail)
-	await test.start();
+	await test.start({
+		pass: async (message?: string, details?: any) =>
+			serverResultResolve({ status: "pass", message, details }),
+		fail: async (message?: string, details?: any) =>
+			serverResultResolve({ status: "fail", message, details }),
+	});
 
 	try {
 		const testUrl = `http://localhost:${test.port}/`;
@@ -158,7 +168,7 @@ async function runTest(
 			(window as any).__runwayNavigate(url);
 		}, testUrl);
 
-		return await waitForResult(timeout);
+		return await Promise.race([waitForResult(timeout), serverResult]);
 	} finally {
 		await test.stop();
 	}

packages/scramjet/packages/runway/src/inspect.ts

@@ -63,7 +63,16 @@ async function main() {
 	// Start all matching tests (only basicTests need servers)
 	for (const test of tests) {
 		if (!test.playwrightFn) {
-			await test.start();
+			await test.start({
+				pass: async (message?: string, details?: any) => {
+					console.log(`\n✅ Test passed${message ? `: ${message}` : ""}`);
+					if (details) console.log("   Details:", details);
+				},
+				fail: async (message?: string, details?: any) => {
+					console.log(`\n❌ Test failed${message ? `: ${message}` : ""}`);
+					if (details) console.log("   Details:", details);
+				},
+			});
 			console.log(
 				`🌐 Test "${test.name}" running at http://localhost:${test.port}/`
 			);

packages/scramjet/packages/runway/src/testcommon.ts

@@ -13,7 +13,10 @@ export type TestContext = {
 export type Test = {
 	name: string;
 	port: number;
-	start: () => Promise<void>;
+	start: (ctx: {
+		pass: (message?: string, details?: any) => Promise<void>;
+		fail: (message?: string, details?: any) => Promise<void>;
+	}) => Promise<void>;
 	stop: () => Promise<void>;
 	/** If defined, this is a playwright test that controls the browser directly */
 	playwrightFn?: (ctx: TestContext) => Promise<void>;
@@ -195,7 +198,14 @@ export function playwrightTest(props: {
 // same as basicTest but gives us a handle to the server
 export function serverTest(props: {
 	name: string;
-	start: (server: http.Server, port: number) => Promise<void>;
+	start: (
+		server: http.Server,
+		port: number,
+		ctx: {
+			pass: (message?: string, details?: any) => Promise<void>;
+			fail: (message?: string, details?: any) => Promise<void>;
+		}
+	) => Promise<void>;
 	autoPass?: boolean;
 	js?: string;
 }) {
@@ -205,7 +215,7 @@ export function serverTest(props: {
 	return {
 		name: props.name,
 		port,
-		async start() {
+		async start({ pass, fail }) {
 			server = http.createServer(
 				{
 					// Only accept websocket upgrades, reject others (like h2c) so they fall back to normal HTTP
@@ -240,7 +250,7 @@ export function serverTest(props: {
 					}
 				}
 			);
-			await props.start(server, port);
+			await props.start(server, port, { pass, fail });
 			return new Promise<void>((resolve) => {
 				server.listen(port, () => resolve());
 			});

packages/scramjet/packages/runway/src/tests/linkheader.ts

@@ -0,0 +1,26 @@
+import { serverTest } from "../testcommon.ts";
+
+export default [
+	serverTest({
+		name: "linkheader-preload",
+		start: async (server, port, { pass, fail }) => {
+			server.on("request", (req, res) => {
+				if (req.url === "/") {
+					res.writeHead(200, {
+						"Content-Type": "text/html",
+						Link: `<http://localhost:${port}/script.js>; rel="preload"; as="script"`,
+					});
+					res.end();
+				} else if (req.url === "/script.js") {
+					console.log("script.js loaded");
+					pass("script.js loaded");
+				} else {
+					console.log("unexpected url request", req.url);
+					res.writeHead(404);
+					res.end("Not found");
+					fail("unexpected url request");
+				}
+			});
+		},
+	}),
+];