Skip to content

Deprecate the 'message' parameter of pytest.raises #4539

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 #3974

@asottile
Copy link
Member

ah doctests, I always forget those too ;)

Copy link
Member

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that one should have been match, but its a good extra example that this is the right path forward

we mess up ourselves as is

@nicoddemus nicoddemus force-pushed the deprecate-pytest-raises-message branch from 9741a8c to 18fd43e Compare December 12, 2018 23:53
@nicoddemus
Copy link
Member Author

Hmmm better to remove the message example instead, will force-push soon.

@nicoddemus nicoddemus force-pushed the deprecate-pytest-raises-message branch from 18fd43e to 5b83417 Compare December 13, 2018 00:26
@nicoddemus
Copy link
Member Author

OK, ready. Also revamped the docs for pytest.raises. 👍

@codecov
Copy link

codecov bot commented Dec 13, 2018

Codecov Report

Merging #4539 into features will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##           features    #4539      +/-   ##
============================================
- Coverage     95.77%   95.76%   -0.02%     
============================================
  Files           111      111              
  Lines         24954    24961       +7     
  Branches       2462     2462              
============================================
+ Hits          23900    23904       +4     
  Misses          747      747              
- Partials        307      310       +3
Flag Coverage Δ
#docs 29.33% <22.22%> (+0.05%) ⬆️
#doctesting 29.33% <22.22%> (+0.05%) ⬆️
#linting 29.33% <22.22%> (+0.05%) ⬆️
#linux 95.58% <100%> (-0.02%) ⬇️
#nobyte 92.42% <100%> (-0.01%) ⬇️
#numpy 93.22% <100%> (-0.02%) ⬇️
#pexpect 41.74% <22.22%> (-0.01%) ⬇️
#py27 93.81% <100%> (-0.02%) ⬇️
#py34 91.93% <100%> (+0.06%) ⬆️
#py35 91.96% <100%> (+0.06%) ⬆️
#py36 91.97% <100%> (+0.06%) ⬆️
#py37 93.95% <100%> (ø) ⬆️
#trial 93.22% <100%> (-0.02%) ⬇️
#windows 93.96% <100%> (-0.02%) ⬇️
#xdist 93.81% <100%> (-0.02%) ⬇️
Impacted Files Coverage Δ
testing/test_pytester.py 87.45% <100%> (ø) ⬆️
src/_pytest/deprecated.py 100% <100%> (ø) ⬆️
testing/deprecated_test.py 100% <100%> (ø) ⬆️
testing/python/raises.py 94.11% <100%> (+0.04%) ⬆️
src/_pytest/python_api.py 97.47% <100%> (+0.01%) ⬆️
src/_pytest/cacheprovider.py 95.75% <0%> (-1.42%) ⬇️

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 110fe24...5b83417. Read the comment docs.

@nicoddemus nicoddemus merged commit 15b0a89 into pytest-dev:features Dec 13, 2018
@nicoddemus nicoddemus deleted the deprecate-pytest-raises-message branch December 13, 2018 08:01
@rth
Copy link

rth commented Dec 13, 2018

Thanks for making this happen!

gene1wood added a commit to gene1wood/pytest that referenced this pull request Sep 9, 2019
Remove reference to the `message` argument in the docs as it was deprecated in pytest-dev#4539
felixbiessmann pushed a commit to felixbiessmann/datawig that referenced this pull request Nov 10, 2019
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.

4 participants