From d2734def8980a201b712b8eb6f168d6cc2ec18ae Mon Sep 17 00:00:00 2001 From: Amit Kumar Date: Fri, 24 Apr 2026 02:33:49 +0000 Subject: [PATCH] ci(ui): exclude screenshots.spec.ts from default playwright run Block 7's screenshots.spec.ts talks to a live docsiq backend on a fixed port; CI has only the Vite dev server booted, so every screenshot test fails with ERR_CONNECTION_REFUSED. Gate the spec behind PLAYWRIGHT_SCREENSHOTS so the dedicated regeneration script opts in and the default e2e suite ignores it. Co-Authored-By: Claude Opus 4.7 (1M context) --- ui/playwright.config.ts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/ui/playwright.config.ts b/ui/playwright.config.ts index c47723f..84dcc37 100644 --- a/ui/playwright.config.ts +++ b/ui/playwright.config.ts @@ -6,6 +6,11 @@ const isCI = !!process.env.CI; export default defineConfig({ testDir: "./e2e", + // Screenshot regeneration is a manual task run against a live docsiq + // backend with seeded fixtures; it has no role in the CI smoke suite + // and fails fast when no backend is reachable. Gate behind the + // PLAYWRIGHT_SCREENSHOTS env var so the dedicated script opts in. + testIgnore: process.env.PLAYWRIGHT_SCREENSHOTS ? undefined : ["**/screenshots.spec.ts"], fullyParallel: true, forbidOnly: isCI, retries: isCI ? 1 : 0,