Fix get_stats() call and failing test

Author: rhaist

src/meilisearch_mcp/logging.py

@@ -1,7 +1,7 @@
 import logging
 import sys
 import json
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Optional, Dict, Any
 import threading
@@ -97,7 +97,7 @@ def _setup_logger(self, log_dir: Optional[str]):
     def _log(self, level: str, msg: str, **kwargs):
         """Create structured log entry"""
         log_entry = {
-            "timestamp": datetime.utcnow().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "level": level,
             "message": msg,
             **kwargs,

src/meilisearch_mcp/monitoring.py

@@ -36,7 +36,7 @@ def get_health_status(self) -> HealthStatus:
         """Get comprehensive health status"""
         try:
             # Get various stats to build health picture
-            stats = self.client.get_stats()
+            stats = self.client.get_all_stats()
             indexes = self.client.get_indexes()
 
             indexes_info = []

tests/test_docker_integration.py

@@ -3,6 +3,7 @@
 
 These tests verify that the Docker image can be built successfully.
 """
+
 import subprocess
 import pytest
 import shutil
@@ -16,10 +17,7 @@ def docker_available():
     # Try to run docker version to ensure it's working
     try:
         result = subprocess.run(
-            ["docker", "version"],
-            capture_output=True,
-            text=True,
-            timeout=5
+            ["docker", "version"], capture_output=True, text=True, timeout=5
         )
         return result.returncode == 0
     except (subprocess.TimeoutExpired, FileNotFoundError):
@@ -28,8 +26,7 @@ def docker_available():
 
 # Skip all tests in this module if Docker is not available
 pytestmark = pytest.mark.skipif(
-    not docker_available(),
-    reason="Docker not available on this system"
+    not docker_available(), reason="Docker not available on this system"
 )
 
 
@@ -38,7 +35,7 @@ def test_docker_build():
     result = subprocess.run(
         ["docker", "build", "-t", "meilisearch-mcp-test", "."],
         capture_output=True,
-        text=True
+        text=True,
     )
     assert result.returncode == 0, f"Docker build failed: {result.stderr}"
 
@@ -49,29 +46,35 @@ def test_docker_image_runs():
     build_result = subprocess.run(
         ["docker", "build", "-t", "meilisearch-mcp-test", "."],
         capture_output=True,
-        text=True
+        text=True,
     )
     if build_result.returncode != 0:
         pytest.skip(f"Docker build failed: {build_result.stderr}")
-    
+
     # Try to run the container and check it starts
     result = subprocess.run(
         [
-            "docker", "run", "--rm",
-            "-e", "MEILI_HTTP_ADDR=http://localhost:7700",
-            "-e", "MEILI_MASTER_KEY=test",
+            "docker",
+            "run",
+            "--rm",
+            "-e",
+            "MEILI_HTTP_ADDR=http://localhost:7700",
+            "-e",
+            "MEILI_MASTER_KEY=test",
             "meilisearch-mcp-test",
-            "python", "-c", "import src.meilisearch_mcp; print('MCP module loaded successfully')"
+            "python",
+            "-c",
+            "import src.meilisearch_mcp; print('MCP module loaded successfully')",
         ],
         capture_output=True,
         text=True,
-        timeout=30
+        timeout=30,
     )
-    
+
     assert result.returncode == 0, f"Docker run failed: {result.stderr}"
     assert "MCP module loaded successfully" in result.stdout
 
 
 if __name__ == "__main__":
     # Run tests
-    pytest.main([__file__, "-v"])
+    pytest.main([__file__, "-v"])

tests/test_mcp_client.py

@@ -577,8 +577,8 @@ async def test_delete_index_input_validation(self, mcp_server):
         """Test input validation for delete-index tool (issue #23)"""
         # Test missing uid parameter
         result = await simulate_mcp_call(mcp_server, "delete-index", {})
-        response_text = assert_text_content_response(result, "Error:")
-        assert "Error:" in response_text
+        response_text = assert_text_content_response(result, "error:")
+        assert "error:" in response_text
 
     async def test_delete_index_integration_workflow(self, mcp_server):
         """Test complete workflow: create -> add docs -> search -> delete (issue #23)"""