test: Refine logger mock and add lint suppressions

Author: alvinveroy

src/tests/agent.test.ts

@@ -166,13 +166,21 @@ describe('Agent', () => {
         // Object.defineProperty is used to make properties configurable and writable if needed,
         // but direct assignment should work for simple mocks if Dirent properties are writable.
         // For simplicity, we'll assume direct assignment works or cast.
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).name = name;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isFile = () => !isDir;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isDirectory = () => isDir;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isBlockDevice = () => false;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isCharacterDevice = () => false;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isSymbolicLink = () => false;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isFIFO = () => false;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (dirent as any).isSocket = () => false;
         // The 'path' property is not standard on Dirent from 'fs', it's usually inferred or constructed.
         // If your code relies on a 'path' or 'parentPath' property on Dirent objects *returned by readdir*,

src/tests/lib/query-refinement.helpers.test.ts

@@ -5,6 +5,8 @@ import { preprocessText } from '../../utils/text-utils';
 import { DetailedQdrantSearchResult } from '../../lib/types'; 
 
 vi.mock('../../utils/text-utils'); // Mock dependencies of helpers
+
+const mockLoggerInstance = { info: vi.fn(), warn: vi.fn(), error: vi.fn(), debug: vi.fn() };
 // Mock configService and logger if they are DIRECTLY used by these helpers
 // If they are only used by searchWithRefinement or refineQuery (dispatcher),
 // then this mock might not be needed here.
@@ -14,22 +16,22 @@ vi.mock('../../lib/config-service', () => ({
       // Add any config values DIRECTLY used by extractKeywords, broadenQuery, etc.
       // If none, this can be simpler or removed if logger isn't used by helpers.
     },
-    logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn(), debug: vi.fn() },
+    logger: mockLoggerInstance,
 }));
 
+// Import logger AFTER the mock is set up
+import { logger as mockedLoggerFromImport } from '../../lib/config-service'; // This will be mockLoggerInstance
 
 describe('Query Refinement Helper Utilities', () => {
   beforeEach(() => {
-    vi.clearAllMocks();
+    vi.clearAllMocks(); // This clears all mocks, including those on mockLoggerInstance
     vi.mocked(preprocessText).mockImplementation(text => text.trim().replace(/\s+/g, ' '));
-     
-    vi.mocked(logger.info).mockClear();
-     
-    vi.mocked(logger.warn).mockClear();
-     
-    vi.mocked(logger.error).mockClear();
-     
-    vi.mocked(logger.debug).mockClear();
+        
+    // Now use the imported mockedLoggerFromImport
+    mockedLoggerFromImport.info.mockClear();
+    mockedLoggerFromImport.warn.mockClear();
+    mockedLoggerFromImport.error.mockClear();
+    mockedLoggerFromImport.debug.mockClear();
   });
   afterEach(() => {
     vi.restoreAllMocks();