Directly access class methods in introspection (fixes #966)

sebastinas · sebastinas · commit 1bc255b87ddd · 2022-08-29T18:24:02.000+02:00
inspect.signature is unable to process classmethod instances. So we
directly access the member via its __get__.
diff --git a/bpython/inspection.py b/bpython/inspection.py
@@ -358,6 +358,9 @@ def getattr_safe(obj: Any, name: str) -> Any:
     # Slots are a MemberDescriptorType
     if isinstance(result, MemberDescriptorType):
         result = getattr(obj, name)
+    # classmethods are safe to access (see #966)
+    if isinstance(result, classmethod):
+        result = result.__get__(obj, obj)
     return result
 
 
diff --git a/bpython/test/test_inspection.py b/bpython/test/test_inspection.py
@@ -180,7 +180,6 @@ def fun_annotations(number: int, lst: List[int] = []) -> List[int]:
         self.assertEqual(props.argspec.args, ["number", "lst"])
         self.assertEqual(props.argspec.defaults[0], [])
 
-    @unittest.expectedFailure
     def test_issue_966_class_method(self):
         class Issue966(Sequence):
             @classmethod
