test: tighten heartbeat runner assertions

steipete · steipete · commit c2a13e292f98 · 2026-05-11T12:15:17.000+01:00
diff --git a/src/infra/heartbeat-runner.respects-ackmaxchars-heartbeat-acks.test.ts b/src/infra/heartbeat-runner.respects-ackmaxchars-heartbeat-acks.test.ts
@@ -370,7 +370,10 @@ describe("runHeartbeatOnce ack handling", () => {
       });
 
       expect(res.status).toBe("skipped");
-      expect(res).toMatchObject({ reason: "whatsapp-not-linked" });
+      if (!("reason" in res)) {
+        throw new Error("expected skipped heartbeat result reason");
+      }
+      expect(res.reason).toBe("whatsapp-not-linked");
       expect(sendWhatsApp).not.toHaveBeenCalled();
     });
   });
@@ -405,11 +408,11 @@ describe("runHeartbeatOnce ack handling", () => {
       });
 
       expect(sendTelegram).toHaveBeenCalledTimes(1);
-      expect(sendTelegram).toHaveBeenCalledWith(
-        TELEGRAM_GROUP,
-        "Hello from heartbeat",
-        expect.objectContaining({ accountId: params.expectedAccountId, verbose: false }),
-      );
+      const [chatId, text, options] = sendTelegram.mock.calls[0] ?? [];
+      expect(chatId).toBe(TELEGRAM_GROUP);
+      expect(text).toBe("Hello from heartbeat");
+      expect(options?.accountId).toBe(params.expectedAccountId);
+      expect(options?.verbose).toBe(false);
     });
   }
 
diff --git a/src/infra/heartbeat-runner.subagent-session-guard.test.ts b/src/infra/heartbeat-runner.subagent-session-guard.test.ts
@@ -68,15 +68,11 @@ describe("runHeartbeatOnce", () => {
       });
 
       expect(replySpy).toHaveBeenCalledTimes(1);
-      expect(replySpy).toHaveBeenCalledWith(
-        expect.objectContaining({
-          SessionKey: mainSessionKey,
-          OriginatingChannel: undefined,
-          OriginatingTo: undefined,
-        }),
-        expect.anything(),
-        cfg,
-      );
+      const [replyParams, _replyRuntime, replyConfig] = replySpy.mock.calls[0] ?? [];
+      expect(replyParams?.SessionKey).toBe(mainSessionKey);
+      expect(replyParams?.OriginatingChannel).toBeUndefined();
+      expect(replyParams?.OriginatingTo).toBeUndefined();
+      expect(replyConfig).toBe(cfg);
     });
   });
 });
