-
Notifications
You must be signed in to change notification settings - Fork 214
pytest-cov 2.1.0 doesn't work with pytest 2.8.0 #91
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
Comments
I've updatd the test deps. Lets see if we got any failures. |
Cool, didn't know about hunter... it will certainly be useful. 😄 |
@ionelmc colitem is |
@kmike quite strange, but this makes your suite pass: diff --git a/pytest.ini b/pytest.ini
index 73d1696..69460aa 100644
--- a/pytest.ini
+++ b/pytest.ini
@@ -1,5 +1,4 @@
[pytest]
-usefixtures = chdir
python_files=test_*.py __init__.py
python_classes=
addopts = --doctest-modules --assert=plain |
With this change it will still fail diff --git a/tox.ini b/tox.ini
index 8b4c9e8..f071189 100644
--- a/tox.ini
+++ b/tox.ini
@@ -15,7 +15,7 @@ deps =
leveldb
-rtests/requirements.txt
commands =
- py.test --cov=scrapy --cov-report= {posargs:scrapy tests}
+ py.test {posargs:scrapy tests}
[testenv:precise]
basepython = python2.7 I'm sorry but I don't think this a pytest-cov issue at all 😁 |
@ionelmc thanks for checking it! |
This is what I get:
pytest-cov is not installed and it still fails |
Maybe @nicoddemus or @RonnyPfannschmidt or @flub have different input here. To me this looks like a unittest integration bug (same finalizers being called more than once). I avoid the TestCases like the plague so it's not something that I understand very well. A more detailed trace:
It seems |
@ionelmc ouch, sorry. I was trying to figure out what's the best place to fire this bug (pytest, pytest-cov or pytest-twisted), and disabled pytest-cov and pytest-twisted plugins, but it was a late night :) You're right, it fails without pytest-cov. Thanks for the help! |
Related bug in pytest: pytest-dev/pytest#1057 |
Hey,
It looks like pytest-cov 2.1.0 is not compatible with pytest 2.8.0 (released a couple days ago): every other test fails. It works fine with pytest 2.7.0 or with pytest-cov plugin disabled.
The text was updated successfully, but these errors were encountered: