fix: make path tests cross-platform compatible

- Use path.join() instead of hardcoded Unix paths
- Use path.sep for platform-specific separators
- Ensures tests work correctly on both Unix and Windows

Author: daniel-lxs

src/services/code-index/shared/__tests__/get-relative-path.spec.ts

@@ -6,56 +6,56 @@ describe("get-relative-path", () => {
 	describe("generateNormalizedAbsolutePath", () => {
 		it("should use provided workspace root", () => {
 			const filePath = "src/file.ts"
-			const workspaceRoot = "/custom/workspace"
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
 			const result = generateNormalizedAbsolutePath(filePath, workspaceRoot)
-			expect(result).toBe(path.normalize("/custom/workspace/src/file.ts"))
+			expect(result).toBe(path.join(workspaceRoot, filePath))
 		})
 
 		it("should handle absolute paths", () => {
-			const filePath = "/absolute/path/file.ts"
-			const workspaceRoot = "/custom/workspace"
+			const filePath = path.join(path.sep, "absolute", "path", "file.ts")
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
 			const result = generateNormalizedAbsolutePath(filePath, workspaceRoot)
-			expect(result).toBe(path.normalize("/absolute/path/file.ts"))
+			expect(result).toBe(filePath)
 		})
 
 		it("should normalize paths with . and .. segments", () => {
 			const filePath = "./src/../src/file.ts"
-			const workspaceRoot = "/custom/workspace"
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
 			const result = generateNormalizedAbsolutePath(filePath, workspaceRoot)
-			expect(result).toBe(path.normalize("/custom/workspace/src/file.ts"))
+			expect(result).toBe(path.join(workspaceRoot, "src", "file.ts"))
 		})
 	})
 
 	describe("generateRelativeFilePath", () => {
 		it("should use provided workspace root", () => {
-			const absolutePath = "/custom/workspace/src/file.ts"
-			const workspaceRoot = "/custom/workspace"
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
+			const absolutePath = path.join(workspaceRoot, "src", "file.ts")
 			const result = generateRelativeFilePath(absolutePath, workspaceRoot)
-			expect(result).toBe(path.normalize("src/file.ts"))
+			expect(result).toBe(path.join("src", "file.ts"))
 		})
 
 		it("should handle paths outside workspace", () => {
-			const absolutePath = "/outside/workspace/file.ts"
-			const workspaceRoot = "/custom/workspace"
+			const absolutePath = path.join(path.sep, "outside", "workspace", "file.ts")
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
 			const result = generateRelativeFilePath(absolutePath, workspaceRoot)
 			// The result will have .. segments to navigate outside
 			expect(result).toContain("..")
 		})
 
 		it("should handle same path as workspace", () => {
-			const absolutePath = "/custom/workspace"
-			const workspaceRoot = "/custom/workspace"
+			const workspaceRoot = path.join(path.sep, "custom", "workspace")
+			const absolutePath = workspaceRoot
 			const result = generateRelativeFilePath(absolutePath, workspaceRoot)
 			expect(result).toBe(".")
 		})
 
 		it("should handle multi-workspace scenarios", () => {
 			// Simulate the error scenario from the issue
-			const absolutePath = "/Users/test/admin/.prettierrc.json"
-			const workspaceRoot = "/Users/test/project"
+			const workspaceRoot = path.join(path.sep, "Users", "test", "project")
+			const absolutePath = path.join(path.sep, "Users", "test", "admin", ".prettierrc.json")
 			const result = generateRelativeFilePath(absolutePath, workspaceRoot)
 			// Should generate a valid relative path, not throw an error
-			expect(result).toBe(path.normalize("../admin/.prettierrc.json"))
+			expect(result).toBe(path.join("..", "admin", ".prettierrc.json"))
 		})
 	})
 })