Test HttpResource.read() instead of excluding it from coverage

The method body had been marked `# pragma: no cover` (widened further once
httpx2 moved into the method). Exercise it against an httpx2 MockTransport
instead, covering both the response body and the raise_for_status path, and
drop the pragma.

Author: maxisbey

src/mcp/server/mcpserver/resources/types.py

@@ -196,7 +196,7 @@ class HttpResource(Resource):
     url: str = Field(description="URL to fetch content from")
     mime_type: str = Field(default="application/json", description="MIME type of the resource content")
 
-    async def read(self) -> str | bytes:  # pragma: no cover
+    async def read(self) -> str | bytes:
         """Read the HTTP content."""
         # httpx2 is imported here rather than at module top: this is the only
         # resource type that needs the HTTP client stack, and a server that

tests/server/mcpserver/resources/test_http_resource.py

@@ -0,0 +1,37 @@
+from collections.abc import Callable
+
+import httpx2
+import pytest
+
+from mcp.server.mcpserver.resources import HttpResource
+
+pytestmark = pytest.mark.anyio
+
+
+def mock_client_factory(handler: Callable[[httpx2.Request], httpx2.Response]) -> Callable[[], httpx2.AsyncClient]:
+    """An `httpx2.AsyncClient` stand-in whose requests are answered by `handler`."""
+    real_client = httpx2.AsyncClient
+
+    def factory() -> httpx2.AsyncClient:
+        return real_client(transport=httpx2.MockTransport(handler))
+
+    return factory
+
+
+async def test_http_resource_reads_the_response_body_as_text(monkeypatch: pytest.MonkeyPatch):
+    """`HttpResource.read()` fetches its URL and returns the response text."""
+
+    def handler(request: httpx2.Request) -> httpx2.Response:
+        return httpx2.Response(200, text=f"body of {request.url}")
+
+    monkeypatch.setattr(httpx2, "AsyncClient", mock_client_factory(handler))
+    resource = HttpResource(uri="resource://remote", url="http://example.test/data")
+    assert await resource.read() == "body of http://example.test/data"
+
+
+async def test_http_resource_read_raises_for_error_status(monkeypatch: pytest.MonkeyPatch):
+    """A non-2xx response surfaces as `httpx2.HTTPStatusError` rather than being read as content."""
+    monkeypatch.setattr(httpx2, "AsyncClient", mock_client_factory(lambda request: httpx2.Response(404)))
+    resource = HttpResource(uri="resource://remote", url="http://example.test/missing")
+    with pytest.raises(httpx2.HTTPStatusError):
+        await resource.read()