Fixed more WindowsPath issues mostly in assertions of unit tests

Signed-off-by: Manuel Bliemel <manuel.bliemel@gmail.com>

Author: manuelbliemel

src/basic_memory/config.py

@@ -100,9 +100,9 @@ def model_post_init(self, __context: Any) -> None:
         """Ensure configuration is valid after initialization."""
         # Ensure main project exists
         if "main" not in self.projects:  # pragma: no cover
-            self.projects["main"] = str(
+            self.projects["main"] = (
                 Path(os.getenv("BASIC_MEMORY_HOME", Path.home() / "basic-memory"))
-            )
+            ).as_posix()
 
         # Ensure default project is valid
         if self.default_project not in self.projects:  # pragma: no cover

tests/api/test_project_router.py

@@ -256,8 +256,8 @@ async def test_update_project_both_params_endpoint(
     """Test the update project endpoint with both path and is_active parameters."""
     # Create a test project to update
     test_project_name = "test-update-both-project"
-    old_path = str(tmp_path / "old-location")
-    new_path = str(tmp_path / "new-location")
+    old_path = (tmp_path / "old-location").as_posix()
+    new_path = (tmp_path / "new-location").as_posix()
 
     await project_service.add_project(test_project_name, old_path)
 

tests/services/test_project_service.py

@@ -1,6 +1,7 @@
 """Tests for ProjectService."""
 
 import os
+from pathlib import Path
 
 import pytest
 
@@ -73,7 +74,7 @@ async def test_project_operations_sync_methods(
     """
     # Generate a unique project name for testing
     test_project_name = f"test-project-{os.urandom(4).hex()}"
-    test_project_path = str(tmp_path / "test-project")
+    test_project_path = (tmp_path / "test-project").as_posix()
 
     # Make sure the test directory exists
     os.makedirs(test_project_path, exist_ok=True)
@@ -590,8 +591,8 @@ async def test_move_project_nonexistent(project_service: ProjectService, tmp_pat
 async def test_move_project_db_mismatch(project_service: ProjectService, tmp_path):
     """Test moving a project that exists in config but not in database."""
     test_project_name = f"test-move-mismatch-{os.urandom(4).hex()}"
-    old_path = str(tmp_path / "old-location")
-    new_path = str(tmp_path / "new-location")
+    old_path = (tmp_path / "old-location").as_posix()
+    new_path = (tmp_path / "new-location").as_posix()
 
     # Create directories
     os.makedirs(old_path, exist_ok=True)
@@ -624,7 +625,7 @@ async def test_move_project_db_mismatch(project_service: ProjectService, tmp_pat
 async def test_move_project_expands_path(project_service: ProjectService, tmp_path):
     """Test that move_project expands ~ and relative paths."""
     test_project_name = f"test-move-expand-{os.urandom(4).hex()}"
-    old_path = str(tmp_path / "old-location")
+    old_path = (tmp_path / "old-location").as_posix()
 
     # Create old directory
     os.makedirs(old_path, exist_ok=True)
@@ -635,7 +636,7 @@ async def test_move_project_expands_path(project_service: ProjectService, tmp_pa
 
         # Use a relative path for the move
         relative_new_path = "./new-location"
-        expected_absolute_path = os.path.abspath(relative_new_path)
+        expected_absolute_path = Path(os.path.abspath(relative_new_path)).as_posix()
 
         # Move project using relative path
         await project_service.move_project(test_project_name, relative_new_path)

tests/services/test_project_service_operations.py

@@ -48,7 +48,7 @@ async def test_add_project_to_config(project_service: ProjectService, tmp_path,
     """Test adding a project to the config manager."""
     # Generate unique project name for testing
     test_project_name = f"config-project-{os.urandom(4).hex()}"
-    test_path = str(tmp_path / "config-project")
+    test_path = (tmp_path / "config-project").as_posix()
 
     # Make sure directory exists
     os.makedirs(test_path, exist_ok=True)

tests/test_config.py

@@ -1,6 +1,5 @@
 """Test configuration management."""
 
-from pathlib import Path
 from basic_memory.config import BasicMemoryConfig
 
 
@@ -15,7 +14,7 @@ def test_default_behavior_without_basic_memory_home(self, config_home, monkeypat
         config = BasicMemoryConfig()
 
         # Should use the default path (home/basic-memory)
-        expected_path = Path(config_home / "basic-memory").as_posix()
+        expected_path = (config_home / "basic-memory").as_posix()
         assert config.projects["main"] == expected_path
 
     def test_respects_basic_memory_home_environment_variable(self, config_home, monkeypatch):
@@ -47,7 +46,7 @@ def test_model_post_init_fallback_without_basic_memory_home(self, config_home, m
         monkeypatch.delenv("BASIC_MEMORY_HOME", raising=False)
 
         # Create config without main project
-        other_path = Path(config_home / "some" / "path").as_posix()
+        other_path = (config_home / "some" / "path").as_posix()
         config = BasicMemoryConfig(projects={"other": other_path})
 
         # model_post_init should have added main project with default path