From 10e8e203f77f1531a743585d7c6910be77a55cdb Mon Sep 17 00:00:00 2001 From: Daniel Riccio Date: Tue, 8 Jul 2025 10:17:29 -0500 Subject: [PATCH] fix: make insertContentTool test platform-agnostic for Windows CI --- src/core/tools/__tests__/insertContentTool.spec.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/core/tools/__tests__/insertContentTool.spec.ts b/src/core/tools/__tests__/insertContentTool.spec.ts index caa25b77eb..14143fde36 100644 --- a/src/core/tools/__tests__/insertContentTool.spec.ts +++ b/src/core/tools/__tests__/insertContentTool.spec.ts @@ -12,7 +12,7 @@ vi.mock("path", async () => { const originalPath = await vi.importActual("path") return { ...originalPath, - resolve: vi.fn().mockImplementation((...args) => args.join("/")), + resolve: vi.fn(), } }) @@ -54,6 +54,7 @@ vi.mock("../../ignore/RooIgnoreController", () => ({ describe("insertContentTool", () => { const testFilePath = "test/file.txt" + // Use a consistent mock absolute path for testing const absoluteFilePath = "/test/file.txt" const mockedFileExistsAtPath = fileExistsAtPath as MockedFunction @@ -70,6 +71,7 @@ describe("insertContentTool", () => { beforeEach(() => { vi.clearAllMocks() + // Mock path.resolve to return our test absolute path mockedPathResolve.mockReturnValue(absoluteFilePath) mockedFileExistsAtPath.mockResolvedValue(true) // Assume file exists by default for insert mockedFsReadFile.mockResolvedValue("") // Default empty file content