Merge pull request #37 from meilisearch/fix/issue-17-default-parameters-clean

Add test for issue #17: verify get-documents works without limit/offset parameters

Author: tpayet

tests/test_mcp_client.py

@@ -320,7 +320,7 @@ async def test_get_connection_settings_format(self, server):
 
 class TestIssue16GetDocumentsJsonSerialization:
     """Test for issue #16 - get-documents should return JSON, not Python object representations"""
-    
+
     @pytest.fixture
     async def server(self):
         """Create server instance for issue #16 tests"""
@@ -333,45 +333,52 @@ async def server(self):
     async def test_get_documents_returns_json_not_python_object(self, server):
         """Test that get-documents returns JSON-formatted text, not Python object string representation (issue #16)"""
         import time
+
         test_index = f"test_issue16_{int(time.time() * 1000)}"
-        
+
         # Create index and add a test document
         await simulate_mcp_call(server, "create-index", {"uid": test_index})
-        
+
         test_document = {"id": 1, "title": "Test Document", "content": "Test content"}
-        await simulate_mcp_call(server, "add-documents", {
-            "indexUid": test_index,
-            "documents": [test_document]
-        })
-        
+        await simulate_mcp_call(
+            server,
+            "add-documents",
+            {"indexUid": test_index, "documents": [test_document]},
+        )
+
         # Wait for indexing
         import asyncio
+
         await asyncio.sleep(0.5)
-        
+
         # Get documents with explicit parameters
-        result = await simulate_mcp_call(server, "get-documents", {
-            "indexUid": test_index,
-            "offset": 0,
-            "limit": 10
-        })
-        
+        result = await simulate_mcp_call(
+            server, "get-documents", {"indexUid": test_index, "offset": 0, "limit": 10}
+        )
+
         assert len(result) == 1
         assert result[0].type == "text"
-        
+
         response_text = result[0].text
-        
+
         # Issue #16 assertion: Should NOT contain Python object representation
-        assert "<meilisearch.models.document.DocumentsResults object at" not in response_text
+        assert (
+            "<meilisearch.models.document.DocumentsResults object at"
+            not in response_text
+        )
         assert "DocumentsResults" not in response_text
-        
+
         # Should contain proper JSON structure
         assert "Documents:" in response_text
-        assert "Test Document" in response_text  # Actual document content should be accessible
+        assert (
+            "Test Document" in response_text
+        )  # Actual document content should be accessible
         assert "Test content" in response_text
-        
+
         # Should be valid JSON after the "Documents:" prefix
         json_part = response_text.replace("Documents:", "").strip()
         import json
+
         try:
             parsed_data = json.loads(json_part)
             assert isinstance(parsed_data, dict)