test(cron): speed up isolated fallback tests

RomneyDa · RomneyDa · commit bb22882c06c6 · 2026-05-27T22:32:01.000-07:00
diff --git a/src/cron/isolated-agent/run.fast-mode.test.ts b/src/cron/isolated-agent/run.fast-mode.test.ts
@@ -25,7 +25,7 @@ function mockSuccessfulModelFallback() {
     return {
       result: {
         payloads: [{ text: "ok" }],
-        meta: { agentMeta: { usage: { input: 10, output: 20 } } },
+        meta: { agentMeta: {} },
       },
       provider,
       model,
@@ -126,7 +126,7 @@ async function runFastModeCase(params: {
 }
 
 describe("runCronIsolatedAgentTurn — fast mode", () => {
-  setupRunCronIsolatedAgentTurnSuite();
+  setupRunCronIsolatedAgentTurnSuite({ fast: true });
 
   it("passes config-driven fast mode into embedded cron runs", async () => {
     await runFastModeCase({
diff --git a/src/cron/isolated-agent/run.payload-fallbacks.test.ts b/src/cron/isolated-agent/run.payload-fallbacks.test.ts
@@ -9,6 +9,7 @@ import {
   loadRunCronIsolatedAgentTurn,
   mockRunCronFallbackPassthrough,
   resolveConfiguredModelRefMock,
+  resolveCliRuntimeExecutionProviderMock,
   resolveAgentModelFallbacksOverrideMock,
   runCliAgentMock,
   runEmbeddedAgentMock,
@@ -50,7 +51,7 @@ function requireEmbeddedRunRequest(): {
 }
 
 describe("runCronIsolatedAgentTurn — payload.fallbacks", () => {
-  setupRunCronIsolatedAgentTurnSuite();
+  setupRunCronIsolatedAgentTurnSuite({ fast: true });
 
   it.each([
     {
@@ -92,13 +93,16 @@ describe("runCronIsolatedAgentTurn — payload.fallbacks", () => {
 
   it("plans Anthropic fallbacks canonically while executing compatible attempts through Claude CLI", async () => {
     isCliProviderMock.mockImplementation((provider: string) => provider === "claude-cli");
+    resolveCliRuntimeExecutionProviderMock.mockImplementation(
+      ({ provider }: { provider: string }) => (provider === "anthropic" ? "claude-cli" : undefined),
+    );
     resolveConfiguredModelRefMock.mockReturnValue({
       provider: "anthropic",
       model: "claude-opus-4-6",
     });
     runCliAgentMock.mockResolvedValue({
       payloads: [{ text: "fallback ok" }],
-      meta: { agentMeta: { usage: { input: 10, output: 20 } } },
+      meta: { agentMeta: {} },
     });
     runWithModelFallbackMock.mockImplementation(async ({ provider, model, run }) => {
       const firstResult = await run(provider, model);
diff --git a/src/cron/isolated-agent/run.suite-helpers.ts b/src/cron/isolated-agent/run.suite-helpers.ts
@@ -8,10 +8,13 @@ import {
   restoreFastTestEnv,
 } from "./run.test-harness.js";
 
-export function setupRunCronIsolatedAgentTurnSuite() {
+export function setupRunCronIsolatedAgentTurnSuite(options?: { fast?: boolean }) {
   let previousFastTestEnv: string | undefined;
   beforeEach(() => {
     previousFastTestEnv = clearFastTestEnv();
+    if (options?.fast) {
+      process.env.OPENCLAW_TEST_FAST = "1";
+    }
     resetRunCronIsolatedAgentTurnHarness();
     resolveCronSessionMock.mockReturnValue(makeCronSession());
   });
diff --git a/src/cron/isolated-agent/run.test-harness.ts b/src/cron/isolated-agent/run.test-harness.ts
@@ -46,6 +46,7 @@ export const resolveAgentModelFallbacksOverrideMock = createMock();
 export const resolveAgentSkillsFilterMock = createMock();
 export const getModelRefStatusMock = createMock();
 export const isCliProviderMock = createMock();
+export const resolveCliRuntimeExecutionProviderMock = createMock();
 export const resolveAllowedModelRefMock = createMock();
 export const resolveConfiguredModelRefMock = createMock();
 export const resolveHooksGmailModelMock = createMock();
@@ -131,6 +132,13 @@ vi.mock("./run.runtime.js", () => ({
   getRemoteSkillEligibility: getRemoteSkillEligibilityMock,
 }));
 
+vi.mock("../../agents/model-runtime-aliases.js", async () => ({
+  ...(await vi.importActual<typeof import("../../agents/model-runtime-aliases.js")>(
+    "../../agents/model-runtime-aliases.js",
+  )),
+  resolveCliRuntimeExecutionProvider: resolveCliRuntimeExecutionProviderMock,
+}));
+
 vi.mock("./run-external-content.runtime.js", () => ({
   buildSafeExternalPrompt: buildSafeExternalPromptMock,
   detectSuspiciousPatterns: detectSuspiciousPatternsMock,
@@ -296,7 +304,7 @@ function makeDefaultModelFallbackResult() {
   return {
     result: {
       payloads: [{ text: "test output" }],
-      meta: { agentMeta: { usage: { input: 10, output: 20 } } },
+      meta: { agentMeta: {} },
     },
     provider: "openai",
     model: "gpt-5.4",
@@ -306,7 +314,7 @@ function makeDefaultModelFallbackResult() {
 function makeDefaultEmbeddedResult() {
   return {
     payloads: [{ text: "test output" }],
-    meta: { agentMeta: { usage: { input: 10, output: 20 } } },
+    meta: { agentMeta: {} },
   };
 }
 
@@ -379,6 +387,7 @@ function resetRunConfigMocks(): void {
   resolveAgentModelFallbacksOverrideMock.mockReturnValue(undefined);
   resolveAgentSkillsFilterMock.mockReturnValue(undefined);
   resolveConfiguredModelRefMock.mockReturnValue({ provider: "openai", model: "gpt-5.4" });
+  resolveCliRuntimeExecutionProviderMock.mockReturnValue(undefined);
   resolveAllowedModelRefMock.mockReturnValue({ ref: { provider: "openai", model: "gpt-5.4" } });
   resolveHooksGmailModelMock.mockReturnValue(null);
   resolveThinkingDefaultMock.mockReturnValue("off");
@@ -389,7 +398,11 @@ function resetRunConfigMocks(): void {
   });
   resolveAgentTimeoutMsMock.mockReturnValue(60_000);
   deriveSessionTotalTokensMock.mockReturnValue(30);
-  hasNonzeroUsageMock.mockReturnValue(true);
+  hasNonzeroUsageMock.mockImplementation(
+    (usage: { input?: unknown; output?: unknown } | undefined) =>
+      (typeof usage?.input === "number" && usage.input > 0) ||
+      (typeof usage?.output === "number" && usage.output > 0),
+  );
   ensureAgentWorkspaceMock.mockResolvedValue({ dir: "/tmp/workspace" });
   normalizeThinkLevelMock.mockImplementation((value: unknown) => value);
   isThinkingLevelSupportedMock.mockReturnValue(true);
