From ea2e330e43c182dc16b0111ebc69ee5a71ee4ce1 Mon Sep 17 00:00:00 2001 From: Gavin Mak Date: Sat, 11 Mar 2023 06:46:20 +0000 Subject: Format codebase with black and check formatting in CQ Apply rules set by https://gerrit-review.googlesource.com/c/git-repo/+/362954/ across the codebase and fix any lingering errors caught by flake8. Also check black formatting in run_tests (and CQ). Bug: b/267675342 Change-Id: I972d77649dac351150dcfeb1cd1ad0ea2efc1956 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/363474 Reviewed-by: Mike Frysinger Tested-by: Gavin Mak Commit-Queue: Gavin Mak --- tests/test_error.py | 60 +++++++++++++++++++++++++++-------------------------- 1 file changed, 31 insertions(+), 29 deletions(-) (limited to 'tests/test_error.py') diff --git a/tests/test_error.py b/tests/test_error.py index 82b00c24..784e2d57 100644 --- a/tests/test_error.py +++ b/tests/test_error.py @@ -22,32 +22,34 @@ import error class PickleTests(unittest.TestCase): - """Make sure all our custom exceptions can be pickled.""" - - def getExceptions(self): - """Return all our custom exceptions.""" - for name in dir(error): - cls = getattr(error, name) - if isinstance(cls, type) and issubclass(cls, Exception): - yield cls - - def testExceptionLookup(self): - """Make sure our introspection logic works.""" - classes = list(self.getExceptions()) - self.assertIn(error.HookError, classes) - # Don't assert the exact number to avoid being a change-detector test. - self.assertGreater(len(classes), 10) - - def testPickle(self): - """Try to pickle all the exceptions.""" - for cls in self.getExceptions(): - args = inspect.getfullargspec(cls.__init__).args[1:] - obj = cls(*args) - p = pickle.dumps(obj) - try: - newobj = pickle.loads(p) - except Exception as e: # pylint: disable=broad-except - self.fail('Class %s is unable to be pickled: %s\n' - 'Incomplete super().__init__(...) call?' % (cls, e)) - self.assertIsInstance(newobj, cls) - self.assertEqual(str(obj), str(newobj)) + """Make sure all our custom exceptions can be pickled.""" + + def getExceptions(self): + """Return all our custom exceptions.""" + for name in dir(error): + cls = getattr(error, name) + if isinstance(cls, type) and issubclass(cls, Exception): + yield cls + + def testExceptionLookup(self): + """Make sure our introspection logic works.""" + classes = list(self.getExceptions()) + self.assertIn(error.HookError, classes) + # Don't assert the exact number to avoid being a change-detector test. + self.assertGreater(len(classes), 10) + + def testPickle(self): + """Try to pickle all the exceptions.""" + for cls in self.getExceptions(): + args = inspect.getfullargspec(cls.__init__).args[1:] + obj = cls(*args) + p = pickle.dumps(obj) + try: + newobj = pickle.loads(p) + except Exception as e: # pylint: disable=broad-except + self.fail( + "Class %s is unable to be pickled: %s\n" + "Incomplete super().__init__(...) call?" % (cls, e) + ) + self.assertIsInstance(newobj, cls) + self.assertEqual(str(obj), str(newobj)) -- cgit v1.2.3-54-g00ecf