From bdc4150a0c05163fd2c99761521c27d4e90d93a3 Mon Sep 17 00:00:00 2001
From: Daniel Izdebski <spidiritsu@gmail.com>
Date: Mon, 23 Oct 2023 01:51:31 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=91=B7=20chore(ci):=20Upload=20entire=20r?=
 =?UTF-8?q?eport=20instead=20of=20individual=20traces?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .github/workflows/test.yml            | 16 ++++++++--------
 wallets/metamask/playwright.config.ts |  4 +++-
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index b9f568d8d..cce497184 100644
--- a/.github/workflows/test.yml
+++ b/.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
diff --git a/wallets/metamask/playwright.config.ts b/wallets/metamask/playwright.config.ts
index e22f8eea5..86de5708c 100644
--- a/wallets/metamask/playwright.config.ts
+++ b/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.