👷 chore(ci): Upload entire report instead of individual traces (#942)

.github/workflows/test.yml

@@ -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:
@@ -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

wallets/metamask/playwright.config.ts

@@ -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.