🧪 Add test verifying check_error reads response body

google-labs-jules[bot] · rnovatorov · google-labs-jules[bot] · commit 4d59e5e095da · 2026-04-01T12:40:24.000Z
Addresses PR feedback by adding `test_check_error_reads_body` which verifies that `response.aread()` is called when `response.is_success` is False, ensuring the body is completely read for streamed responses. Also fixed PEP8 import order.

Co-authored-by: rnovatorov &lt;20299819+rnovatorov@users.noreply.github.com&gt;
diff --git a/tests/unit/test_http/test_api/test_errors.py b/tests/unit/test_http/test_api/test_errors.py
@@ -1,3 +1,5 @@
+from unittest.mock import AsyncMock
+
 import httpx
 import pytest
 
@@ -80,3 +82,18 @@ def test_multi_error_from_dto_empty_list():
     """Test MultiError.from_dto raises ValueError on empty error list."""
     with pytest.raises(ValueError, match="empty error list"):
         MultiError.from_dto({"errors": []})
+
+
+@pytest.mark.asyncio
+async def test_check_error_reads_body():
+    """Test that check_error reads the response body."""
+    response = httpx.Response(
+        status_code=500,
+        content=b"Internal Server Error",
+        request=httpx.Request("GET", "https://example.com"),
+    )
+    response.aread = AsyncMock()
+    with pytest.raises(httpx.HTTPStatusError):
+        await check_error(response)
+
+    response.aread.assert_awaited_once()
