bodhi icon indicating copy to clipboard operation
bodhi copied to clipboard

Satisfied required tests are not marked as such

Open msrb opened this issue 3 years ago • 6 comments

Hello,

There is this update: https://bodhi.fedoraproject.org/updates/FEDORA-2021-ace6e91761

The package has this gating.yaml: https://src.fedoraproject.org/rpms/binutils/blob/rawhide/f/gating.yaml

Bodhi correctly states that "All required tests passed", which is correct.

However, when I click on the "Automated Tests" tab, no tests are marked (with "*") as required.

This used to work 4 days ago: https://bodhi.fedoraproject.org/updates/FEDORA-2021-2d7967c391 (gating.yaml has changed since then)

Any idea what is going on there?

Thanks! :wink:

msrb avatar Feb 05 '21 11:02 msrb

Hi, just ran into this today with the following update:

  • https://bodhi.fedoraproject.org/updates/FEDORA-2021-be96b935c4

The tests tab says All required tests passed which seems to be a clickable link but nothing happens when used. And I see no mark which would indicate which tests are actually required.

psss avatar Sep 23 '21 12:09 psss

@AdamSaleh, could you please have a look?

psss avatar Oct 19 '21 15:10 psss

This might have been something that is fixed in new release currently in staging. I.e. I can see https://bodhi.stg.fedoraproject.org/updates/FEDORA-2021-ace6e91761 does have the star as expected.

AdamSaleh avatar Oct 19 '21 21:10 AdamSaleh

Nice, thanks for the update! Looking forward to the new release :)

psss avatar Oct 20 '21 07:10 psss

Ok, I have doublechecked the code, and would advise you to temper the excitement for new release.

Way we have written the code, the star/asterix marking required tests is applied only on unsatisfied requirements. We get both list of satisfied and unsatisfied requirements from greenwave, but only take process the unsatisfied. I will find somebody that would be able to make this ui improvement, but it won't make the release we are testing in staging.

AdamSaleh avatar Oct 20 '21 08:10 AdamSaleh

Understood, thanks much for moving this forward!

psss avatar Oct 20 '21 13:10 psss

I fixed this in 36cc800257005041362be3a8e9cf431993a3084c . I believe the ticket can be closed.

AdamWill avatar Sep 01 '23 00:09 AdamWill