fix(test): add missing media dedup state fields to mock contexts

Pre-existing test mocks lacked pendingMessagingMediaUrls and
messagingToolSentMediaUrls fields added by the media dedup feature,
causing runtime errors in handleToolExecutionEnd.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Yaroslav Boiko
2026-02-16 22:48:58 +01:00
committed by Peter Steinberger
parent 838259331f
commit a02bcb3620
4 changed files with 5 additions and 1 deletions

View File

@@ -57,6 +57,7 @@ const makeAttempt = (overrides: Partial<EmbeddedRunAttemptResult>): EmbeddedRunA
lastAssistant: undefined,
didSendViaMessagingTool: false,
messagingToolSentTexts: [],
messagingToolSentMediaUrls: [],
messagingToolSentTargets: [],
cloudCodeAssistFormatError: false,
...overrides,

View File

@@ -23,8 +23,10 @@ function createMockContext(overrides?: {
toolSummaryById: new Set(),
pendingMessagingTexts: new Map(),
pendingMessagingTargets: new Map(),
pendingMessagingMediaUrls: new Map(),
messagingToolSentTexts: [],
messagingToolSentTextsNormalized: [],
messagingToolSentMediaUrls: [],
messagingToolSentTargets: [],
},
log: { debug: vi.fn(), warn: vi.fn() },

View File

@@ -264,6 +264,5 @@ describe("messaging tool media URL tracking", () => {
expect(ctx.state.messagingToolSentMediaUrls).toHaveLength(0);
expect(ctx.state.pendingMessagingMediaUrls.has("tool-m3")).toBe(false);
>>>>>>> 018297172 (test(media-dedup): add missing coverage for Discord media dedup wiring)
});
});

View File

@@ -41,8 +41,10 @@ function createToolHandlerCtx(params: {
lastToolError: undefined,
pendingMessagingTexts: new Map<string, string>(),
pendingMessagingTargets: new Map<string, unknown>(),
pendingMessagingMediaUrls: new Map<string, string>(),
messagingToolSentTexts: [] as string[],
messagingToolSentTextsNormalized: [] as string[],
messagingToolSentMediaUrls: [] as string[],
messagingToolSentTargets: [] as unknown[],
blockBuffer: "",
},