fix: disable pylint import-error for optional MCP dependency

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: root

tests/test_mcp_server.py

@@ -6,7 +6,7 @@
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import pytest
-from mcp.types import TextContent, Tool
+from mcp.types import TextContent, Tool  # pylint: disable=import-error
 
 # Import the module functions and server instance
 from gitingest.mcp_server import (
@@ -23,7 +23,7 @@ class TestMCPListTools:
     """Test cases for the list_tools handler."""
 
     @pytest.mark.asyncio
-    async def test_list_tools_returns_correct_tools(self):
+    async def test_list_tools_returns_correct_tools(self) -> None:
         """Test that list_tools returns the expected tools."""
         tools = await list_tools()
 
@@ -36,7 +36,7 @@ async def test_list_tools_returns_correct_tools(self):
         assert "ingest a git repository" in tool.description.lower()
 
     @pytest.mark.asyncio
-    async def test_list_tools_schema_validation(self):
+    async def test_list_tools_schema_validation(self) -> None:
         """Test that the ingest_repository tool has correct schema."""
         tools = await list_tools()
         ingest_tool = tools[0]
@@ -69,14 +69,15 @@ async def test_list_tools_schema_validation(self):
             assert param in properties
 
     @pytest.mark.asyncio
-    async def test_list_tools_source_examples(self):
+    async def test_list_tools_source_examples(self) -> None:
         """Test that the source parameter has proper examples."""
         tools = await list_tools()
         source_prop = tools[0].inputSchema["properties"]["source"]
 
         assert "examples" in source_prop
         examples = source_prop["examples"]
-        assert len(examples) >= 3
+        min_examples = 3
+        assert len(examples) >= min_examples
         assert any("github.com" in ex for ex in examples)
         assert any("/path/to/" in ex for ex in examples)
         assert "." in examples
@@ -86,7 +87,7 @@ class TestMCPCallTool:
     """Test cases for the call_tool handler."""
 
     @pytest.mark.asyncio
-    async def test_call_tool_ingest_repository_success(self):
+    async def test_call_tool_ingest_repository_success(self) -> None:
         """Test successful repository ingestion through call_tool."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = (
@@ -110,7 +111,7 @@ async def test_call_tool_ingest_repository_success(self):
             assert "Generated by Gitingest MCP Server" in content
 
     @pytest.mark.asyncio
-    async def test_call_tool_unknown_tool(self):
+    async def test_call_tool_unknown_tool(self) -> None:
         """Test handling of unknown tool calls."""
         result = await call_tool("unknown_tool", {})
 
@@ -120,7 +121,7 @@ async def test_call_tool_unknown_tool(self):
         assert "Unknown tool: unknown_tool" in result[0].text
 
     @pytest.mark.asyncio
-    async def test_call_tool_exception_handling(self):
+    async def test_call_tool_exception_handling(self) -> None:
         """Test exception handling in call_tool."""
         with patch("gitingest.mcp_server._handle_ingest_repository") as mock_handle:
             mock_handle.side_effect = Exception("Test exception")
@@ -132,7 +133,7 @@ async def test_call_tool_exception_handling(self):
             assert "Error executing ingest_repository: Test exception" in result[0].text
 
     @pytest.mark.asyncio
-    async def test_call_tool_logs_errors(self):
+    async def test_call_tool_logs_errors(self) -> None:
         """Test that call_tool logs errors properly."""
         with (
             patch("gitingest.mcp_server._handle_ingest_repository") as mock_handle,
@@ -144,15 +145,15 @@ async def test_call_tool_logs_errors(self):
             await call_tool("ingest_repository", {"source": "test"})
 
             mock_logger.exception.assert_called_once()
-            args, kwargs = mock_logger.exception.call_args
-            assert "Error in tool call ingest_repository" in args[0]
+            args, _kwargs = mock_logger.exception.call_args
+            assert args == ("Error in tool call %s", "ingest_repository")
 
 
 class TestHandleIngestRepository:
     """Test cases for the _handle_ingest_repository helper function."""
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_minimal_args(self):
+    async def test_handle_ingest_repository_minimal_args(self) -> None:
         """Test repository ingestion with minimal arguments."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
@@ -177,7 +178,7 @@ async def test_handle_ingest_repository_minimal_args(self):
             )
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_all_args(self):
+    async def test_handle_ingest_repository_all_args(self) -> None:
         """Test repository ingestion with all arguments."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
@@ -207,12 +208,12 @@ async def test_handle_ingest_repository_all_args(self):
                 branch="develop",
                 include_gitignored=True,
                 include_submodules=True,
-                token="ghp_test_token",
+                token="test_token_123",  # noqa: S106
                 output=None,
             )
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_pattern_conversion(self):
+    async def test_handle_ingest_repository_pattern_conversion(self) -> None:
         """Test that patterns are correctly converted to sets."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
@@ -232,7 +233,7 @@ async def test_handle_ingest_repository_pattern_conversion(self):
             assert call_args["exclude_patterns"] == {"*.txt"}
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_none_patterns(self):
+    async def test_handle_ingest_repository_none_patterns(self) -> None:
         """Test handling of None patterns."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
@@ -250,7 +251,7 @@ async def test_handle_ingest_repository_none_patterns(self):
             assert call_args["exclude_patterns"] is None
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_exception(self):
+    async def test_handle_ingest_repository_exception(self) -> None:
         """Test exception handling in _handle_ingest_repository."""
         with (
             patch("gitingest.mcp_server.ingest_async") as mock_ingest,
@@ -267,11 +268,11 @@ async def test_handle_ingest_repository_exception(self):
 
             # Verify error was logged
             mock_logger.exception.assert_called_once()
-            args, kwargs = mock_logger.exception.call_args
-            assert "Error during ingestion" in args[0]
+            args, _kwargs = mock_logger.exception.call_args
+            assert args == ("Error during ingestion",)
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_logs_info(self):
+    async def test_handle_ingest_repository_logs_info(self) -> None:
         """Test that _handle_ingest_repository logs info messages."""
         with (
             patch("gitingest.mcp_server.ingest_async") as mock_ingest,
@@ -289,7 +290,7 @@ async def test_handle_ingest_repository_logs_info(self):
             )
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_response_format(self):
+    async def test_handle_ingest_repository_response_format(self) -> None:
         """Test the format of the response content."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = (
