-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Fix up some pytest style issues #6968
Conversation
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Is there a link that you can provide, describing this pytest style preference? |
@radarhere Kind of:
IOW, Pillow's tests themselves overwhelmingly use |
And I find |
Could this be merged? CI is green across the board. |
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
This PR fixes tests to use
pytest.raises
andpytest.warns
as context managers everywhere possible.