You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Docstrings in wrapped functions must be detected as well.
>>> 'one other test'
'one other test'
"""
The problem is that it does not check that tests are actually executed.
If you remove this doctest from it, this will be the result:
» ./python.exe -m test test_doctest
Using random seed: 2548604185
0:00:00 load avg: 1.71 Run 1 test sequentially
0:00:00 load avg: 1.71 [1/1] test_doctest
== Tests result: SUCCESS ==
1 test OK.
Total duration: 955 ms
Total tests: run=67
Total test files: run=1/1
Result: SUCCESS
So, it does not test anything. And it will continue to pass if Wrapper won't handle __doc__ correctly at some point.
Uh oh!
There was an error while loading. Please reload this page.
Bug report
Here's a test that ensures that
Wrapper
on top of it handles__doc__
correcty:cpython/Lib/test/test_doctest/test_doctest.py
Lines 2539 to 2554 in d8f3503
The problem is that it does not check that tests are actually executed.
If you remove this doctest from it, this will be the result:
So, it does not test anything. And it will continue to pass if
Wrapper
won't handle__doc__
correctly at some point.I have a PR ready.
Linked PRs
test_doctest.test_look_in_unwrapped
#118122test_doctest.test_look_in_unwrapped
(#118122) #118129The text was updated successfully, but these errors were encountered: