Skip to content

Add tests for PartialUpdateImage

Erwan Rouchet requested to merge add-partialupdateimage-tests into master

While working on #1424 (closed), since I could not reproduce things easily in unit tests and the original Sentry issue was on PartialUpdateImage and not CreateImage, I started updating a whole bunch of tests. It turns out we have some tests for UpdateImage but not PartialUpdateImage.

I also added some assertNumQueries calls, which have made me notice a duplicate UPDATE that occurs when an image check fails. The image is twice updated to the Error state. I fixed that by adding a save=False to the perform_check call.

Merge request reports