You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When using pytest.raises() as a context manager, you can pass an exception type or tuple of exception types; it's then an error if no exception is raised or if the exception raised is not an instance of the expected type(s). The same logic applies to pytest.warns(), which has a near-identical API.
The problem is that if you pass the empty tuple (), this will always result in an error: even if an exception is raised, it can't be an instance of ()! I think we should explicitly check tuple inputs, and raise a more helpful error message if they're empty. For example:
"Passing expected_exception=() is an error, because it's impossible to raise an exception which is not an instance of any type. Raising exceptions is already understood as failing the test, so you don't need any special code to say 'this should never raise an exception'."
(for bonus points, pytest.raises(None) should have the same message, with =None instead of =())
The same logic, and same error message, applies to the raises= argument to pytest.mark.xfail().
When using
pytest.raises()as a context manager, you can pass an exception type or tuple of exception types; it's then an error if no exception is raised or if the exception raised is not an instance of the expected type(s). The same logic applies topytest.warns(), which has a near-identical API.The problem is that if you pass the empty tuple
(), this will always result in an error: even if an exception is raised, it can't be an instance of()! I think we should explicitly check tuple inputs, and raise a more helpful error message if they're empty. For example:expected_exception=()is an error, because it's impossible to raise an exception which is not an instance of any type. Raising exceptions is already understood as failing the test, so you don't need any special code to say 'this should never raise an exception'."(for bonus points,
pytest.raises(None)should have the same message, with=Noneinstead of=())raises=argument topytest.mark.xfail().expected_warning=()is an error, because it's impossible to emit a warning which is not an instance of any type. To assert that no warnings are emitted, use <whatever we come up with for Improve pytest.warns() docs to clarify difference with catch_warnings() #9002>