-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Sort parametrize params to test_external_plugins_integrated #5563
Conversation
This might cause problems during collection with pytest-xdist; we didn't see any so far mostly by luck I think. Shame on me for letting that slip in.
tox.ini
Outdated
@@ -131,6 +131,7 @@ filterwarnings = | |||
ignore:yield tests are deprecated, and scheduled to be removed in pytest 4.0:pytest.RemovedInPytest4Warning | |||
ignore:Metafunc.addcall is deprecated and scheduled to be removed in pytest 4.0:pytest.RemovedInPytest4Warning | |||
ignore::pytest.RemovedInPytest4Warning | |||
default:Using or importing the ABCs:DeprecationWarning |
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.
Unrelated to the main purpose of the PR, but Travis is now issuing this warning:
/home/travis/build/pytest-dev/pytest/.tox/py37-lsof-numpy-twisted-xdist/lib/python3.7/site-packages/unittest2/compatibility.py:143: in <module>
class ChainMap(collections.MutableMapping):
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
name = 'MutableMapping'
def __getattr__(name):
# For backwards compatibility, continue to make the collections ABCs
# through Python 3.6 available through the collections module.
# Note, no new collections ABCs were added in Python 3.7
if name in _collections_abc.__all__:
obj = getattr(_collections_abc, name)
import warnings
warnings.warn("Using or importing the ABCs from 'collections' instead "
"of from 'collections.abc' is deprecated, "
"and in 3.8 it will stop working",
> DeprecationWarning, stacklevel=2)
E DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
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.
I think this is redundant with #5565, right? In which case I'd rather wait for that rather than suppress the warning, since it could indicate a real problem somewhere else!
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.
Changed this to be specific to unittest2
now, given that we won't be actually dropping unittest2 until 6.0.
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.
@Zac-HD can you please approve if this looks good to you now? Thanks! 🙇
Codecov Report
@@ Coverage Diff @@
## master #5563 +/- ##
==========================================
+ Coverage 93.83% 96.11% +2.27%
==========================================
Files 117 117
Lines 25713 25713
Branches 2493 2493
==========================================
+ Hits 24128 24714 +586
+ Misses 1256 695 -561
+ Partials 329 304 -25
Continue to review full report at Codecov.
|
This is being raised by `unittest2.compatibility`: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
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.
Looks good to me!
This might cause problems during collection with pytest-xdist; we
didn't see any so far mostly by luck I think.
Shame on me for letting that slip in. 😓