Codecov

testing/test_hookcaller.py

@@ -410,7 +410,8 @@ def hello(self, arg: object) -> None:
 
     pm.add_hookspecs(Api)
 
-    # make sure a bad signature still raises an error when using specname
+    """make sure a bad signature still raises an error when using specname"""
+
     class Plugin:
         @hookimpl(specname="hello")
         def foo(self, arg: int, too, many, args) -> int:
@@ -419,8 +420,9 @@ def foo(self, arg: int, too, many, args) -> int:
     with pytest.raises(PluginValidationError):
         pm.register(Plugin())
 
-    # make sure check_pending still fails if specname doesn't have a
-    # corresponding spec.  EVEN if the function name matches one.
+    """make sure check_pending still fails if specname doesn't have a
+    corresponding spec.  EVEN if the function name matches one."""
+
     class Plugin2:
         @hookimpl(specname="bar")
         def hello(self, arg: int) -> int:
@@ -456,9 +458,9 @@ def test_hookcaller_repr_with_saferepr_failure(
 ) -> None:
     @addmeth()
     def he_method2() -> None:
-        # Intentional error to make the repr fail
+        """Intentional error to make the repr fail"""
         raise ValueError("Intentional error in he_method2")
 
-    # Verify that HookCaller.repr with saferepr still works despite the error
+    """Verify that HookCaller.repr with saferepr still works despite the error"""
     expected_repr = f"<HookCaller {saferepr(hc.name)}>"
     assert repr(hc) == expected_repr

testing/test_tracer.py

@@ -243,9 +243,6 @@ def test_saferepr_broken_getattribute():
     """
 
     class SomeClass:
-        def __getattribute__(self, attr):
-            raise RuntimeError
-
         def __repr__(self):
             raise RuntimeError