From 8d2ccd851c9b096e32bdae2b4636e0bc5e524c18 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Tue, 7 Apr 2026 11:15:57 +0800
Subject: [PATCH] Tests: drop duplicate cron retry success case

---
 ...p-recipient-besteffortdeliver-true.test.ts | 26 -------------------
 1 file changed, 26 deletions(-)

diff --git a/src/cron/isolated-agent.skips-delivery-without-whatsapp-recipient-besteffortdeliver-true.test.ts b/src/cron/isolated-agent.skips-delivery-without-whatsapp-recipient-besteffortdeliver-true.test.ts
index db553815031..a7375835375 100644
--- a/src/cron/isolated-agent.skips-delivery-without-whatsapp-recipient-besteffortdeliver-true.test.ts
+++ b/src/cron/isolated-agent.skips-delivery-without-whatsapp-recipient-besteffortdeliver-true.test.ts
@@ -5,7 +5,6 @@ import { runSubagentAnnounceFlow } from "../agents/subagent-announce.js";
 import type { CliDeps } from "../cli/deps.js";
 import {
   createCliDeps,
-  expectDirectTelegramDelivery,
   mockAgentPayloads,
   runTelegramAnnounceTurn,
 } from "./isolated-agent.delivery.test-helpers.js";
@@ -111,22 +110,6 @@ function expectFailedTelegramDeliveryResult(params: {
   expect(params.deps.sendMessageTelegram).toHaveBeenCalledTimes(1);
 }
 
-async function runTelegramDeliveryResult(bestEffort: boolean) {
-  let outcome:
-    | {
-        res: Awaited<ReturnType<typeof runCronIsolatedAgentTurn>>;
-        deps: CliDeps;
-      }
-    | undefined;
-  await withTelegramTextDelivery({ bestEffort }, async ({ res, deps }) => {
-    outcome = { res, deps };
-  });
-  if (!outcome) {
-    throw new Error("telegram delivery did not produce an outcome");
-  }
-  return outcome;
-}
-
 function expectSuccessfulTelegramTextDelivery(params: {
   res: Awaited<ReturnType<typeof runCronIsolatedAgentTurn>>;
   deps: CliDeps;
@@ -207,15 +190,6 @@ describe("runCronIsolatedAgentTurn", () => {
     setupIsolatedAgentTurnMocks({ fast: true });
   });
 
-  it("delivers text directly when best-effort is disabled", async () => {
-    const { res, deps } = await runTelegramDeliveryResult(false);
-    expectSuccessfulTelegramTextDelivery({ res, deps });
-    expectDirectTelegramDelivery(deps, {
-      chatId: "123",
-      text: "hello from cron",
-    });
-  });
-
   it("retries transient text direct delivery failures before succeeding", async () => {
     const previousFastMode = process.env.OPENCLAW_TEST_FAST;
     process.env.OPENCLAW_TEST_FAST = "1";