Fix test_utils.make_mocked_request behaviour for empty payload (#7168)

CHANGES/7167.bugfix.rst

@@ -0,0 +1 @@
+Changed ``make_mocked_request()`` to use empty payload by default -- by :user:`rahulnht`.

CONTRIBUTORS.txt

@@ -282,6 +282,7 @@ Philipp A.
 Pieter van Beek
 Qiao Han
 Rafael Viotti
+Rahul Nahata
 Raphael Bialon
 Raúl Cumplido
 Required Field

aiohttp/test_utils.py

@@ -42,6 +42,7 @@
 from .client_ws import ClientWebSocketResponse
 from .helpers import sentinel
 from .http import HttpVersion, RawRequestMessage
+from .streams import EMPTY_PAYLOAD, StreamReader
 from .typedefs import StrOrURL
 from .web import (
  Application,
@@ -594,7 +595,7 @@ def make_mocked_request(
  writer: Any = sentinel,
  protocol: Any = sentinel,
  transport: Any = sentinel,
- payload: Any = sentinel,
+ payload: StreamReader = EMPTY_PAYLOAD,
  sslcontext: Optional[SSLContext] = None,
  client_max_size: int = 1024**2,
  loop: Any = ...,
@@ -663,9 +664,6 @@ def make_mocked_request(
  protocol.transport = transport
  protocol.writer = writer
 
- if payload is sentinel:
- payload = mock.Mock()
-
  req = Request(
  message, payload, protocol, writer, task, loop, client_max_size=client_max_size
  )

tests/test_test_utils.py

@@ -218,6 +218,11 @@ def test_make_mocked_request_content() -> None:
  assert req.content is payload
 
 
+async def test_make_mocked_request_empty_payload() -> None:
+ req = make_mocked_request("GET", "/")
+ assert await req.read() == b""
+
+
 def test_make_mocked_request_transport() -> None:
  transport = mock.Mock()
  req = make_mocked_request("GET", "/", transport=transport)