Use Playwright baseURL for fixture navigation

Author: sm17p

e2e/constants.ts

@@ -1,2 +1,2 @@
 export const FIXTURE_SERVER_PORT = 51234;
-export const STABLE_TEST_PAGE_URL = `http://127.0.0.1:${FIXTURE_SERVER_PORT}/inspector-playground.html`;
+export const STABLE_TEST_PAGE_PATH = "/inspector-playground.html";

e2e/pages/web.ts

@@ -1,7 +1,7 @@
 import { BrowserContext, Page } from "@playwright/test";
 import path from "path";
 
-import { STABLE_TEST_PAGE_URL } from "../constants";
+import { STABLE_TEST_PAGE_PATH } from "../constants";
 
 declare const chrome: {
 	storage: {
@@ -12,7 +12,7 @@ declare const chrome: {
 };
 
 export async function openStableTestPage(page: Page) {
-	await page.goto(STABLE_TEST_PAGE_URL);
+	await page.goto(STABLE_TEST_PAGE_PATH);
 	await page.waitForLoadState("domcontentloaded");
 	await page.waitForSelector('[data-testid="playground-main"]');
 }
@@ -35,7 +35,7 @@ export async function enableDomainInStorage(context: BrowserContext, domain: str
 }
 
 export async function enableStableDomainInStorage(context: BrowserContext) {
-	const domain = new URL(STABLE_TEST_PAGE_URL).host;
+	const domain = "127.0.0.1:51234";
 	await enableDomainInStorage(context, domain);
 }
 

playwright.config.ts

@@ -15,19 +15,19 @@ export default defineConfig({
 	testDir: "./e2e",
 	testMatch: "tests/**/*.spec.ts",
 	/* Run tests in files in parallel */
-	// fullyParallel: true,
+	fullyParallel: true,
 	/* Fail the build on CI if you accidentally left test.only in the source code. */
 	forbidOnly: !!process.env.CI,
 	/* Retry on CI only */
-	retries: process.env.CI ? 2 : 0,
+	retries: process.env.CI ? 1 : 0,
 	/* Opt out of parallel tests on CI. */
-	workers: process.env.CI ? 1 : undefined,
+	// workers: process.env.CI ? 1 : undefined,
 	/* Reporter to use. See https://playwright.dev/docs/test-reporters */
 	reporter: process.env.CI ? [["list"], ["html"], ["github"]] : "list",
 	/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
 	use: {
 		/* Base URL to use in actions like `await page.goto('/')`. */
-		// baseURL: 'http://localhost:3000',
+		baseURL: "http://127.0.0.1:51234",
 
 		/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
 		trace: "on-first-retry",
@@ -61,7 +61,7 @@ export default defineConfig({
 		//   use: { ...devices['Desktop Chrome'], channel: 'chrome' },
 		// },
 	],
-	timeout: process.env.CI ? 60_000 : 5_000,
+	timeout: process.env.CI ? 30_000 : 5_000,
 	webServer: {
 		command: "node e2e/serve-fixtures.mjs 51234",
 		url: "http://127.0.0.1:51234/inspector-playground.html",