Serhiy Storchaka <[email protected]> added the comment:
Thank you Nikita.
The difference between the original and the proposed in PR 28060 code is that
in the original code the conditional was tested at the testing time, while in
the proposed code it is tested at the loading time. I do not know what effect
it causes on tests. If it does not matter, the code can be simpler:
return unittest.skipIf(should_be_skipped,
f"hash digest '{digestname}' is not available.")
But if it matters, there are two options:
1. In requires_hashdigest() raise exception if func is a class. Remove the
decorator from classes and add it to test and setup methods or just to the
setUpClass() classmethod.
2. In requires_hashdigest() check if func is a class, and if it is true, patch
its test and setup methods (or just add a decorated setUpClass() classmethod).
----------
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue45042>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
