-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doctest: hack in handling mock style objects #4212
Doctest: hack in handling mock style objects #4212
Conversation
Just FYI: GitHub is facing some issues so webhooks are disabled for the moment: https://status.github.com/messages |
i am aware ^^ |
Codecov Report
@@ Coverage Diff @@
## master #4212 +/- ##
==========================================
+ Coverage 95.73% 95.75% +0.02%
==========================================
Files 111 111
Lines 24686 24947 +261
Branches 2446 2503 +57
==========================================
+ Hits 23632 23887 +255
- Misses 741 748 +7
+ Partials 313 312 -1
Continue to review full report at Codecov.
|
cb22ac7
to
92873fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clever solution, awesome!
Please just add a docstring to _is_mocked
and _patch_unwrap_mock_aware
explaining what's going on to the unaware, including a reference to the issue. 🙇
92873fb
to
a6988aa
Compare
great, thanks |
@RonnyPfannschmidt Thx for this hack! I have applied it in pytest-sphinx as well. |
@Thisch im glad it could help, i was repulsed as i made it ^^ |
Resolves #3456