Fix resources __repr__ formatting (#2935)

aio-libs · Apr 13, 2018 · 55a6782 · 55a6782
1 parent 1dd1185
commit 55a6782
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/CHANGES/2935.bugfix b/CHANGES/2935.bugfix
@@ -0,0 +1 @@
+A closing bracket was added to __repr__ of resources
diff --git a/CONTRIBUTORS.txt b/CONTRIBUTORS.txt
@@ -185,6 +185,7 @@ Thanos Lefteris
 Thijs Vermeir
 Thomas Grainger
 Tolga Tezel
+Vadim Suharnikov
 Vaibhav Sagar
 Vamsi Krishna Avula
 Vasiliy Faronov

diff --git a/aiohttp/web_urldispatcher.py b/aiohttp/web_urldispatcher.py
@@ -325,8 +325,8 @@ def url_for(self):
 
     def __repr__(self):
         name = "'" + self.name + "' " if self.name is not None else ""
-        return "<PlainResource {name} {path}".format(name=name,
-                                                     path=self._path)
+        return "<PlainResource {name} {path}>".format(name=name,
+                                                      path=self._path)
 
 
 class DynamicResource(Resource):
@@ -399,7 +399,7 @@ def url_for(self, **parts):
 
     def __repr__(self):
         name = "'" + self.name + "' " if self.name is not None else ""
-        return ("<DynamicResource {name} {formatter}"
+        return ("<DynamicResource {name} {formatter}>"
                 .format(name=name, formatter=self._formatter))
 
 
@@ -602,7 +602,7 @@ def _directory_as_html(self, filepath):
 
     def __repr__(self):
         name = "'" + self.name + "'" if self.name is not None else ""
-        return "<StaticResource {name} {path} -> {directory!r}".format(
+        return "<StaticResource {name} {path} -> {directory!r}>".format(
             name=name, path=self._prefix, directory=self._directory)