feat(e2e): Debuging PW web tests on CI

trezor · Dec 3, 2024 · eb86a7d · eb86a7d
1 parent 2377817
commit eb86a7d
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 10 deletions.
diff --git a/.github/workflows/test-suite-web-e2e-pw.yml b/.github/workflows/test-suite-web-e2e-pw.yml
@@ -150,17 +150,19 @@ jobs:
       - name: Install Playwright browsers
         run: npx playwright install
 
+        ## MOVE PULL HERE
+
       - name: Run Playwright e2e tests
         env:
           COMPOSE_FILE: ./docker/docker-compose.suite-ci.yml
-          BASE_URL: https://dev.suite.sldev.cz/suite-web/feat-parametrize-pw-suite/web/
-
+          BASE_URL: https://dev.suite.sldev.cz/suite-web/convert-cy-pw-settings/web/
           ## reporter url
           TRACK_SUITE_URL: https://track-suite-ff9ad9f5b4f6.herokuapp.com
           ## when debugging or developing tests it does not make sense to have retries,
           ## in other cases retries are useful to avoid occasional failures due to flaky tests
           ALLOW_RETRY: false
           TEST_GROUP: ${{ matrix.TEST_GROUP }}
+          GITHUB_ACTION: true
           CI_JOB_ID: ${{ github.run_id }}
           CI_COMMIT_SHA: ${{ github.sha }}
           CI_JOB_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}

diff --git a/docker/docker-compose.suite-ci.yml b/docker/docker-compose.suite-ci.yml
@@ -5,15 +5,10 @@ services:
     environment:
       - SDL_VIDEODRIVER=dummy
       - XDG_RUNTIME_DIR=/var/tmp
-    ports:
-      - "9001:9001"
-      - "21325:21325"
-      - "21326:21326"
+    network_mode: bridge # this makes docker reuse existing networks
 
   bitcoin-regtest:
     image: ghcr.io/trezor/trezor-user-env-regtest # this is a special image that runs regtest and blockbook
     depends_on:
       - trezor-user-env-unix
-    ports:
-      - "50001:50001"
     network_mode: service:trezor-user-env-unix
diff --git a/packages/suite-desktop-core/e2e/support/fixtures.ts b/packages/suite-desktop-core/e2e/support/fixtures.ts
@@ -1,6 +1,6 @@
 /* eslint-disable react-hooks/rules-of-hooks */
 
-import { test as base, ElectronApplication, Page } from '@playwright/test';
+import { test as base, ElectronApplication, Page, expect } from '@playwright/test';
 
 import { TrezorUserEnvLink, TrezorUserEnvLinkClass } from '@trezor/trezor-user-env-link';
 
@@ -72,7 +72,13 @@ const test = base.extend<Fixtures>({
             const tracePath = `${testInfo.outputDir}/trace.electron.zip`;
             await window.context().tracing.stop({ path: tracePath });
         } else {
-            await page.goto('/');
+            // eslint-disable-next-line no-console
+            console.log('Config:', process.env.BASE_URL || 'http://localhost:8000');
+            // eslint-disable-next-line no-console
+            console.log('Fixture BASE_URL:', process.env.BASE_URL);
+            await page.goto(process.env.BASE_URL!);
+            // await page.goto('/');
+            await expect(page).toHaveURL(process.env.BASE_URL!);
             await use(page);
         }
     },

diff --git a/packages/suite-desktop-core/e2e/support/pageActions/onboardingActions.ts b/packages/suite-desktop-core/e2e/support/pageActions/onboardingActions.ts
@@ -20,6 +20,8 @@ export class OnboardingActions {
     }
 
     async optionallyDismissFwHashCheckError() {
+        // eslint-disable-next-line no-console
+        console.log('Onboarding BASE_URL:', process.env.BASE_URL);
         await expect(this.welcomeTitle).toBeVisible();
         // dismisses the error modal only if it appears (handle it async in parallel, not necessary to block the rest of the flow)
         this.window
@@ -28,6 +30,8 @@ export class OnboardingActions {
     }
 
     async completeOnboarding() {
+        // eslint-disable-next-line no-console
+        console.log('Onboarding BASE_URL:', process.env.BASE_URL);
         await this.optionallyDismissFwHashCheckError();
         await this.analyticsContinueButton.click();
         await this.onboardingContinueButton.click();