Skip to content

-W now takes precedence over filters in ini files #3947

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

Merged

Conversation

nicoddemus
Copy link
Member

Fix #3946

cc @charris

@nicoddemus nicoddemus changed the base branch from master to features September 6, 2018 21:04
@coveralls
Copy link

coveralls commented Sep 6, 2018

Coverage Status

Coverage increased (+0.02%) to 93.905% when pulling ab40696 on nicoddemus:warnings-filter-precedence into d12f46c on pytest-dev:features.

@blueyed
Copy link
Contributor

blueyed commented Sep 9, 2018

Coverage appears to be broken in features? https://ci.appveyor.com/project/pytestbot/pytest/build/1.0.5140/job/g4rvb1s3n5vpsb8v

call scripts\prepare-coverage.bat
'scripts\prepare-coverage.bat' is not recognized as an internal or external command,
operable program or batch file.

But scripts\prepare-coverage.bat appears to be in origin/features for me?!

@blueyed
Copy link
Contributor

blueyed commented Sep 11, 2018

Ah, it was created before #3949 was merged.

@nicoddemus
I am going ahead and rebase it (old HEAD: cb2f525).

@blueyed blueyed force-pushed the warnings-filter-precedence branch from cb2f525 to ab40696 Compare September 11, 2018 17:00
@codecov
Copy link

codecov bot commented Sep 11, 2018

Codecov Report

Merging #3947 into features will decrease coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           features   #3947      +/-   ##
===========================================
- Coverage     94.48%   94.4%   -0.09%     
===========================================
  Files           107     107              
  Lines         23651   23665      +14     
  Branches       2349    2350       +1     
===========================================
- Hits          22347   22341       -6     
- Misses          993    1011      +18     
- Partials        311     313       +2
Flag Coverage Δ
#doctesting 28.51% <11.76%> (+0.04%) ⬆️
#linux 94.4% <100%> (+0.04%) ⬆️
#nobyte 0% <0%> (ø) ⬆️
#numpy 28.14% <11.76%> (-0.01%) ⬇️
#pexpect 0% <0%> (ø) ⬆️
#pluggymaster 93.55% <100%> (-0.06%) ⬇️
#py27 92.54% <100%> (-0.06%) ⬇️
#py34 92.08% <100%> (-0.07%) ⬇️
#py35 92.09% <100%> (-0.07%) ⬇️
#py36 92.62% <100%> (-0.11%) ⬇️
#py37 92.25% <100%> (-0.11%) ⬇️
#trial 31.04% <23.52%> (-0.18%) ⬇️
#windows ?
#xdist 17.97% <11.76%> (-0.49%) ⬇️
Impacted Files Coverage Δ
src/_pytest/warnings.py 88.6% <100%> (ø) ⬆️
testing/test_warnings.py 98.47% <100%> (+0.16%) ⬆️
testing/test_tmpdir.py 95.06% <0%> (-4.94%) ⬇️
src/_pytest/capture.py 86.72% <0%> (-3.21%) ⬇️
src/_pytest/nodes.py 93.69% <0%> (-0.85%) ⬇️
testing/acceptance_test.py 97.14% <0%> (-0.66%) ⬇️
src/_pytest/pytester.py 85.56% <0%> (-0.46%) ⬇️
testing/test_capture.py 98.94% <0%> (-0.31%) ⬇️
src/_pytest/fixtures.py 96.99% <0%> (-0.28%) ⬇️
testing/python/metafunc.py 95.06% <0%> (ø) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d12f46c...ab40696. Read the comment docs.

@nicoddemus nicoddemus merged commit 53c9124 into pytest-dev:features Sep 13, 2018
@nicoddemus
Copy link
Member Author

Thanks @blueyed for picking this up!

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.

3 participants