Aktualisiert CodeIndex-Tests für Singleton-Muster

Passt die Testfälle an das neue Singleton-Muster des CodeIndexManagers an:

- Ersetzt direkte Instanziierung durch getInstance-Aufrufe
- Aktualisiert Methodennamen gemäß neuer API
- Fügt disposeAll-Aufrufe für bessere Testisolierung hinzu
- Verbessert Null-Prüfungen in Qdrant-Tests

Die Änderungen gewährleisten die korrekte Funktionsweise der Tests mit der überarbeiteten Architektur.

Author: yavpungggi

src/services/code-index/__tests__/manager-watcher-integration.spec.ts

@@ -10,11 +10,11 @@ vi.mock("../service-factory")
 vi.mock("../git-branch-watcher")
 vi.mock("../../../utils/git")
 
-import { ServiceFactory } from "../service-factory"
+import { CodeIndexServiceFactory } from "../service-factory"
 import { GitBranchWatcher } from "../git-branch-watcher"
 import { getCurrentBranch } from "../../../utils/git"
 
-const mockedServiceFactory = vi.mocked(ServiceFactory)
+const mockedServiceFactory = vi.mocked(CodeIndexServiceFactory)
 const mockedGitBranchWatcher = vi.mocked(GitBranchWatcher)
 const mockedGetCurrentBranch = vi.mocked(getCurrentBranch)
 
@@ -97,15 +97,16 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		if (manager) {
 			manager.dispose()
 		}
+		CodeIndexManager.disposeAll()
 		branchChangeCallback = null
 	})
 
 	describe("branch change handling", () => {
 		it("should invalidate cache and reinitialize vector store on branch change", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
 			// Start the manager
-			await manager.start()
+			await manager.startIndexing()
 
 			expect(mockWatcher.initialize).toHaveBeenCalled()
 			expect(mockOrchestrator.startIndexing).toHaveBeenCalled()
@@ -129,9 +130,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		})
 
 		it("should recreate services if orchestrator doesn't exist", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			// Simulate orchestrator being disposed
 			mockOrchestrator.getVectorStore.mockReturnValue(null)
@@ -148,9 +149,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		})
 
 		it("should handle branch change errors gracefully", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			// Make vector store initialization fail
 			mockVectorStore.initialize.mockRejectedValueOnce(new Error("Init failed"))
@@ -168,10 +169,10 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		})
 
 		it("should not process branch changes when manager is stopped", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
-			await manager.stop()
+			await manager.startIndexing()
+			manager.stopWatcher()
 
 			vi.clearAllMocks()
 
@@ -187,9 +188,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 
 	describe("watcher lifecycle", () => {
 		it("should initialize watcher when manager starts", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			expect(mockedGitBranchWatcher).toHaveBeenCalledWith(
 				"/test/workspace",
@@ -203,9 +204,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		})
 
 		it("should dispose watcher when manager is disposed", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			manager.dispose()
 
@@ -219,19 +220,19 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 				embedderProvider: "openai",
 			})
 
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			expect(mockedGitBranchWatcher).not.toHaveBeenCalled()
 		})
 	})
 
 	describe("state consistency", () => {
 		it("should maintain consistent state across multiple branch changes", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			// First branch change
 			mockVectorStore.getCurrentBranch.mockReturnValue("feature-1")
@@ -261,9 +262,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 		})
 
 		it("should handle rapid branch changes with debouncing", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			// Simulate rapid branch changes (watcher handles debouncing)
 			// The callback should only be called once per actual change
@@ -280,9 +281,9 @@ describe("CodeIndexManager + GitBranchWatcher Integration", () => {
 
 	describe("error recovery", () => {
 		it("should recover from vector store initialization failure", async () => {
-			manager = new CodeIndexManager(mockContext, "/test/workspace")
+			manager = CodeIndexManager.getInstance(mockContext, "/test/workspace")!
 
-			await manager.start()
+			await manager.startIndexing()
 
 			// First branch change fails
 			mockVectorStore.initialize.mockRejectedValueOnce(new Error("Init failed"))

src/services/code-index/vector-store/__tests__/qdrant-client.branch-isolation.spec.ts

@@ -474,8 +474,8 @@ describe("QdrantVectorStore - Branch Isolation", () => {
 
 			// Verify results are from feature branch only
 			expect(searchResults).toHaveLength(1)
-			expect(searchResults[0].payload.filePath).toBe("feature.ts")
-			expect(searchResults[0].payload.codeChunk).toBe("feature branch code")
+			expect(searchResults[0]?.payload?.filePath).toBe("feature.ts")
+			expect(searchResults[0]?.payload?.codeChunk).toBe("feature branch code")
 
 			// Verify main branch document is NOT in results
 			expect(searchResults).not.toContainEqual(
@@ -550,7 +550,7 @@ describe("QdrantVectorStore - Branch Isolation", () => {
 			expect(mockQdrantClient.query).toHaveBeenCalledWith(mainCollectionName, expect.any(Object))
 
 			// Should get main branch results
-			expect(results[0].payload.filePath).toBe("main.ts")
+			expect(results[0]?.payload?.filePath).toBe("main.ts")
 		})
 	})
 })