From ab3d2b44ac0615a67a377c7a328f129f42f1faed Mon Sep 17 00:00:00 2001
From: wanglu241 <wanglu241@jd.com>
Date: Sat, 6 Jun 2026 11:12:36 +0800
Subject: [PATCH] test(announce-delivery): clean up temp session store on
 assertion failure

---
 src/agents/subagent-announce-delivery.test.ts | 84 ++++++++++---------
 1 file changed, 43 insertions(+), 41 deletions(-)

diff --git a/src/agents/subagent-announce-delivery.test.ts b/src/agents/subagent-announce-delivery.test.ts
index 29e82f3d6ec9..958314820204 100644
--- a/src/agents/subagent-announce-delivery.test.ts
+++ b/src/agents/subagent-announce-delivery.test.ts
@@ -4886,48 +4886,50 @@ describe("deliverSubagentAnnouncement requester session backfill (issue #86034)"
       "utf-8",
     );
 
-    const dispatchGatewayMethodInProcess = createInProcessGatewayMock({
-      result: { payloads: [{ text: "requester voice completion" }] },
-    });
-    testing.setDepsForTest({
-      dispatchGatewayMethodInProcess,
-      getRequesterSessionActivity: () => ({
-        sessionId: "telegram-session-1",
-        isActive: false,
-      }),
-      getRuntimeConfig: () => ({ session: { store: storeTemplate } }) as never,
-    });
+    try {
+      const dispatchGatewayMethodInProcess = createInProcessGatewayMock({
+        result: { payloads: [{ text: "requester voice completion" }] },
+      });
+      testing.setDepsForTest({
+        dispatchGatewayMethodInProcess,
+        getRequesterSessionActivity: () => ({
+          sessionId: "telegram-session-1",
+          isActive: false,
+        }),
+        getRuntimeConfig: () => ({ session: { store: storeTemplate } }) as never,
+      });
 
-    const result = await deliverSubagentAnnouncement({
-      requesterSessionKey: sessionKey,
-      targetRequesterSessionKey: sessionKey,
-      triggerMessage: "image done",
-      steerMessage: "image done",
-      // Origin carries channel/accountId but NOT `to` — simulates an
-      // image_generate task created off the direct-reply path.
-      requesterOrigin: { channel: "telegram", accountId: "bot-1" },
-      requesterSessionOrigin: { channel: "telegram", accountId: "bot-1" },
-      completionDirectOrigin: { channel: "telegram", accountId: "bot-1" },
-      directOrigin: { channel: "telegram", accountId: "bot-1" },
-      requesterIsSubagent: false,
-      expectsCompletionMessage: true,
-      bestEffortDeliver: true,
-      directIdempotencyKey: "announce-86034-backfill",
-      sourceTool: "image_generate",
-    });
+      const result = await deliverSubagentAnnouncement({
+        requesterSessionKey: sessionKey,
+        targetRequesterSessionKey: sessionKey,
+        triggerMessage: "image done",
+        steerMessage: "image done",
+        // Origin carries channel/accountId but NOT `to` — simulates an
+        // image_generate task created off the direct-reply path.
+        requesterOrigin: { channel: "telegram", accountId: "bot-1" },
+        requesterSessionOrigin: { channel: "telegram", accountId: "bot-1" },
+        completionDirectOrigin: { channel: "telegram", accountId: "bot-1" },
+        directOrigin: { channel: "telegram", accountId: "bot-1" },
+        requesterIsSubagent: false,
+        expectsCompletionMessage: true,
+        bestEffortDeliver: true,
+        directIdempotencyKey: "announce-86034-backfill",
+        sourceTool: "image_generate",
+      });
 
-    expectRecordFields(result, {
-      delivered: true,
-      path: "direct",
-    });
-    // The deliverability decision must see the backfilled `to`.
-    expectInProcessAgentParams(dispatchGatewayMethodInProcess, {
-      deliver: true,
-      channel: "telegram",
-      accountId: "bot-1",
-      to: "5866004662",
-    });
-
-    await fs.rm(storePath, { force: true });
+      expectRecordFields(result, {
+        delivered: true,
+        path: "direct",
+      });
+      // The deliverability decision must see the backfilled `to`.
+      expectInProcessAgentParams(dispatchGatewayMethodInProcess, {
+        deliver: true,
+        channel: "telegram",
+        accountId: "bot-1",
+        to: "5866004662",
+      });
+    } finally {
+      await fs.rm(storePath, { force: true });
+    }
   });
 });