-
Notifications
You must be signed in to change notification settings - Fork 48
Pytest fixes #9
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
Pytest fixes #9
Conversation
I am looking into integrating coverage again, let's wait for this to be rebased on that then to ensure nothing is missed. |
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.
This looks generally good. I agree that waiting on the coverage changes (which I'll look at in a sec) is a good idea.
Fixes > "cannot collect test class %r because it has a __init__ constructor Ref: pytest-dev/pytest#2007
This is faster with test collection.
Fixes > /usr/lib/python3.7/site-packages/_pytest/python.py:764: > RemovedInPytest4Warning: usage of Generator.Function is deprecated, > please use pytest.Function instead
Codecov Report
@@ Coverage Diff @@
## master #9 +/- ##
==========================================
- Coverage 52.94% 52.92% -0.02%
==========================================
Files 125 125
Lines 14746 14741 -5
Branches 2502 2500 -2
==========================================
- Hits 7807 7802 -5
- Misses 6399 6400 +1
+ Partials 540 539 -1
Continue to review full report at Codecov.
|
Should be good it seems. |
No description provided.