Merge pull request #1472 from xxNB/dev

Remove unwanted None check for __repr__ in `Request` class
sanic-org · Jan 20, 2019 · 2758a3a · 2758a3a
2 parents ef3c9ea + af7ad0a
commit 2758a3a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/sanic/request.py b/sanic/request.py
@@ -115,8 +115,6 @@ def __init__(self, url_bytes, headers, version, method, transport):
         self.endpoint = None
 
     def __repr__(self):
-        if self.method is None or not self.path:
-            return "<{0}>".format(self.__class__.__name__)
         return "<{0}: {1} {2}>".format(
             self.__class__.__name__, self.method, self.path
         )

diff --git a/tests/test_requests.py b/tests/test_requests.py
@@ -566,7 +566,7 @@ def handler(request):
     assert repr(request) == "<Request: GET />"
 
     request.method = None
-    assert repr(request) == "<Request>"
+    assert repr(request) == "<Request: None />"
 
 
 def test_request_bool(app):