Implement aiohttp.ClientResponse.ok property

CHANGES/4711.feature

@@ -0,0 +1 @@
+Add ClientResponse.ok property for checking status code under 400.

CONTRIBUTORS.txt

@@ -100,6 +100,7 @@ Eugene Ershov
 Eugene Naydenov
 Eugene Nikolaiev
 Eugene Tolmachev
+Evan Kepner
 Evert Lammerts
 Felix Yan
 Fernanda Guimarães

aiohttp/client_reqrep.py

@@ -876,6 +876,19 @@ def release(self) -> Any:
         self._cleanup_writer()
         return noop()
 
+    @property
+    def ok(self) -> bool:
+        """Returns ``True`` if ``status`` is less than ``400``, ``False`` if not.
+
+        This is **not** a check for ``200 OK`` but a check that the response
+        status is under 400.
+        """
+        try:
+            self.raise_for_status()
+        except ClientResponseError:
+            return False
+        return True
+
     def raise_for_status(self) -> None:
         if 400 <= self.status:
             # reason should always be not None for a started response

docs/client_reference.rst

@@ -1091,6 +1091,11 @@ Response object
 
       HTTP status reason of response (:class:`str`), e.g. ``"OK"``.
 
+   .. attribute:: ok
+
+      Boolean representation of HTTP status code (:class:`bool`).
+      ``True`` if ``status`` is less than ``400``; otherwise, ``False``.
+
    .. attribute:: method
 
       Request's method (:class:`str`).

tests/test_client_functional.py

@@ -2202,6 +2202,30 @@ async def handler_redirect(request):
     assert data == body
 
 
+@pytest.mark.parametrize(
+    ("status", "expected_ok"),
+    (
+        (200, True),
+        (201, True),
+        (301, True),
+        (400, False),
+        (403, False),
+        (500, False),
+    ),
+    ids=lambda c, o: f"HTTP status code {c} is{' ' if o else ' not'} ok",
+)
+async def test_ok_from_status(aiohttp_client, status, expected_ok) -> None:
+
+    async def handler(request):
+        return web.Response(status=status, body=b'')
+
+    app = web.Application()
+    app.router.add_route('GET', '/endpoint', handler)
+    client = await aiohttp_client(app, raise_for_status=False)
+    resp = await client.get('/endpoint')
+
+    assert resp.ok is expected_ok
+
 async def test_raise_for_status(aiohttp_client) -> None:
 
     async def handler(request):