ci: improve e2e tests22222222

Author: shijistar

e2e/fixtures/EnumPage.ts

@@ -4,12 +4,13 @@ export default class EnumPage {
   constructor(public readonly page: Page) {}
 
   async gotoModern() {
-    await this.page.goto('/modern.html');
+    await this.page.goto('/modern.html', { waitUntil: 'domcontentloaded' });
   }
   async gotoLegacy() {
-    await this.page.goto('/legacy.html');
+    await this.page.goto('/legacy.html', { waitUntil: 'domcontentloaded' });
   }
   async setupLang() {
+    await this.page.waitForLoadState('domcontentloaded');
     await this.page.evaluate(() => {
       const {
         EnumPlus: { Enum, defaultLocalize },
@@ -20,6 +21,7 @@ export default class EnumPage {
     });
   }
   async resetLang() {
+    await this.page.waitForLoadState('domcontentloaded');
     await this.page.evaluate(() => {
       const {
         EnumPlus: { Enum },

e2e/fixtures/EnumTest.ts

@@ -9,7 +9,8 @@ export const test = base.extend<{
   enumPage: async ({ page }, use) => {
     const enumPage = new EnumPage(page);
     await enumPage.gotoModern();
-
+    // delay to ensure the page is fully loaded
+    await page.waitForLoadState('domcontentloaded');
     await use(enumPage);
   },
 });

playwright.config.ts

@@ -1,23 +1,14 @@
 import type { PlaywrightTestConfig } from '@playwright/test';
 import { devices } from '@playwright/test';
 
-const targetProject = process.argv.find((arg) => arg.startsWith('--project='))?.split('=')[1];
-const PORTS: Record<string, number> = {
-  chrome: 7080,
-  firefox: 7081,
-  webkit: 7082,
-  edge: 7083,
-  'chromium-legacy': 7084,
-};
-const port = PORTS[targetProject ?? ''] ?? PORTS.chrome;
+const port = 7080;
 
 const config: PlaywrightTestConfig = {
   testDir: './e2e/specs',
   testMatch: '**/*.spec.ts',
   outputDir: './e2e/test-results',
   globalSetup: require.resolve('./e2e/global.setup.ts'),
   globalTeardown: require.resolve('./e2e/global.teardown.ts'),
-  workers: 16,
   forbidOnly: !!process.env.CI,
   retries: 2,
   use: {
@@ -29,19 +20,23 @@ const config: PlaywrightTestConfig = {
     {
       name: 'chrome',
       use: devices['Desktop Chrome'],
+      workers: 1,
+    },
+    {
+      name: 'firefox',
+      use: devices['Desktop Firefox'],
+      workers: 1,
+    },
+    {
+      name: 'webkit',
+      use: devices['Desktop Safari'],
+      workers: 1,
+    },
+    {
+      name: 'edge',
+      use: devices['Desktop Edge'],
+      workers: 1,
     },
-    // {
-    //   name: 'firefox',
-    //   use: devices['Desktop Firefox'],
-    // },
-    // {
-    //   name: 'webkit',
-    //   use: devices['Desktop Safari'],
-    // },
-    // {
-    //   name: 'edge',
-    //   use: devices['Desktop Edge'],
-    // },
     // Legacy browsers (simulated)
     // {
     //   name: 'chromium-legacy',
@@ -56,8 +51,8 @@ const config: PlaywrightTestConfig = {
   ],
   webServer: {
     command: `npx http-server ./e2e/fixtures -p ${port}`,
-    port,
-    reuseExistingServer: !process.env.CI,
+    port: 7080,
+    reuseExistingServer: false,
   },
 };
 

test/engines/playwright.ts

@@ -30,7 +30,7 @@ export class PlaywrightEngine extends TestEngineBase {
           WeekData,
           SerializeJavascript,
         };
-        console.log('window', runtimeContext);
+        // console.log('window', runtimeContext);
         const { serializeJavascript: serialize, deserializeJavascript: deserialize } = SerializeJavascript;
         const args = deserialize(contextStr) as { evaluateFn: typeof evaluate };
         const { evaluateFn, ...rest } = args;