Open daviddavis opened 1 year ago
This is by design/deliberate (which does not mean it should not be changed). Our reasoning was that we have configured the CI and our dev boxes to allow these checksums to facilitate these tests, so there was never any pressure to change. If you know of a better way to facilitate testing on boxes that don't have the setting, I am open to changing things.
Did you catch my proposal to skip the test if the checksum is not enabled? Here's an example from pulp_rpm:
@daviddavis I read the proposal to skip but I did not look at the RPM example. (One can see a lot by looking)
I am all for copying pulp_rpm and keeping plugins similar. Will remove the Triage-Needed
label.
If you don't configure ALLOWED_CONTENT_CHECKSUMS, it defaults to everything but md5 and sha1. And it looks like tests will fail in this case:
I think ideally the test should either not rely on these checksums or be skipped if the checksums they require a certain checksum that is not enabled. Here's an example of the latter:
https://github.com/pulp/pulp_rpm/blob/77b19890daf5ceb7a11aa701033d81e97983468d/pulp_rpm/tests/functional/api/test_sync.py#L797-L800