Do not wrap resource __repr__ output in <> to render tracebacks properly in browser.

Author: lenadax

CHANGES.rst

@@ -4,7 +4,9 @@ Changelog
 1.3 (unreleased)
 ----------------
 
-- Nothing changed yet.
+- Do not wrap resource ``__repr__`` output in ``<>`` to render tracebacks
+  properly in browser.
+  [lenadax]
 
 
 1.2 (2022-12-21)

webresource/_api.py

@@ -237,7 +237,7 @@ def _render_tag(self, tag, closing_tag, **attrs):
 
     def __repr__(self):
         return (
-            '<{} name="{}", depends="{}">'
+            '{} name="{}", depends="{}"'
         ).format(
             self.__class__.__name__,
             self.name,
@@ -585,7 +585,7 @@ def _filtered_resources(self, type_, members=None):
         return resources
 
     def __repr__(self):
-        return '<{} name="{}">'.format(
+        return '{} name="{}"'.format(
             self.__class__.__name__,
             self.name
         )

webresource/tests.py

@@ -111,7 +111,7 @@ def test_Resource(self, tempdir):
         self.assertEqual(resource.type_, None)
         self.assertEqual(
             repr(resource),
-            '<Resource name="res", depends="None">'
+            'Resource name="res", depends="None"'
         )
 
         resource = Resource(name='res', resource='res.ext')
@@ -239,7 +239,7 @@ def test_ScriptResource(self, tempdir):
         self.assertEqual(script.nomodule, None)
         self.assertEqual(
             repr(script),
-            '<ScriptResource name="js_res", depends="None">'
+            'ScriptResource name="js_res", depends="None"'
         )
         self.assertEqual(
             script.render('https://tld.org'),
@@ -301,7 +301,7 @@ def test_LinkMixin(self):
         self.assertEqual(link.title, None)
         self.assertEqual(
             repr(link),
-            '<LinkMixin name="link_res", depends="None">'
+            'LinkMixin name="link_res", depends="None"'
         )
         link.hreflang = 'en'
         link.media = 'screen'
@@ -327,7 +327,7 @@ def test_LinkResource(self):
         self.assertIsInstance(link, LinkMixin)
         self.assertEqual(
             repr(link),
-            '<LinkResource name="icon_res", depends="None">'
+            'LinkResource name="icon_res", depends="None"'
         )
         link.rel = 'icon'
         link.type_ = 'image/png'
@@ -355,7 +355,7 @@ def test_StyleResource(self):
         self.assertEqual(style.rel, 'stylesheet')
         self.assertEqual(
             repr(style),
-            '<StyleResource name="css_res", depends="None">'
+            'StyleResource name="css_res", depends="None"'
         )
         self.assertEqual(style.render('https://tld.org'), (
             '<link href="https://tld.org/res.css" media="all" '
@@ -377,7 +377,7 @@ def test_ResourceGroup(self):
         self.assertIsInstance(group, ResourceMixin)
         self.assertEqual(group.name, 'groupname')
         self.assertEqual(group.members, [])
-        self.assertEqual(repr(group), '<ResourceGroup name="groupname">')
+        self.assertEqual(repr(group), 'ResourceGroup name="groupname"')
 
         res = wr.ScriptResource(name='name', resource='name.js')
         group.add(res)
@@ -452,15 +452,15 @@ def test_ResourceCircularDependencyError(self):
         err = wr.ResourceCircularDependencyError([resource])
         self.assertEqual(str(err), (
             'Resources define circular dependencies: '
-            '[<Resource name="res1", depends="[\'res2\']">]'
+            '[Resource name="res1", depends="[\'res2\']"]'
         ))
 
     def test_ResourceMissingDependencyError(self):
         resource = Resource(name='res', resource='res.ext', depends='missing')
         err = wr.ResourceMissingDependencyError(resource)
         self.assertEqual(str(err), (
             'Resource defines missing dependency: '
-            '<Resource name="res", depends="[\'missing\']">'
+            'Resource name="res", depends="[\'missing\']"'
         ))
 
     def test_ResourceResolver__flat_resources(self):