IntenseWebs/sphinx
3
0
Fork 0
mirror of https://github.com/sphinx-doc/sphinx.git synced 2025-02-25 18:55:22 -06:00
Code Issues Packages Projects Releases Wiki Activity

Use recursive call to ismock() on unbound method instead of using temporary object.

Browse Source
This commit is contained in:
Christian Walch 2022-03-31 09:33:54 +02:00
parent d3949f8481
commit 4a977ba4e1
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sphinx/ext/autodoc/mock.py
View File

@ -173,13 +173,11 @@ def ismock(subject: Any) -> bool:
# check the object is bound method # check the object is bound method
if isinstance(subject, MethodType) and _method_is_bound(subject): if isinstance(subject, MethodType) and _method_is_bound(subject):
tmp_subject = subject.__func__ return ismock(subject.__func__)
else:
tmp_subject = subject
try: try:
# check the object is mocked object # check the object is mocked object
__mro__ = safe_getattr(type(tmp_subject), '__mro__', []) __mro__ = safe_getattr(type(subject), '__mro__', [])
if len(__mro__) > 2 and __mro__[-2] is _MockObject: if len(__mro__) > 2 and __mro__[-2] is _MockObject:
# A mocked object has a MRO that ends with (..., _MockObject, object). # A mocked object has a MRO that ends with (..., _MockObject, object).
return True return True