Skip to content

Remove pytest warnings and fix CI#92

Merged
oremanj merged 6 commits intopython-trio:masterfrom
pquentin:remove-pytest-warnings
May 20, 2020
Merged

Remove pytest warnings and fix CI#92
oremanj merged 6 commits intopython-trio:masterfrom
pquentin:remove-pytest-warnings

Conversation

@pquentin
Copy link
Member

Closes #90, Closes #91

We no longer have the trio-asyncio problem, but had to bump PyPI and stop testing Python 3.5.2 because pytest fails to start with it. I also added entries for more recent Python versions.

@pquentin pquentin force-pushed the remove-pytest-warnings branch from 581dff9 to f14615e Compare March 11, 2020 13:17
@codecov
Copy link

codecov bot commented Mar 11, 2020

Codecov Report

Merging #92 into master will decrease coverage by 0.40%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
- Coverage   99.79%   99.39%   -0.41%     
==========================================
  Files          19       18       -1     
  Lines         498      495       -3     
  Branches       43        0      -43     
==========================================
- Hits          497      492       -5     
- Misses          1        3       +2     
Impacted Files Coverage Δ
pytest_trio/_tests/conftest.py 100.00% <ø> (ø)
pytest_trio/_tests/test_fixture_names.py 100.00% <100.00%> (ø)
pytest_trio/plugin.py 99.04% <100.00%> (-0.47%) ⬇️
pytest_trio/_tests/test_async_yield_fixture.py 97.67% <0.00%> (-2.33%) ⬇️
pytest_trio/_tests/test_basic.py 100.00% <0.00%> (ø)
pytest_trio/_tests/test_trio_mode.py 100.00% <0.00%> (ø)
pytest_trio/_tests/test_contextvars.py 100.00% <0.00%> (ø)
pytest_trio/_tests/test_sync_fixture.py 100.00% <0.00%> (ø)
pytest_trio/_tests/test_async_fixture.py 100.00% <0.00%> (ø)
... and 6 more

@pquentin
Copy link
Member Author

Hmm, the Windows builds are full of failures, and it's surprising that we get coverage for all installed packages for Python 3.8 in Travis. So maybe that's a start to understand the coverage problem here.

@oremanj
Copy link
Member

oremanj commented May 20, 2020

Remaining coverage changes will be fixed by #94. Going to merge this to unblock downstream projects from updating to support 0.15.0.

@oremanj oremanj merged commit 10de71f into python-trio:master May 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests fail with pytest 5.1.0 no attribute RemovedInPytest4Warning

3 participants