Add test for usage of saferepr

testing/test_hookcaller.py

@@ -11,6 +11,7 @@
 from pluggy import PluginValidationError
 from pluggy._hooks import HookCaller
 from pluggy._hooks import HookImpl
+from pluggy._tracing import saferepr
 
 hookspec = HookspecMarker("example")
 hookimpl = HookimplMarker("example")
@@ -448,3 +449,21 @@ def conflict(self) -> None:
         "Hook 'conflict' is already registered within namespace "
         "<class 'test_hookcaller.test_hook_conflict.<locals>.Api1'>"
     )
+
+def test_hookcaller_repr_with_saferepr_failure(hc: HookCaller, addmeth: AddMeth) -> None:
+    @addmeth()
+    def he_method1() -> None:
+        pass
+
+    @addmeth()
+    def he_method2() -> None:
+        # Intentional error to make the repr fail
+        raise ValueError("Intentional error in he_method2")
+
+    @addmeth()
+    def he_method3() -> None:
+        pass
+
+    # Verify that HookCaller.repr with saferepr still works despite the error
+    expected_repr = f"<HookCaller {saferepr(hc.name)}>"
+    assert repr(hc) == expected_repr