Skip to content

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

Merged
nicoddemus merged 1 commit into
pytest-dev:featuresfrom
nicoddemus:deprecate-pytest-raises-message
Dec 13, 2018
Merged

Deprecate the 'message' parameter of pytest.raises#4539
nicoddemus merged 1 commit into
pytest-dev:featuresfrom
nicoddemus:deprecate-pytest-raises-message

Conversation

@nicoddemus

Copy link
Copy Markdown
Member

Fix #3974

@asottile

Copy link
Copy Markdown
Member

ah doctests, I always forget those too ;)

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Copy link
Copy Markdown
Member

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

Comment thread testing/test_pytester.py
@nicoddemus
nicoddemus force-pushed the deprecate-pytest-raises-message branch from 9741a8c to 18fd43e Compare December 12, 2018 23:53
@nicoddemus

Copy link
Copy Markdown
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
Copy Markdown
Member Author

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

@codecov

codecov Bot commented Dec 13, 2018

Copy link
Copy Markdown

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

rth commented Dec 13, 2018

Copy link
Copy Markdown

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