From e8e79f9f95e9ac3bd898c55f7924aeccdbca1420 Mon Sep 17 00:00:00 2001 From: Ralf Anton Beier Date: Sun, 26 Apr 2026 01:42:48 +0200 Subject: [PATCH] fix(playwright): bump per-test timeout 30s -> 60s for graph render MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Latest CI on main showed the diagram-viewer.spec.ts:graph tests still failing despite cc8403e's ?limit=2000 — the graph DID render (page snapshot shows "742 nodes, 1477 edges") but layout + SVG generation took ~25-30s synchronously in render_graph_view, leaving <5s for the .toBeVisible() assertion before the 30s test timeout fired. Bumping the global per-test timeout to 60s gives the heavy graph render breathing room. Most tests complete in <2s so the bump is inert for them; it specifically unblocks the SVG-heavy assertions on the dogfood dataset. A future optimization would move layout/SVG out of the synchronous response (lazy-load via HTMX), but that's a code change for a separate PR. Trace: skip --- tests/playwright/playwright.config.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/playwright/playwright.config.ts b/tests/playwright/playwright.config.ts index 3a6cff7..83e3375 100644 --- a/tests/playwright/playwright.config.ts +++ b/tests/playwright/playwright.config.ts @@ -3,7 +3,11 @@ import { defineConfig } from "@playwright/test"; export default defineConfig({ testDir: ".", testMatch: "*.spec.ts", - timeout: 30_000, + // 60s per-test timeout: /graph with the dogfood dataset (742 nodes, 1477 + // edges) takes ~25-30s for synchronous layout + SVG generation in + // render_graph_view, leaving <5s for the .toBeVisible() assertion under + // the previous 30s budget. CI runner load makes this tighter still. + timeout: 60_000, retries: process.env.CI ? 1 : 0, workers: 1, // serial — single server instance use: {