Fix and test UrlPattern/configutils.Values stringification

qutebrowser/utils/urlmatch.py

@@ -107,6 +107,12 @@ class UrlPattern:
         # pylint: disable=protected-access
         return self._to_tuple() == other._to_tuple()
 
+    def __repr__(self):
+        return utils.get_repr(self, pattern=self._pattern, constructor=True)
+
+    def __str__(self):
+        return self._pattern
+
     def _fixup_pattern(self, pattern):
         """Make sure the given pattern is parseable by urllib.parse."""
         if pattern.startswith('*:'):  # Any scheme, but *:// is unparseable
@@ -188,9 +194,6 @@ class UrlPattern:
             raise ParseError("Ports are unsupported with {} scheme".format(
                 self._scheme))
 
-    def __repr__(self):
-        return utils.get_repr(self, pattern=self._pattern, constructor=True)
-
     def _matches_scheme(self, scheme):
         return self._scheme is None or self._scheme == scheme
 

tests/unit/config/test_configutils.py

@@ -55,6 +55,14 @@ def test_repr(opt, values):
     assert repr(values) == expected
 
 
+def test_str(values):
+    expected = [
+        'example.option = global value',
+        '*://www.example.com/: example.option = example value',
+    ]
+    assert str(values) == '\n'.join(expected)
+
+
 def test_str_empty(opt):
     values = configutils.Values(opt)
     assert str(values) == 'example.option: <unchanged>'

tests/unit/utils/test_urlmatch.py

@@ -517,3 +517,16 @@ def test_equal(text1, text2, equal):
 
     assert (pat1 == pat2) == equal
     assert (hash(pat1) == hash(pat2)) == equal
+
+
+def test_repr():
+    pat = urlmatch.UrlPattern('https://www.example.com/')
+    expected = ("qutebrowser.utils.urlmatch.UrlPattern("
+                "pattern='https://www.example.com/')")
+    assert repr(pat) == expected
+
+
+def test_str():
+    text = 'https://www.example.com/'
+    pat = urlmatch.UrlPattern(text)
+    assert str(pat) == text