From 5dc7a05f7f2d44279b5ea5b9f55451bed245c365 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Mon, 11 May 2026 16:51:54 +0100
Subject: [PATCH] test: remove acp approval retry timer flush

---
 src/acp/translator.permission-relay.test.ts | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/src/acp/translator.permission-relay.test.ts b/src/acp/translator.permission-relay.test.ts
index a659dd31697..3282711d1ac 100644
--- a/src/acp/translator.permission-relay.test.ts
+++ b/src/acp/translator.permission-relay.test.ts
@@ -139,6 +139,15 @@ function approvalResolveCalls(request: ReturnType<typeof vi.fn>) {
   return request.mock.calls.filter(([method]) => method === "exec.approval.resolve");
 }
 
+function hasApprovalRelay(agent: AcpGatewayAgent, approvalId: string): boolean {
+  const relayMap = (
+    agent as unknown as {
+      approvalRelays: Map<string, unknown>;
+    }
+  ).approvalRelays;
+  return relayMap.has(approvalId);
+}
+
 function requireRecord(value: unknown): Record<string, unknown> {
   expect(value).toBeTruthy();
   expect(typeof value).toBe("object");
@@ -319,7 +328,9 @@ describe("ACP translator permission relay", () => {
       expect(harness.requestPermission).toHaveBeenCalledTimes(1);
       expect(resolveApproval).toHaveBeenCalledTimes(1);
     });
-    await new Promise((resolve) => setTimeout(resolve, 0));
+    await vi.waitFor(() => {
+      expect(hasApprovalRelay(harness.agent, "approval-retry")).toBe(false);
+    });
 
     await harness.agent.handleGatewayEvent(event);