chore: add dependency locking step in CI workflow and improve type hinting in tests

Author: pengqun

.github/workflows/test.yml

@@ -40,6 +40,9 @@ jobs:
           uv.lock
           pyproject.toml
 
+    - name: Lock dependencies for current matrix
+      run: uv lock
+
     - name: Install dependencies
       run: uv sync --locked
 

tests/tools/test_aigc_error_handling.py

@@ -69,7 +69,7 @@ async def test_polling_task_failed_status(self, mcp_server, mocker):
         )
 
         # Mock polling to return FAILED status
-        mock_get = mocker.patch(
+        _ = mocker.patch(
             "modelscope_mcp_server.client.ModelScopeClient.get",
             new_callable=mocker.AsyncMock,
             return_value={
@@ -244,7 +244,7 @@ async def test_missing_output_images_in_success(self, mcp_server, mocker):
             return_value={"task_id": "no-images-task"},
         )
 
-        mock_get = mocker.patch(
+        _ = mocker.patch(
             "modelscope_mcp_server.client.ModelScopeClient.get",
             new_callable=mocker.AsyncMock,
             return_value={
@@ -273,7 +273,7 @@ async def test_empty_output_images_array(self, mcp_server, mocker):
             return_value={"task_id": "empty-images-task"},
         )
 
-        mock_get = mocker.patch(
+        _ = mocker.patch(
             "modelscope_mcp_server.client.ModelScopeClient.get",
             new_callable=mocker.AsyncMock,
             return_value={
@@ -341,7 +341,7 @@ async def mock_get_side_effect(url, *args, **kwargs):
                 return responses.pop(0)
             return {"task_status": "PENDING"}
 
-        mock_get = mocker.patch(
+        _ = mocker.patch(
             "modelscope_mcp_server.client.ModelScopeClient.get",
             new_callable=mocker.AsyncMock,
             side_effect=mock_get_side_effect,
@@ -363,11 +363,15 @@ async def mock_get_side_effect(url, *args, **kwargs):
             results = await asyncio.gather(*tasks, return_exceptions=True)
 
         # Verify outcomes
+        from typing import Any, cast
+
         assert not isinstance(results[0], Exception)  # First succeeded
-        assert results[0].data.image_url == "https://example.com/image1.jpg"
+        result0 = cast(Any, results[0])
+        assert result0.data.image_url == "https://example.com/image1.jpg"
 
         assert isinstance(results[1], Exception)  # Second failed
         assert "Task 2 failed" in str(results[1])
 
         assert not isinstance(results[2], Exception)  # Third succeeded
-        assert results[2].data.image_url == "https://example.com/image3.jpg"
+        result2 = cast(Any, results[2])
+        assert result2.data.image_url == "https://example.com/image3.jpg"

tests/tools/test_generate_image.py

@@ -141,7 +141,11 @@ async def test_generate_image_api_error_response(mcp_server, mocker):
     mocker.patch(
         "modelscope_mcp_server.client.ModelScopeClient.post",
         new_callable=mocker.AsyncMock,
-        side_effect=httpx.HTTPStatusError("404 Client Error: Not Found", request=None, response=None),
+        side_effect=httpx.HTTPStatusError(
+            "404 Client Error: Not Found",
+            request=httpx.Request("POST", "https://example.com"),
+            response=httpx.Response(404, request=httpx.Request("POST", "https://example.com")),
+        ),
     )
 
     async with Client(mcp_server) as client: