From 36ff031d726adeb69f54989a6178b29b17894386 Mon Sep 17 00:00:00 2001 From: Roo Code Date: Sun, 13 Jul 2025 15:16:38 +0000 Subject: [PATCH] fix: Update FileWatcher tests to work with new RooIgnoreController interface - Fixed FileWatcher test constructor to match new signature (removed ignore instance parameter) - Added proper mocking for RooIgnoreController.initialize() method - Enhanced ignore module mock to provide required add() and ignores() methods - All code-index tests now pass (341/341) --- .../processors/__tests__/file-watcher.spec.ts | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/src/services/code-index/processors/__tests__/file-watcher.spec.ts b/src/services/code-index/processors/__tests__/file-watcher.spec.ts index 2a3b7e1167..7378374621 100644 --- a/src/services/code-index/processors/__tests__/file-watcher.spec.ts +++ b/src/services/code-index/processors/__tests__/file-watcher.spec.ts @@ -18,9 +18,15 @@ vi.mock("../../cache-manager") vi.mock("../../../core/ignore/RooIgnoreController", () => ({ RooIgnoreController: vi.fn().mockImplementation(() => ({ validateAccess: vi.fn().mockReturnValue(true), + initialize: vi.fn().mockResolvedValue(undefined), + })), +})) +vi.mock("ignore", () => ({ + default: vi.fn(() => ({ + add: vi.fn(), + ignores: vi.fn().mockReturnValue(false), })), })) -vi.mock("ignore") vi.mock("../parser", () => ({ codeParser: { parseFile: vi.fn().mockResolvedValue([]), @@ -124,14 +130,7 @@ describe("FileWatcher", () => { ignores: vi.fn().mockReturnValue(false), } - fileWatcher = new FileWatcher( - "/mock/workspace", - mockContext, - mockCacheManager, - mockEmbedder, - mockVectorStore, - mockIgnoreInstance, - ) + fileWatcher = new FileWatcher("/mock/workspace", mockContext, mockCacheManager, mockEmbedder, mockVectorStore) }) describe("file filtering", () => {