Make testtools ruff-clean

Author: jelmer

testtools/__init__.py

@@ -47,9 +47,7 @@
     ]
 
 from testtools.helpers import try_import
-from testtools.matchers._impl import Matcher
-# Shut up, pyflakes. We are importing for documentation, not for namespacing.
-Matcher
+from testtools.matchers._impl import Matcher  # noqa: F401
 
 from testtools.runtest import (
     MultipleExceptions,

testtools/compat.py

@@ -83,8 +83,8 @@ def text_repr(text, multiline=None):
         # making sure that quotes are not escaped.
         offset = len(prefix) + 1
         lines = []
-        for l in text.split(nl):
-            r = repr(l)
+        for line in text.split(nl):
+            r = repr(line)
             q = r[-1]
             lines.append(r[offset:-1].replace("\\" + q, q))
         # Combine the escaped lines and append two of the closing quotes,

testtools/matchers/_datastructures.py

@@ -1,12 +1,5 @@
 # Copyright (c) 2009-2015 testtools developers. See LICENSE for details.
 
-__all__ = [
-    'ContainsAll',
-    'MatchesListwise',
-    'MatchesSetwise',
-    'MatchesStructure',
-    ]
-
 """Matchers that operate with knowledge of Python data structures."""
 
 from ..helpers import map_values
@@ -17,6 +10,15 @@
     )
 from ._impl import Mismatch
 
+__all__ = [
+    'ContainsAll',
+    'MatchesListwise',
+    'MatchesSetwise',
+    'MatchesStructure',
+    ]
+
+
+
 
 def ContainsAll(items):
     """Make a matcher that checks whether a list of things is contained

testtools/matchers/_dict.py

@@ -180,7 +180,8 @@ class MatchesDict(_CombinedMatcher):
         'Differences': _MatchCommonKeys,
         }
 
-    format_expected = lambda self, expected: _format_matcher_dict(expected)
+    def format_expected(self, expected) -> str:
+        return _format_matcher_dict(expected)
 
 
 class ContainsDict(_CombinedMatcher):
@@ -203,7 +204,8 @@ class ContainsDict(_CombinedMatcher):
         'Differences': _MatchCommonKeys,
         }
 
-    format_expected = lambda self, expected: _format_matcher_dict(expected)
+    def format_expected(self, expected):
+        return _format_matcher_dict(expected)
 
 
 class ContainedByDict(_CombinedMatcher):
@@ -226,7 +228,8 @@ class ContainedByDict(_CombinedMatcher):
         'Differences': _MatchCommonKeys,
         }
 
-    format_expected = lambda self, expected: _format_matcher_dict(expected)
+    def format_expected(self, expected):
+        return _format_matcher_dict(expected)
 
 
 class KeysEqual(Matcher):

testtools/matchers/_exception.py

@@ -98,7 +98,7 @@ def match(self, matchee):
             return Mismatch(f'{matchee!r} returned {result!r}')
         # Catch all exceptions: Raises() should be able to match a
         # KeyboardInterrupt or SystemExit.
-        except:
+        except BaseException:
             exc_info = sys.exc_info()
             if self.exception_matcher:
                 mismatch = self.exception_matcher.match(exc_info)

testtools/runtest.py

@@ -191,7 +191,7 @@ def _run_user(self, fn, *args, **kwargs):
         """
         try:
             return fn(*args, **kwargs)
-        except:
+        except BaseException:
             return self._got_user_exception(sys.exc_info())
 
     def _got_user_exception(self, exc_info, tb_label='traceback'):

testtools/testcase.py

@@ -737,7 +737,7 @@ def useFixture(self, fixture):
                     e.args[-1][0] is fixtures.fixture.SetupError):
                 gather_details(e.args[-1][1].args[0], self.getDetails())
             raise
-        except:
+        except BaseException:
             exc_info = sys.exc_info()
             try:
                 # fixture._details is not available if using the newer
@@ -750,7 +750,7 @@ def useFixture(self, fixture):
                     fixture._details is not None
                 ):
                     gather_details(fixture.getDetails(), self.getDetails())
-            except:
+            except BaseException:
                 # Report the setUp exception, then raise the error during
                 # gather_details.
                 self._report_traceback(exc_info)

testtools/tests/matchers/test_basic.py

@@ -157,7 +157,8 @@ class TestIsInterface(TestCase, TestMatchersInterface):
 
 class TestIsInstanceInterface(TestCase, TestMatchersInterface):
 
-    class Foo:pass
+    class Foo:
+        pass
 
     matches_matcher = IsInstance(Foo)
     matches_matches = [Foo()]

testtools/tests/test_run.py

@@ -146,7 +146,7 @@ def list(self, test):
                     in testtools.testsuite.iterate_tests(test)})
         out = io.StringIO()
         try:
-            program = run.TestProgram(
+            run.TestProgram(
                 argv=['prog', '-l', 'testtools.runexample.test_suite'],
                 stdout=out, testRunner=CaptureList)
         except SystemExit:
@@ -310,7 +310,7 @@ def test_run_locals(self):
 
         class Failing(TestCase):
             def test_a(self):
-                a = 1
+                a = 1  # noqa: F841
                 self.fail('a')
         runner = run.TestToolsTestRunner(tb_locals=True, stdout=stdout.stream)
         runner.run(Failing('test_a'))
@@ -319,7 +319,6 @@ def test_a(self):
 
     def test_stdout_honoured(self):
         self.useFixture(SampleTestFixture())
-        tests = []
         out = io.StringIO()
         exc = self.assertRaises(SystemExit, run.main,
             argv=['prog', 'testtools.runexample.test_suite'],

testtools/tests/test_testcase.py

@@ -205,7 +205,7 @@ class TestErrorHolder(TestCase):
     def makeException(self):
         try:
             raise RuntimeError("danger danger")
-        except:
+        except BaseException:
             return sys.exc_info()
 
     def makePlaceHolder(self, test_id="foo", error=None,
@@ -1909,12 +1909,12 @@ def foo():
     def test_called_with_arguments(self):
         # The function is called with the arguments given to Nullary's
         # constructor.
-        l = []
+        line = []
         def foo(*args, **kwargs):
-            l.append((args, kwargs))
+            line.append((args, kwargs))
         wrapped = Nullary(foo, 1, 2, a="b")
         wrapped()
-        self.assertEqual(l, [((1, 2), {'a': 'b'})])
+        self.assertEqual(line, [((1, 2), {'a': 'b'})])
 
     def test_returns_wrapped(self):
         # Calling Nullary returns whatever the function returns.