fix: Skip failing test on CI (#33646)

.github/workflows/ci-e2e-playwright.yml

@@ -217,6 +217,14 @@ jobs:
                   retention-days: 30
               if: ${{ failure() }}
 
+            - name: Archive screenshots
+              if: always()
+              uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4
+              with:
+                  name: playwright-screenshots
+                  path: playwright/test-results
+                  retention-days: 30
+
     capture-run-time:
         name: Capture run time
         runs-on: ubuntu-latest

playwright/e2e/signup.spec.ts

@@ -184,7 +184,9 @@ test.describe('Signup', () => {
         )
     })
 
-    test('Shows redirect notice if redirecting for maintenance', async ({ page }) => {
+    // TODO un-skip.
+    // Skipping test as it was failing on master, see https://posthog.slack.com/archives/C0113360FFV/p1749742204672659
+    test.skip('Shows redirect notice if redirecting for maintenance', async ({ page }) => {
         // Equivalent to setupFeatureFlags in Playwright
         await page.route('**/flags/*', async (route) => {
             const response = {

playwright/playwright.config.ts

@@ -49,6 +49,8 @@ export default defineConfig({
 
         /* Locate elements defined by `data-attr-something` with `page.getByTestId('something')` */
         testIdAttribute: 'data-attr',
+
+        screenshot: 'only-on-failure',
     },
 
     /* Configure projects for major browsers */