fix: only upload video and traces on failure.

This was accidentally changed in 21ebd32

Author: bznein

.github/workflows/playwright.yml

@@ -95,7 +95,7 @@ jobs:
         run: make webe2etest
 
       - name: Upload Playwright artifacts
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: playwright-artifacts

frontends/web/playwright.config.ts

@@ -17,14 +17,15 @@ export default defineConfig({
       timeout: 120_000,
     },
   ],
-  timeout: 120_000,
+  timeout: 180_000,
   workers: 1, // Tests are not parallel-safe yet.
   use: {
     baseURL: `http://${HOST}:${FRONTEND_PORT}`,
     headless: true,
-    video: 'on',
+    video: 'retain-on-failure',
     screenshot: 'only-on-failure',
-    trace: 'on',
+    trace: 'retain-on-failure',
+    bypassCSP: true, // add this to disable cors
     launchOptions: {
       // By default, tests are not run in slow motion.
       // Can be enabled by setting the PLAYWRIGHT_SLOW_MO environment variable to a value > 0.