Merge pull request #1354 from gdyuldin/no_raise_message

Add expected exceptions to pytest.raises fail message
This commit is contained in:
Ronny Pfannschmidt 2016-02-03 13:54:24 +01:00
commit 05a26d995b
3 changed files with 8 additions and 2 deletions

View File

@ -49,6 +49,7 @@
``--failed-first`` respectively.
Thanks `@MichaelAquilina`_ for the PR.
* Added expected exceptions to pytest.raises fail message
**Bug Fixes**

View File

@ -1311,7 +1311,7 @@ def raises(expected_exception, *args, **kwargs):
func(*args[1:], **kwargs)
except expected_exception:
return _pytest._code.ExceptionInfo()
pytest.fail("DID NOT RAISE")
pytest.fail("DID NOT RAISE {0}".format(expected_exception))
class RaisesContext(object):
def __init__(self, expected_exception):
@ -1770,7 +1770,6 @@ class FixtureLookupError(LookupError):
# the last fixture raise an error, let's present
# it at the requesting side
stack = stack[:-1]
for function in stack:
fspath, lineno = getfslineno(function)
try:

View File

@ -70,3 +70,9 @@ class TestRaises:
def test_tuple(self):
with pytest.raises((KeyError, ValueError)):
raise KeyError('oops')
def test_no_raise_message(self):
try:
pytest.raises(ValueError, int, '0')
except pytest.raises.Exception as e:
assert e.msg == "DID NOT RAISE {0}".format(repr(ValueError))