Skip to content

Commit

Permalink
👷 ci: Use self-hosted runner for E2E tests
Browse files Browse the repository at this point in the history
  • Loading branch information
duckception committed Nov 30, 2023
1 parent 96c800d commit fb47009
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
11 changes: 6 additions & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:

test-e2e-headful:
name: Run E2E tests (headful)
runs-on: ubuntu-latest
runs-on: self-hosted
steps:
- uses: actions/checkout@v3

Expand All @@ -41,10 +41,11 @@ jobs:
- name: Build project
run: pnpm run build

- name: Install linux dependencies
run: |
sudo apt-get install --no-install-recommends -y \
xvfb
# Skipping this for now, `xvfb` is already installed on the self-hosted runner.
#- name: Install linux dependencies
# run: |
# sudo apt-get install --no-install-recommends -y \
# xvfb

- name: Build project
run: pnpm run build
Expand Down
4 changes: 2 additions & 2 deletions wallets/metamask/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,10 @@ export default defineConfig({
forbidOnly: !!process.env.CI,

// Fail all remaining tests on CI after the first failure. We want to reduce the feedback loop on CI to minimum.
maxFailures: process.env.CI ? 1 : 0,
maxFailures: process.env.CI ? 0 : 0,

// Opt out of parallel tests on CI since it supports only 1 worker.
workers: process.env.CI ? 1 : undefined,
workers: process.env.CI ? undefined : undefined,

// Concise 'dot' for CI, default 'html' when running locally.
// See https://playwright.dev/docs/test-reporters.
Expand Down

0 comments on commit fb47009

Please sign in to comment.