Skip to content

Commit 2286816

Browse files
ryan-williamsclaude
andcommitted
fix: exclude defaultBrowserType from iPhone device settings in E2E tests
The mobile-hover tests were spreading `devices['iPhone 12']` which includes `defaultBrowserType: 'webkit'`. This caused CI to fail because webkit isn't installed. Now we destructure out `defaultBrowserType` so tests run on chromium with iPhone viewport/touch settings. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 2f2ec3a commit 2286816

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

www/test/e2e/mobile-hover.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ import { test, expect, devices } from '@playwright/test'
66
const __filename = fileURLToPath(import.meta.url)
77
const __dirname = path.dirname(__filename)
88

9-
// Use iPhone 12 for mobile emulation
10-
const iPhone = devices['iPhone 12']
11-
test.use({ ...iPhone })
9+
// Use iPhone 12 viewport/touch settings but stay on chromium (avoid webkit dependency)
10+
const { defaultBrowserType: _, ...iPhone } = devices['iPhone 12']
11+
test.use(iPhone)
1212

1313
test.describe('Mobile Hover Dismiss', () => {
1414

0 commit comments

Comments
 (0)