test: use platform-agnostic tmp_path in DB mapping tests

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: georgeh0

tests/test_settings.py

@@ -214,51 +214,78 @@ def _clear_cache(self, monkeypatch: pytest.MonkeyPatch) -> Iterator[None]:
         yield
         _reset_db_path_mapping_cache()
 
-    def test_no_mapping(self) -> None:
-        assert resolve_db_dir(Path("/workspace/myproject")) == Path(
-            "/workspace/myproject/.cocoindex_code"
-        )
-
-    def test_single_mapping_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
-        assert resolve_db_dir(Path("/workspace/myproject")) == Path("/db-files/myproject")
-
-    def test_exact_root_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
-        assert resolve_db_dir(Path("/workspace")) == Path("/db-files")
-
-    def test_no_match_falls_back(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
-        assert resolve_db_dir(Path("/other/myproject")) == Path("/other/myproject/.cocoindex_code")
-
-    def test_multiple_mappings_first_wins(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db1,/workspace/sub=/db2")
-        assert resolve_db_dir(Path("/workspace/sub/proj")) == Path("/db1/sub/proj")
-
-    def test_multiple_mappings_second_matches(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db1,/other=/db2")
-        assert resolve_db_dir(Path("/other/proj")) == Path("/db2/proj")
-
-    def test_no_partial_component_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
-        assert resolve_db_dir(Path("/workspace2/proj")) == Path("/workspace2/proj/.cocoindex_code")
+    def test_no_mapping(self, tmp_path: Path) -> None:
+        project = tmp_path / "myproject"
+        assert resolve_db_dir(project) == project / ".cocoindex_code"
+
+    def test_single_mapping_match(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src = tmp_path / "workspace"
+        dst = tmp_path / "db-files"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst}")
+        assert resolve_db_dir(src / "myproject") == dst / "myproject"
+
+    def test_exact_root_match(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src = tmp_path / "workspace"
+        dst = tmp_path / "db-files"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst}")
+        assert resolve_db_dir(src) == dst
+
+    def test_no_match_falls_back(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src = tmp_path / "workspace"
+        dst = tmp_path / "db-files"
+        other = tmp_path / "other" / "myproject"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst}")
+        assert resolve_db_dir(other) == other / ".cocoindex_code"
+
+    def test_multiple_mappings_first_wins(
+        self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
+    ) -> None:
+        src = tmp_path / "workspace"
+        dst1 = tmp_path / "db1"
+        dst2 = tmp_path / "db2"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst1},{src / 'sub'}={dst2}")
+        assert resolve_db_dir(src / "sub" / "proj") == dst1 / "sub" / "proj"
+
+    def test_multiple_mappings_second_matches(
+        self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
+    ) -> None:
+        src1 = tmp_path / "workspace"
+        src2 = tmp_path / "other"
+        dst1 = tmp_path / "db1"
+        dst2 = tmp_path / "db2"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src1}={dst1},{src2}={dst2}")
+        assert resolve_db_dir(src2 / "proj") == dst2 / "proj"
+
+    def test_no_partial_component_match(
+        self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
+    ) -> None:
+        src = tmp_path / "workspace"
+        dst = tmp_path / "db-files"
+        other = tmp_path / "workspace2" / "proj"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst}")
+        assert resolve_db_dir(other) == other / ".cocoindex_code"
 
     def test_rejects_relative_source(self, monkeypatch: pytest.MonkeyPatch) -> None:
         monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "relative/path=/db-files")
         with pytest.raises(ValueError, match="source path must be absolute"):
             resolve_db_dir(Path("/anything"))
 
-    def test_rejects_relative_target(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=relative/path")
+    def test_rejects_relative_target(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src = tmp_path / "workspace"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}=relative/path")
         with pytest.raises(ValueError, match="target path must be absolute"):
-            resolve_db_dir(Path("/anything"))
-
-    def test_skips_empty_entries(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files,,/other=/db2,")
-        assert resolve_db_dir(Path("/other/proj")) == Path("/db2/proj")
-
-    def test_nested_project(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
-        assert resolve_db_dir(Path("/workspace/org/repo/subdir")) == Path(
-            "/db-files/org/repo/subdir"
-        )
+            resolve_db_dir(tmp_path / "anything")
+
+    def test_skips_empty_entries(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src1 = tmp_path / "workspace"
+        src2 = tmp_path / "other"
+        dst1 = tmp_path / "db-files"
+        dst2 = tmp_path / "db2"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src1}={dst1},,{src2}={dst2},")
+        assert resolve_db_dir(src2 / "proj") == dst2 / "proj"
+
+    def test_nested_project(self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+        src = tmp_path / "workspace"
+        dst = tmp_path / "db-files"
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", f"{src}={dst}")
+        assert resolve_db_dir(src / "org" / "repo" / "subdir") == dst / "org" / "repo" / "subdir"