fix tests

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

Author: phernandez

src/basic_memory/services/initialization.py

@@ -119,7 +119,7 @@ async def migrate_legacy_project_data(project: Project, legacy_dir: Path) -> boo
 async def initialize_file_sync(
     app_config: BasicMemoryConfig,
 ):
-    """Initialize file synchronization services.
+    """Initialize file synchronization services. This function starts the watch service and does not return
 
     Args:
         app_config: The Basic Memory project configuration
@@ -157,15 +157,15 @@ async def initialize_file_sync(
         sync_dir = Path(project.path)
 
         try:
-            #await sync_service.sync(sync_dir)
+            await sync_service.sync(sync_dir)
             logger.info(f"Sync completed successfully for project: {project.name}")
         except Exception as e:  # pragma: no cover
             logger.error(f"Error syncing project {project.name}: {e}")
             # Continue with other projects even if one fails
 
     # Then start the watch service in the background
     logger.info("Starting watch service for all projects")
-    # Create a background task for the watch service
+    # run the watch service
     try:
         await watch_service.run()
         logger.info("Watch service started")
@@ -215,5 +215,4 @@ def ensure_initialization(app_config: BasicMemoryConfig) -> None:
         logger.exception(f"Error during initialization: {e}")
         # Continue execution even if initialization fails
         # The command might still work, or will fail with a
-        # more specific error message
-
+        # more specific error message

tests/mcp/test_tool_utils.py

@@ -52,7 +52,9 @@ async def test_call_get_error(mock_response):
 async def test_call_post_success(mock_response):
     """Test successful POST request."""
     client = AsyncClient()
-    client.post = lambda *args, **kwargs: AsyncMock(return_value=mock_response())()
+    response = mock_response()
+    response.json = lambda: {"test": "data"}
+    client.post = lambda *args, **kwargs: AsyncMock(return_value=response)()
 
     response = await call_post(client, "http://test.com", json={"test": "data"})
     assert response.status_code == 200
@@ -62,7 +64,10 @@ async def test_call_post_success(mock_response):
 async def test_call_post_error(mock_response):
     """Test POST request with error."""
     client = AsyncClient()
-    client.post = lambda *args, **kwargs: AsyncMock(return_value=mock_response(500))()
+    response = mock_response(500)
+    response.json = lambda: {"test": "error"}
+    
+    client.post = lambda *args, **kwargs: AsyncMock(return_value=response)()
 
     with pytest.raises(ToolError) as exc:
         await call_post(client, "http://test.com", json={"test": "data"})
@@ -159,12 +164,15 @@ async def test_get_error_message():
 async def test_call_post_with_json(mock_response):
     """Test POST request with JSON payload."""
     client = AsyncClient()
-    mock_post = AsyncMock(return_value=mock_response())
+    response = mock_response()
+    response.json = lambda: {"test": "data"}
+    
+    mock_post = AsyncMock(return_value=response)
     client.post = mock_post
 
     json_data = {"key": "value", "nested": {"test": "data"}}
     await call_post(client, "http://test.com", json=json_data)
 
     mock_post.assert_called_once()
     call_kwargs = mock_post.call_args[1]
-    assert call_kwargs["json"] == json_data
+    assert call_kwargs["json"] == json_data

tests/services/test_initialization.py

@@ -53,7 +53,7 @@ async def test_initialize_app(
     mock_initialize_database.assert_called_once_with(app_config)
     mock_reconcile_projects.assert_called_once_with(app_config)
     mock_migrate_legacy_projects.assert_called_once_with(app_config)
-    mock_initialize_file_sync.assert_called_once_with(app_config)
+    mock_initialize_file_sync.assert_not_called()
     assert result is None
 
 
@@ -352,4 +352,4 @@ async def test_initialize_file_sync_sequential(
         mock_watch_service.run.assert_called_once()
 
         # Should return None
-        assert result is None
+        assert result is None