From d2f5dfd24cfbbe5969b16e9780196f2595248de6 Mon Sep 17 00:00:00 2001 From: Hannes Rudolph Date: Tue, 23 Dec 2025 18:40:26 -0700 Subject: [PATCH] fix: update searchAndReplaceTool tests to match implementation - Remove expectation for recordToolUsage call (metrics are recorded by presentAssistantMessage) - Fix handleError assertion to use 'edit_file_anthropic' instead of 'search and replace' --- src/core/tools/__tests__/searchAndReplaceTool.spec.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core/tools/__tests__/searchAndReplaceTool.spec.ts b/src/core/tools/__tests__/searchAndReplaceTool.spec.ts index e35e27e35d..a8b933276c 100644 --- a/src/core/tools/__tests__/searchAndReplaceTool.spec.ts +++ b/src/core/tools/__tests__/searchAndReplaceTool.spec.ts @@ -331,7 +331,7 @@ describe("searchAndReplaceTool", () => { expect(mockTask.diffViewProvider.saveChanges).toHaveBeenCalled() expect(mockTask.didEditFile).toBe(true) - expect(mockTask.recordToolUsage).toHaveBeenCalledWith("edit_file_anthropic") + // Tool usage metrics are recorded by presentAssistantMessage(), not in the tool itself }) it("reverts changes when user rejects", async () => { @@ -390,7 +390,7 @@ describe("searchAndReplaceTool", () => { await executeSearchAndReplaceTool() - expect(mockHandleError).toHaveBeenCalledWith("search and replace", expect.any(Error)) + expect(mockHandleError).toHaveBeenCalledWith("edit_file_anthropic", expect.any(Error)) expect(mockTask.diffViewProvider.reset).toHaveBeenCalled() }) })