test: stabilize Windows path-sensitive tests (normalize fs paths, deterministic roomodes/mcp discovery)

Author: roomote

src/core/config/__tests__/CustomModesManager.yamlEdgeCases.spec.ts

@@ -60,6 +60,7 @@ describe("CustomModesManager - YAML Edge Cases", () => {
 	}
 
 	beforeEach(() => {
+		process.env.NODE_ENV = "test"
 		mockOnUpdate = vi.fn()
 		mockContext = {
 			globalState: {
@@ -98,6 +99,10 @@ describe("CustomModesManager - YAML Edge Cases", () => {
 		}
 		;(vscode.workspace.createFileSystemWatcher as Mock).mockReturnValue(mockWatcher)
 
+		// Force deterministic paths independent of OS path semantics
+		vi.spyOn(CustomModesManager.prototype as any, "getHierarchicalRoomodes").mockResolvedValue([mockRoomodes])
+		vi.spyOn(CustomModesManager.prototype as any, "getCustomModesFilePath").mockResolvedValue(mockSettingsPath)
+
 		manager = new CustomModesManager(mockContext, mockOnUpdate)
 	})
 

src/services/mcp/__tests__/hierarchical-mcp.spec.ts

@@ -51,6 +51,12 @@ vi.mock("vscode", () => ({
 	},
 }))
 
+// Normalize paths across platforms for test lookups
+const normalizePathForTest = (p: string) =>
+	String(p)
+		.replace(/^[A-Za-z]:/i, "")
+		.replace(/\\/g, "/")
+
 describe("Hierarchical project MCP configuration resolution", () => {
 	const fileMap: Record<string, string> = {}
 	const workspaceRoot = "/home/user/mono-repo/packages/frontend"
@@ -63,18 +69,18 @@ describe("Hierarchical project MCP configuration resolution", () => {
 		vi.clearAllMocks()
 		process.env.NODE_ENV = "test"
 
-		// Prepare hierarchical files
-		fileMap[repoLevel] = JSON.stringify({
+		// Prepare hierarchical files (store using normalized keys)
+		fileMap[normalizePathForTest(repoLevel)] = JSON.stringify({
 			mcpServers: {
 				alpha: { type: "stdio", command: "node", args: ["repo.js"], disabled: true },
 			},
 		})
-		fileMap[packagesLevel] = JSON.stringify({
+		fileMap[normalizePathForTest(packagesLevel)] = JSON.stringify({
 			mcpServers: {
 				beta: { type: "stdio", command: "node", args: ["pkg.js"], disabled: true },
 			},
 		})
-		fileMap[appLevel] = JSON.stringify({
+		fileMap[normalizePathForTest(appLevel)] = JSON.stringify({
 			mcpServers: {
 				// Override alpha at most specific level
 				alpha: { type: "stdio", command: "node", args: ["app.js"], disabled: true },
@@ -83,14 +89,14 @@ describe("Hierarchical project MCP configuration resolution", () => {
 
 		// Configure fs.promises mocks
 		;(fs as any).access.mockImplementation(async (p: any) => {
-			const key = String(p)
+			const key = normalizePathForTest(String(p))
 			if (fileMap[key]) return
 			const err: any = new Error("ENOENT")
 			err.code = "ENOENT"
 			throw err
 		})
 		;(fs as any).readFile.mockImplementation(async (p: any, _enc?: any) => {
-			const key = String(p)
+			const key = normalizePathForTest(String(p))
 			if (fileMap[key]) return fileMap[key]
 			// default empty config to avoid syntax errors
 			return "{}"