Skip to content

Commit

Permalink
👷 chore(ci): Upload entire report instead of individual traces
Browse files Browse the repository at this point in the history
  • Loading branch information
duckception committed Oct 22, 2023
1 parent 059611a commit bdc4150
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 9 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,15 @@ jobs:
run: pnpm run build

- name: Run E2E tests (headless)
run: pnpm run test:e2e:headless -- --output "test-results-headless"
run: pnpm run test:e2e:headless

- name: Archive Playwright traces
- name: Archive Playwright report
uses: actions/upload-artifact@v3
if: success() || failure()
with:
name: playwright-traces-headless
name: playwright-report-headless
path: |
wallets/metamask/test-results-headless/
wallets/metamask/playwright-report-headless/
if-no-files-found: error

test-e2e-headful:
Expand Down Expand Up @@ -76,13 +76,13 @@ jobs:

- name: Run E2E tests (headful)
run: |
xvfb-run pnpm run test:e2e:headful -- --output "test-results-headful"
xvfb-run pnpm run test:e2e:headful
- name: Archive Playwright traces
- name: Archive Playwright report
uses: actions/upload-artifact@v3
if: success() || failure()
with:
name: playwright-traces-headful
name: playwright-report-headful
path: |
wallets/metamask/test-results-headful/
wallets/metamask/playwright-report-headful/
if-no-files-found: error
4 changes: 3 additions & 1 deletion wallets/metamask/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ export default defineConfig({

// Concise 'dot' for CI, default 'html' when running locally.
// See https://playwright.dev/docs/test-reporters.
reporter: process.env.CI ? 'dot' : 'html',
reporter: process.env.CI
? [['html', { open: 'never', outputFolder: `playwright-report-${process.env.HEADLESS ? 'headless' : 'headful'}` }]]
: 'html',

// Shared settings for all the projects below.
// See https://playwright.dev/docs/api/class-testoptions.
Expand Down

0 comments on commit bdc4150

Please sign in to comment.