-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add mark that allows us to do skip tests on backends via decorators #353
Conversation
Err, sorry can you include the consumers of this mark and the removal of "only_if" from the generate functions, but not anything else? Sorry if my previous comment was confusing. |
Test PASSed. |
Test PASSed. |
only_if=lambda backend: backend.hash_supported(hashes.SHA1), | ||
skip_message="Does not support SHA1", | ||
) | ||
def test_SHA1(self, backend): | ||
base_hash_test(backend, hashes.SHA1(), digest_size=20, block_size=64) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes here with removing the generate_*
stuff is what I didn't want to review in this patch, can bit be removed?
Test FAILed. |
Add mark that allows us to do skip tests on backends via decorators
check the result here
Splitting the actual pytest changes into a separate PR from #348.