-
-
Notifications
You must be signed in to change notification settings - Fork 60
Fail tests with uncaught warnings #102
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
Comments
do you mean to change the pytest settings to strict? Because I believe that by default pytest should fail test if it has uncaught exception |
Oh I mean uncaught warnings |
I'd like to do it then, do you have something specific in mind? I'd just create the pytest.ini file and use "addopts = -W error" global setting, then all warnings should fail the test. Later on we can exclude specific types of warnings if need be |
According to pytest you can specify
https://docs.pytest.org/en/7.1.x/how-to/capture-warnings.html Yes that should be a good start. Feel free to open a PR |
The earlier we start, the easier it is :)
The text was updated successfully, but these errors were encountered: