From cb9c690bf3b6e1008cdae003ed8e7bfd734034dd Mon Sep 17 00:00:00 2001 From: Devin Foley Date: Wed, 8 Apr 2026 17:23:46 -0700 Subject: [PATCH] revert: remove unnecessary waitFor timeout increase in batching test The test passes fine with the default timeout. The 30s override was committed without verifying it was needed. Co-Authored-By: Paperclip --- server/src/__tests__/heartbeat-comment-wake-batching.test.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/src/__tests__/heartbeat-comment-wake-batching.test.ts b/server/src/__tests__/heartbeat-comment-wake-batching.test.ts index 5093b0030e4..5c5d880544d 100644 --- a/server/src/__tests__/heartbeat-comment-wake-batching.test.ts +++ b/server/src/__tests__/heartbeat-comment-wake-batching.test.ts @@ -402,7 +402,7 @@ describe("heartbeat comment wake batching", () => { gateway.releaseFirstWait(); - await waitFor(() => gateway.getAgentPayloads().length === 2, 30_000); + await waitFor(() => gateway.getAgentPayloads().length === 2); await waitFor(async () => { const runs = await db.select().from(heartbeatRuns).where(eq(heartbeatRuns.agentId, agentId)); return runs.length === 2 && runs.every((run) => run.status === "succeeded");