Codecov

Author: Anusha Shekhar

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:
@@ -451,14 +453,55 @@ def conflict(self) -> None:
     )
 
 
-def test_hookcaller_repr_with_saferepr_failure(
-    hc: HookCaller, addmeth: AddMeth
-) -> None:
-    @addmeth()
-    def he_method2() -> None:
-        # 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
-    expected_repr = f"<HookCaller {saferepr(hc.name)}>"
-    assert repr(hc) == expected_repr
+def test_hook_impl_initialization() -> None:
+    # Mock data
+    plugin = "example_plugin"
+    plugin_name = "ExamplePlugin"
+    function = lambda x: x
+    hook_impl_opts = {
+        "wrapper": False,
+        "hookwrapper": False,
+        "optionalhook": False,
+        "tryfirst": False,
+        "trylast": False,
+    }
+
+    # Initialize HookImpl
+    hook_impl = HookImpl(plugin, plugin_name, function, hook_impl_opts)
+
+    # Verify attributes are set correctly
+    assert hook_impl.function == function
+    assert hook_impl.argnames == ("x",) 
+    assert hook_impl.kwargnames == ()
+    assert hook_impl.plugin == plugin
+    assert hook_impl.opts == hook_impl_opts
+    assert hook_impl.plugin_name == plugin_name
+    assert not hook_impl.wrapper
+    assert not hook_impl.hookwrapper
+    assert not hook_impl.optionalhook
+    assert not hook_impl.tryfirst
+    assert not hook_impl.trylast
+
+
+def test_hook_impl_representation() -> None:
+    # Mock data
+    plugin = "example_plugin"
+    plugin_name = "ExamplePlugin"
+    function = lambda x: x
+    hook_impl_opts = {
+        "wrapper": False,
+        "hookwrapper": False,
+        "optionalhook": False,
+        "tryfirst": False,
+        "trylast": False,
+    }
+
+    # Initialize HookImpl
+    hook_impl = HookImpl(plugin, plugin_name, function, hook_impl_opts)
+
+    # Verify __repr__ method
+    expected_repr = (
+        f"<HookImpl plugin_name={saferepr(plugin_name)}, "
+        f"plugin={saferepr(plugin)}>"
+    )
+    assert repr(hook_impl) == 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