ci: upload Playwright tests' results even in PR builds

We do that in all other workflows that run the Playwright tests, and it
is very helpful when diagnosing issues (especially the screenshots that
are stored on failure).

This needs a slight change in the Playwright configuration where we now
want to have the `github` reporter _in addition_ to the `html` reporter
(and not instead of it).

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

.github/workflows/ci.yml

@@ -82,4 +82,9 @@ jobs:
     - name: Run Playwright tests
       env:
         PLAYWRIGHT_TEST_URL: http://localhost:5000/
-      run: npx playwright test --project=chrome
+      run: npx playwright test --project=chrome
+    - uses: actions/upload-artifact@v4
+      if: always()
+      with:
+        name: playwright-report
+        path: playwright-report/

playwright.config.js

@@ -22,7 +22,11 @@ module.exports = defineConfig({
   workers: process.env.CI ? 1 : undefined,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */
   /* Uses 'github' for GitHub Actions CI to generate annotations, otherwise 'html' */
-  reporter: process.env.CI && process.env.PLAYWRIGHT_TEST_URL === 'http://localhost:5000/' ? 'github' : 'html',
+  reporter: [
+    [ 'html' ],
+    ...(process.env.CI && process.env.PLAYWRIGHT_TEST_URL === 'http://localhost:5000/'
+      ? [ [ 'github' ] ] : [])
+  ],
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Base URL to use in actions like `await page.goto('/')`. */