Changes from comments on PR #3492:

- Also inherit from ``Web3ValueError`` for ``ReadBufferLimitReached``.

- Properly assert the response for configured read limit test.

Author: fselmo

tests/core/providers/test_async_ipc_provider.py

@@ -337,4 +337,8 @@ async def test_async_ipc_read_buffer_limit_is_configurable(
             pathlib.Path(jsonrpc_ipc_pipe_path), read_buffer_limit=TWENTY_MB + 1024
         )
     ) as w3:
-        await w3.provider.make_request("method", [])
+        response = await w3.provider.make_request("method", [])
+        assert (
+            len(response["result"])
+            == TWENTY_MB - len(SIZED_MSG_START) - len(SIZED_MSG_END) + 1024
+        )

web3/exceptions.py

@@ -340,7 +340,7 @@ class PersistentConnectionError(Web3Exception):
     """
 
 
-class ReadBufferLimitReached(PersistentConnectionError):
+class ReadBufferLimitReached(PersistentConnectionError, Web3ValueError):
     """
     Raised when the read buffer limit is reached while reading data from a persistent
     connection.