test: close logging coverage gaps

Author: Genmin

tests/cli/test_utils.py

@@ -8,6 +8,7 @@
 from mcp.cli.cli import (  # type: ignore[reportPrivateUsage]
     _build_uv_command,
     _get_npx_command,
+    _get_server_log_level,
     _parse_file_path,
     run,
 )
@@ -78,6 +79,16 @@ def fake_configure_logging(level: str) -> None:
     ]
 
 
+def test_get_server_log_level_defaults_invalid_setting():
+    class Settings:
+        log_level = "VERBOSE"
+
+    class Server:
+        settings = Settings()
+
+    assert _get_server_log_level(Server()) == "INFO"
+
+
 def test_build_uv_command_minimal():
     """Should emit core command when no extras specified."""
     cmd = _build_uv_command("foo.py")

tests/server/mcpserver/test_server.py

@@ -48,16 +48,12 @@
 
 class TestServer:
     def test_create_server_does_not_configure_application_logging(self, monkeypatch: pytest.MonkeyPatch):
-        calls: list[tuple[tuple[Any, ...], dict[str, Any]]] = []
-
-        def fake_basic_config(*args: Any, **kwargs: Any) -> None:
-            calls.append((args, kwargs))
-
-        monkeypatch.setattr(logging, "basicConfig", fake_basic_config)
+        basic_config = MagicMock()
+        monkeypatch.setattr(logging, "basicConfig", basic_config)
 
         MCPServer("test")
 
-        assert calls == []
+        basic_config.assert_not_called()
 
     async def test_create_server(self):
         mcp = MCPServer(