fix tests

Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

src/basic_memory/repository/relation_repository.py

@@ -140,7 +140,8 @@ async def add_all_ignore_duplicates(self, relations: List[Relation]) -> int:
                 stmt = stmt.on_conflict_do_nothing()
 
             result = await session.execute(stmt)
-            return result.rowcount if result.rowcount else 0
+            # rowcount can be -1 for ON CONFLICT DO NOTHING (undefined affected rows)
+            return result.rowcount if result.rowcount > 0 else 0
 
     def get_load_options(self) -> List[LoaderOption]:
         return [selectinload(Relation.from_entity), selectinload(Relation.to_entity)]

tests/services/test_initialization.py

@@ -182,7 +182,7 @@ async def test_initialize_file_sync_background_tasks(
 
 @pytest.mark.asyncio
 @patch("basic_memory.services.initialization.db.get_or_create_db")
-@patch("basic_memory.cli.commands.sync.get_sync_service")
+@patch("basic_memory.sync.sync_service.get_sync_service")
 @patch("basic_memory.sync.WatchService")
 @patch("basic_memory.services.initialization.asyncio.create_task")
 @patch.dict("os.environ", {"BASIC_MEMORY_MCP_PROJECT": "project1"})