fix(test): properly exclude tests using project-level testIgnore arrays

ROOT CAUSE: Playwright project-level testIgnore REPLACES (not extends) top-level config.
Previous commits (72531af, fb0070b) added patterns to top-level testIgnore, but these
were completely ignored because each project had its own testIgnore that overrode them.

SOLUTION: Create EXCLUDED_TESTS array and apply to all main browser projects at the
project level where it actually takes effect.

Changes:
- playwright.config.ci.ts:
  * Created EXCLUDED_TESTS constant with all 6 exclusion patterns
  * Applied to all 5 main browser projects (chromium/firefox/webkit desktop/mobile)
  * Consent projects continue using testMatch (they only run consent tests)

- playwright.config.ts:
  * Same fix applied for local/CI parity
  * EXCLUDED_TESTS array applied to all 5 projects

Tests excluded:
- waitlist-functional.spec.ts (incomplete functionality)
- purchase-payment-element.spec.ts (incomplete payment tests)
- journeys.spec.ts (incomplete journey tests)
- _debug/*.spec.ts (diagnostic/debug tests)
- waitlist.spec.ts (11 tests, did NOT run in local baseline 5438b418d431bae7)
- consent*.spec.ts (run separately in dedicated consent projects)

Expected impact:
- Test count should drop from 665 to ~615 tests
- Removes ~50 test executions (10 waitlist tests × 5 browsers)
- Establishes local/CI parity matching baseline run 5438b418d431bae7

Documentation: See SCRATCHPAD.md "testIgnore Override Issue" section

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: mirqtio

playwright.config.ci.ts

@@ -11,6 +11,19 @@ import base from './playwright.config'
  * - Chromium only (vs 5 browsers locally)
  * - Longer timeouts for cold starts
  */
+
+// Tests to exclude from main browser projects (incomplete/debug/consent tests)
+// IMPORTANT: Project-level testIgnore REPLACES (not extends) top-level config
+// Must combine all patterns at project level for proper exclusion
+const EXCLUDED_TESTS = [
+  /.*waitlist-functional\.spec\.ts$/,
+  /.*purchase-payment-element\.spec\.ts$/,
+  /.*journeys\.spec\.ts$/,
+  /.*\/_debug\/.*\.spec\.ts$/,
+  /.*waitlist\.spec\.ts$/, // Did NOT run in local baseline run 5438b418d431bae7
+  /.*consent.*\.spec\.ts$/, // Consent tests run separately in dedicated projects
+]
+
 export default defineConfig({
   ...base,
   workers: 6, // Conservative parallelism for CI
@@ -19,21 +32,13 @@ export default defineConfig({
   expect: {
     timeout: 15_000, // 15s for production build hydration (increased from 8s)
   },
-  // Skip incomplete/debug tests (match local run 5438b418d431bae7)
-  testIgnore: [
-    /.*waitlist-functional\.spec\.ts$/,
-    /.*purchase-payment-element\.spec\.ts$/,
-    /.*journeys\.spec\.ts$/,
-    /.*\/_debug\/.*\.spec\.ts$/,
-    /.*waitlist\.spec\.ts$/, // Did NOT run in local baseline run
-  ],
   // CI tests all 5 browsers via matrix strategy
   // Regular tests use storageState to bypass consent modal (fixes 80% timeout issue)
   // Consent tests run without storageState to actually test the consent modal
   projects: [
     {
       name: 'chromium-desktop',
-      testIgnore: /.*consent.*\.spec\.ts$/,
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Chrome'],
         viewport: { width: 1920, height: 1080 },
@@ -51,7 +56,7 @@ export default defineConfig({
     },
     {
       name: 'firefox-desktop',
-      testIgnore: /.*consent.*\.spec\.ts$/,
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Firefox'],
         viewport: { width: 1920, height: 1080 },
@@ -71,7 +76,7 @@ export default defineConfig({
     },
     {
       name: 'webkit-desktop',
-      testIgnore: /.*consent.*\.spec\.ts$/,
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Safari'],
         viewport: { width: 1920, height: 1080 },
@@ -91,7 +96,7 @@ export default defineConfig({
     },
     {
       name: 'chromium-mobile',
-      testIgnore: /.*consent.*\.spec\.ts$/,
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Pixel 7'],
         storageState: 'e2e/storage/consent-accepted.json',
@@ -107,7 +112,7 @@ export default defineConfig({
     },
     {
       name: 'webkit-mobile',
-      testIgnore: /.*consent.*\.spec\.ts$/,
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['iPhone 14'],
         actionTimeout: 20_000, // Extra time for WebKit

playwright.config.ts

@@ -1,5 +1,16 @@
 import { defineConfig, devices } from '@playwright/test'
 
+// Tests to exclude (incomplete/debug tests)
+// IMPORTANT: Project-level testIgnore REPLACES (not extends) top-level config
+// Must combine all patterns at project level for proper exclusion
+const EXCLUDED_TESTS = [
+  /.*waitlist-functional\.spec\.ts$/,
+  /.*purchase-payment-element\.spec\.ts$/,
+  /.*journeys\.spec\.ts$/,
+  /.*\/_debug\/.*\.spec\.ts$/,
+  /.*waitlist\.spec\.ts$/, // Did NOT run in local baseline run 5438b418d431bae7
+]
+
 export default defineConfig({
   testDir: './e2e',
   fullyParallel: true,
@@ -10,14 +21,6 @@ export default defineConfig({
   expect: {
     timeout: 5_000,
   },
-  // Skip incomplete/debug tests (match local run 5438b418d431bae7)
-  testIgnore: [
-    /.*waitlist-functional\.spec\.ts$/,
-    /.*purchase-payment-element\.spec\.ts$/,
-    /.*journeys\.spec\.ts$/,
-    /.*\/_debug\/.*\.spec\.ts$/,
-    /.*waitlist\.spec\.ts$/, // Did NOT run in local baseline run
-  ],
 
   reporter: [
     ['list'],
@@ -36,13 +39,15 @@ export default defineConfig({
   projects: [
     {
       name: 'chromium-desktop',
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Chrome'],
         viewport: { width: 1920, height: 1080 },
       },
     },
     {
       name: 'firefox-desktop',
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Firefox'],
         viewport: { width: 1920, height: 1080 },
@@ -53,19 +58,22 @@ export default defineConfig({
     },
     {
       name: 'webkit-desktop',
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Desktop Safari'],
         viewport: { width: 1920, height: 1080 },
       },
     },
     {
       name: 'chromium-mobile',
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['Pixel 7'],
       },
     },
     {
       name: 'webkit-mobile',
+      testIgnore: EXCLUDED_TESTS,
       use: {
         ...devices['iPhone 14'],
       },