From 8301df1492a89fae448f28c8402ab7fdd2d6fb7d Mon Sep 17 00:00:00 2001 From: James Date: Mon, 2 Nov 2026 07:38:14 +0000 Subject: [PATCH] test(studio): the dev server render job tests wait up to 20 s for the render vi.waitFor gives up after 1 s by default, inside a describe that already allows 30 s per test. On a loaded Windows runner the blur test saw the job still "rendering" at 1 s and failed the #6980 merge group (job 125103352086). Reproduced by making the mocked render take 1.5 s: main fails with the same "expected 'rendering' to be 'complete'"; this passes. Co-Authored-By: Claude Opus 5.5 --- packages/studio/vite.adapter.render.test.ts | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/studio/vite.adapter.render.test.ts b/packages/studio/vite.adapter.render.test.ts index c199c322a3b..06ab27c7667 100644 --- a/packages/studio/vite.adapter.render.test.ts +++ b/packages/studio/vite.adapter.render.test.ts @@ -86,11 +86,14 @@ describe("Vite GIF rendering", () => { ); }); +// vi.waitFor gives up after 1 s by default; a loaded Windows runner took longer to finish a mocked render. +const RENDER_DONE = { timeout: 20_000 }; + describe("Studio dev server render jobs", { timeout: 30_000 }, () => { it("hands the route's pageMotionBlur to the job, so a preview that could not blur renders unblurred", async () => { producer.executeRenderJob.mockResolvedValue(undefined); const { state } = await fixture({ format: "mp4", pageMotionBlur: "off" }); - await vi.waitFor(() => expect(state.status, state.error).toBe("complete")); + await vi.waitFor(() => expect(state.status, state.error).toBe("complete"), RENDER_DONE); expect(producer.createRenderJob).toHaveBeenCalledWith( expect.objectContaining({ pageMotionBlur: "off" }), ); @@ -99,7 +102,7 @@ describe("Studio dev server render jobs", { timeout: 30_000 }, () => { it("leaves the job's blur to the composition when the route sends none", async () => { producer.executeRenderJob.mockResolvedValue(undefined); const { state } = await fixture({ format: "mp4" }); - await vi.waitFor(() => expect(state.status, state.error).toBe("complete")); + await vi.waitFor(() => expect(state.status, state.error).toBe("complete"), RENDER_DONE); expect(producer.createRenderJob.mock.calls[0]?.[0]).not.toHaveProperty("pageMotionBlur"); }); });