Skip to content

Commit 35f1228

Browse files
mirqtioclaude
andcommitted
fix(ci): prevent consent test duplication across main and consent projects
Main projects now explicitly exclude consent tests with testIgnore, while consent-specific projects run only consent tests with testMatch. This prevents the 14 consent tests from running twice (once in main, once in consent projects). Before: - Main projects: 129 tests (including 14 consent) - Consent projects: 14 tests - Total: 143 tests per browser (14 duplicates) After: - Main projects: 115 tests (excluding consent) - Consent projects: 14 tests - Total: 129 tests per browser (no duplicates) Note: Current codebase has 129 tests vs baseline's 118 (+11) due to: - Added waitlist.spec.ts (+10) - Added tests in homepage-mode-detection (+1) and purchase-flow (+2) - Removed _debug/chunk-loading-probe.spec.ts (-2) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 381c1e9 commit 35f1228

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

playwright.config.ci.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,13 @@ export default defineConfig({
2121
timeout: 15_000, // 15s for production build hydration (increased from 8s)
2222
},
2323
// CI tests all 5 browsers via matrix strategy
24+
// Main projects exclude consent tests (which run separately in consent-* projects)
2425
// Regular tests use storageState to bypass consent modal (fixes 80% timeout issue)
2526
// Consent tests run without storageState to actually test the consent modal
2627
projects: [
2728
{
2829
name: 'chromium-desktop',
30+
testIgnore: /.*consent.*\.spec\.ts$/, // Consent tests run in consent-chromium-desktop
2931
use: {
3032
...devices['Desktop Chrome'],
3133
viewport: { width: 1920, height: 1080 },
@@ -43,6 +45,7 @@ export default defineConfig({
4345
},
4446
{
4547
name: 'firefox-desktop',
48+
testIgnore: /.*consent.*\.spec\.ts$/, // Consent tests run in consent-firefox-desktop
4649
use: {
4750
...devices['Desktop Firefox'],
4851
viewport: { width: 1920, height: 1080 },
@@ -62,6 +65,7 @@ export default defineConfig({
6265
},
6366
{
6467
name: 'webkit-desktop',
68+
testIgnore: /.*consent.*\.spec\.ts$/, // Consent tests run in consent-webkit-desktop
6569
use: {
6670
...devices['Desktop Safari'],
6771
viewport: { width: 1920, height: 1080 },
@@ -81,6 +85,7 @@ export default defineConfig({
8185
},
8286
{
8387
name: 'chromium-mobile',
88+
testIgnore: /.*consent.*\.spec\.ts$/, // Consent tests run in consent-chromium-mobile
8489
use: {
8590
...devices['Pixel 7'],
8691
storageState: 'e2e/storage/consent-accepted.json',
@@ -96,6 +101,7 @@ export default defineConfig({
96101
},
97102
{
98103
name: 'webkit-mobile',
104+
testIgnore: /.*consent.*\.spec\.ts$/, // Consent tests run in consent-webkit-mobile
99105
use: {
100106
...devices['iPhone 14'],
101107
actionTimeout: 20_000, // Extra time for WebKit

0 commit comments

Comments
 (0)