@@ -317,13 +318,13 @@ class TestMCPServerIntegration:
     """Integration tests for the MCP server."""
 
     @pytest.mark.asyncio
-    async def test_server_instance_created(self):
+    async def test_server_instance_created(self) -> None:
         """Test that the MCP server instance is properly created."""
         assert app is not None
         assert app.name == "gitingest"
 
     @pytest.mark.asyncio
-    async def test_start_mcp_server_calls_stdio(self):
+    async def test_start_mcp_server_calls_stdio(self) -> None:
         """Test that start_mcp_server calls the stdio runner."""
         with patch("gitingest.mcp_server._run_stdio") as mock_run_stdio:
             mock_run_stdio.return_value = AsyncMock()
@@ -333,7 +334,7 @@ async def test_start_mcp_server_calls_stdio(self):
             mock_run_stdio.assert_called_once()
 
     @pytest.mark.asyncio
-    async def test_start_mcp_server_logs_startup(self):
+    async def test_start_mcp_server_logs_startup(self) -> None:
         """Test that start_mcp_server logs startup message."""
         with (
             patch("gitingest.mcp_server._run_stdio") as mock_run_stdio,
@@ -348,7 +349,7 @@ async def test_start_mcp_server_logs_startup(self):
             )
 
     @pytest.mark.asyncio
-    async def test_run_stdio_integration(self):
+    async def test_run_stdio_integration(self) -> None:
         """Test _run_stdio function integration."""
         with patch("gitingest.mcp_server.stdio_server") as mock_stdio_server:
             # Mock the async context manager
@@ -374,14 +375,15 @@ async def test_run_stdio_integration(self):
                 # Verify app.run was called with streams and init options
                 mock_run.assert_called_once()
                 call_args = mock_run.call_args[0]
-                assert len(call_args) == 3  # read_stream, write_stream, init_options
+                expected_args = 3  # read_stream, write_stream, init_options
+                assert len(call_args) == expected_args
 
 
 class TestMCPServerParameterValidation:
     """Test parameter validation for MCP server tools."""
 
     @pytest.mark.asyncio
-    async def test_ingest_repository_missing_source(self):
+    async def test_ingest_repository_missing_source(self) -> None:
         """Test that missing source parameter is handled."""
         # This should raise a KeyError which gets caught by call_tool
         result = await call_tool("ingest_repository", {})
@@ -391,7 +393,7 @@ async def test_ingest_repository_missing_source(self):
         assert "Error ingesting repository" in result[0].text
 
     @pytest.mark.asyncio
-    async def test_ingest_repository_invalid_parameters(self):
+    async def test_ingest_repository_invalid_parameters(self) -> None:
         """Test handling of invalid parameter types."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             # ingest_async should handle type validation, but let's test edge cases
@@ -410,7 +412,7 @@ async def test_ingest_repository_invalid_parameters(self):
             assert "Error ingesting repository: Invalid parameter type" in result[0].text
 
     @pytest.mark.asyncio
-    async def test_ingest_repository_empty_patterns(self):
+    async def test_ingest_repository_empty_patterns(self) -> None:
         """Test handling of empty pattern lists."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
@@ -433,7 +435,7 @@ class TestMCPServerEdgeCases:
     """Test edge cases and error scenarios."""
 
     @pytest.mark.asyncio
-    async def test_call_tool_empty_arguments(self):
+    async def test_call_tool_empty_arguments(self) -> None:
         """Test call_tool with empty arguments dict."""
         result = await call_tool("ingest_repository", {})
 
@@ -442,7 +444,7 @@ async def test_call_tool_empty_arguments(self):
         assert "Error ingesting repository" in result[0].text
 
     @pytest.mark.asyncio
-    async def test_handle_ingest_repository_partial_results(self):
+    async def test_handle_ingest_repository_partial_results(self) -> None:
         """Test handling when ingest_async returns partial results."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             # Test with empty strings
@@ -459,17 +461,18 @@ async def test_handle_ingest_repository_partial_results(self):
             assert "## Content" in content
 
     @pytest.mark.asyncio
-    async def test_concurrent_tool_calls(self):
+    async def test_concurrent_tool_calls(self) -> None:
         """Test that concurrent tool calls work correctly."""
         with patch("gitingest.mcp_server.ingest_async") as mock_ingest:
             mock_ingest.return_value = ("summary", "tree", "content")
 
             # Create multiple concurrent calls
-            tasks = [call_tool("ingest_repository", {"source": f"test-{i}"}) for i in range(3)]
+            num_tasks = 3
+            tasks = [call_tool("ingest_repository", {"source": f"test-{i}"}) for i in range(num_tasks)]
 
             results = await asyncio.gather(*tasks)
 
-            assert len(results) == 3
+            assert len(results) == num_tasks
             for result in results:
                 assert isinstance(result, list)
                 assert len(result) == 1