A test was missing that body_init/body_push/body_finish are never called. Rewritten using receive_body and case switching to make it fail if bypassed.

tests/test_custom_request.py

@@ -6,18 +6,13 @@ from sanic.response import json_dumps, text
 
 
 class CustomRequest(Request):
-    __slots__ = ("body_buffer",)
+    """Alternative implementation for loading body (non-streaming handlers)"""
-
+    async def receive_body(self):
-    def body_init(self):
+        buffer = BytesIO()
-        self.body_buffer = BytesIO()
+        async for data in self.stream:
-
+            buffer.write(data)
-    def body_push(self, data):
+        self.body = buffer.getvalue().upper()
-        self.body_buffer.write(data)
+        buffer.close()
-
-    def body_finish(self):
-        self.body = self.body_buffer.getvalue()
-        self.body_buffer.close()
-
 
 def test_custom_request():
     app = Sanic(request_class=CustomRequest)
@@ -37,8 +32,8 @@ def test_custom_request():
         "/post", data=json_dumps(payload), headers=headers
     )
 
-    assert request.body == b'{"test":"OK"}'
+    assert request.body == b'{"TEST":"OK"}'
-    assert request.json.get("test") == "OK"
+    assert request.json.get("TEST") == "OK"
     assert response.text == "OK"
     assert response.status == 200