test(dashboard): widen waitFor bound on planning-flow respond assertion

The newly-gated backfill file flaked on a loaded CI shard: waitFor's
private 1s default (independent of the 15s vitest testTimeout) raced the
click->respondToPlanning state-update chain. Passes deterministically in
isolation; 5s bound absorbs shard CPU starvation without masking real
regressions.
This commit is contained in:
gsxdsm
2026-06-03 20:46:54 -07:00
parent de3156e4ad
commit ca3aebeac8

View File

@@ -337,14 +337,20 @@ describe("PlanningModeModal", () => {
fireEvent.click(screen.getByText("Small"));
fireEvent.click(screen.getByText("Continue"));
await waitFor(() => {
expect(mockRespondToPlanning).toHaveBeenCalledWith(
"session-123",
{ "q-scope": "small" },
undefined,
"tab-self",
);
});
await waitFor(
() => {
expect(mockRespondToPlanning).toHaveBeenCalledWith(
"session-123",
{ "q-scope": "small" },
undefined,
"tab-self",
);
},
// waitFor's private 1s default (independent of vitest testTimeout) has
// flaked under loaded CI shards; the click->respond chain crosses
// several state-update hops. Generous bound, still fails fast locally.
{ timeout: 5000 },
);
});
it("shows stop action in loading and stops generation", async () => {