Fix Playwright github reporter to run in the pull_request context onc…

…e more (#12509) Signed-off-by: Michael Telatynski <[email protected]>
tchapgouv · May 10, 2024 · c6f6f1d · c6f6f1d
1 parent eee0b2a
commit c6f6f1d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/end-to-end-tests-netlify.yaml b/.github/workflows/end-to-end-tests-netlify.yaml
@@ -41,7 +41,7 @@ jobs:
                   merge-multiple: true
 
             - name: Merge into HTML Report
-              run: yarn playwright merge-reports --reporter=html,github,./playwright/flaky-reporter.ts ./all-blob-reports
+              run: yarn playwright merge-reports --reporter=html,./playwright/flaky-reporter.ts ./all-blob-reports
               env:
                   # Only pass creds to the flaky-reporter on main branch runs
                   GITHUB_TOKEN: ${{ github.event.workflow_run.head_branch == 'develop' && secrets.ELEMENT_BOT_TOKEN || '' }}

diff --git a/playwright.config.ts b/playwright.config.ts
@@ -41,7 +41,7 @@ export default defineConfig<TestOptions>({
     outputDir: "playwright/test-results",
     workers: 1,
     retries: process.env.CI ? 2 : 0,
-    reporter: process.env.CI ? "blob" : [["html", { outputFolder: "playwright/html-report" }]],
+    reporter: process.env.CI ? [["blob"], ["github"]] : [["html", { outputFolder: "playwright/html-report" }]],
     projects: [
         {
             name: "Legacy Crypto",