[PR #10172/e45c3b8e backport][3.12] Ensure Response is True even when map is empty (#10177)

**This is a backport of PR #10172 as merged into master
(e45c3b8).**

Fixes #10119

Technically a breaking change, but I can't imagine anyone depending on
this. 
Co-authored-by: Robin <[email protected]>

Author: patchback[bot]

CHANGES/10119.bugfix.rst

@@ -0,0 +1 @@
+Response is now always True, instead of using MutableMapping behaviour (False when map is empty)

aiohttp/web_response.py

@@ -609,6 +609,9 @@ def __hash__(self) -> int:
     def __eq__(self, other: object) -> bool:
         return self is other
 
+    def __bool__(self) -> bool:
+        return True
+
 
 class Response(StreamResponse):
 

tests/test_web_request.py

@@ -311,6 +311,7 @@ def test_match_info() -> None:
 def test_request_is_mutable_mapping() -> None:
     req = make_mocked_request("GET", "/")
     assert isinstance(req, MutableMapping)
+    assert req  # even when the MutableMapping is empty, request should always be True
     req["key"] = "value"
     assert "value" == req["key"]
 

tests/test_web_response.py

@@ -106,6 +106,7 @@ def test_stream_response_eq() -> None:
 def test_stream_response_is_mutable_mapping() -> None:
     resp = StreamResponse()
     assert isinstance(resp, collections.abc.MutableMapping)
+    assert resp  # even when the MutableMapping is empty, response should always be True
     resp["key"] = "value"
     assert "value" == resp["key"]