[scramjet/core] fix playwright tests

Author: Percslol

packages/scramjet/devserver.ts

@@ -27,7 +27,7 @@ const branch = execSync("git rev-parse --abbrev-ref HEAD", {
 const packagejson = JSON.parse(await fs.readFile("./package.json", "utf-8"));
 const version = packagejson.version;
 
-const DEMO_PORT = process.env.CHROME_PORT || 4141;
+const DEMO_PORT = process.env.DEMO_PORT || 4141;
 const WISP_PORT = process.env.WISP_PORT || 4142;
 
 process.env.VITE_WISP_URL =

packages/scramjet/jsconfig.json

@@ -15,5 +15,5 @@
 		"types": []
 	},
 	"include": ["static/**/*", "dist/types/index.d.ts"],
-	"exclude": ["node_modules", "dist", "src/**/*", "tests/**/*"]
+	"exclude": ["node_modules", "dist", "src/**/*", "packages/core/tests/**/*"]
 }

packages/scramjet/package.json

@@ -8,6 +8,7 @@
 	"license": "AGPL-3.0-only",
 	"packageManager": "[email protected]",
 	"scripts": {
+		"dev": "node --no-warnings=ExperimentalWarning devserver.ts",
 		"preinstall": "npx only-allow pnpm"
 	},
 	"devDependencies": {

packages/scramjet/packages/core/playwright.config.ts

@@ -0,0 +1,52 @@
+import { defineConfig, devices } from "@playwright/test";
+import type { GitHubActionOptions } from "@estruyf/github-actions-reporter";
+
+/**
+ * See https://playwright.dev/docs/test-configuration.
+ */
+export default defineConfig({
+	testDir: "./tests/integration/site",
+	fullyParallel: true,
+	forbidOnly: !!process.env.CI,
+	retries: 2,
+	reporter: process.env.CI
+		? [
+				[
+					"@estruyf/github-actions-reporter",
+					{
+						title: "Test summary",
+						useDetails: true,
+						showError: true,
+					} as GitHubActionOptions,
+				],
+				["github"],
+			]
+		: "html",
+	timeout: 20000,
+	/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
+	use: {
+		/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
+		trace: "on-first-retry",
+		actionTimeout: 10000,
+		baseURL: "http://localhost:4141",
+	},
+
+	/* Configure projects for major browsers */
+	projects: [
+		{
+			name: "chromium",
+			use: { ...devices["Desktop Chrome"] },
+		},
+		// {
+		//   name: "firefox",
+		//   use: { ...devices["Desktop Firefox"] },
+		// },
+	],
+
+	/* Run your local dev server before starting the tests */
+	webServer: {
+		command: "cd .. && pnpm run dev",
+		url: "http://localhost:4141",
+		reuseExistingServer: false,
+	},
+});

packages/scramjet/tests/ci/packageValidation.js renamed to packages/scramjet/packages/core/tests/ci/packageValidation.js

@@ -14,9 +14,10 @@ import { existsSync } from "node:fs";
  * These aren't globs.
  */
 const EXPECTED_DIST_FILES = [
-	"dist/scramjet.all.js",
-	"dist/scramjet.bundle.js",
-	"dist/scramjet.sync.js",
+	"dist/scramjet.js",
+	"dist/scramjet.mjs",
+	"dist/scramjet_bundled.js",
+	"dist/scramjet_bundled.mjs",
 	"dist/scramjet.wasm.wasm",
 ];
 

packages/scramjet/tests/integration/site/google.spec.ts renamed to packages/scramjet/packages/core/tests/integration/site/google.spec.ts

@@ -10,11 +10,9 @@ export async function setupPage(
 	await page.route("**", (route) => route.continue());
 	// Goto base url defined in config.
 	await page.goto("/");
-	await page.waitForSelector(".version > b");
-	const bar = page.locator(".bar");
-	const title = await page.locator(".version > b").textContent();
+	await page.waitForSelector("#search");
+	const bar = page.locator("#search");
 	const frame = page.frameLocator("iframe");
-	expect(title).toBe("scramjet");
 
 	expect(bar).not.toBeNull();
 

packages/scramjet/tests/integration/site/youtube.spec.ts renamed to packages/scramjet/packages/core/tests/integration/site/youtube.spec.ts

@@ -26,7 +26,12 @@ export const App: Component<
 					this.frame.go(this.url);
 				}}
 			>
-				<input type="text" value={use(this.url)} placeholder="Enter URL" />
+				<input
+					id="search"
+					type="text"
+					value={use(this.url)}
+					placeholder="Enter URL"
+				/>
 			</form>
 			<iframe this={use(this.frameel)}></iframe>
 		</div>