1
0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2025-12-24 01:05:58 +00:00

desktop_update_graphical: don't hit download twice

We already try to only hit download once by dropping it out of
the tags after the first time we hit it, but because we need
this check_screen to catch when *both* apply and download are
visible, we will click download twice if both apply and download
are visible on later loop iterations. So, let's guard against
that.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2025-12-03 09:08:12 -08:00
parent 712ca067a8
commit e310f6ed16

View File

@ -102,7 +102,7 @@ sub run {
# if we have a download button, we want to hit it, even if
# we also have a restart button. then continue waiting for
# apply (only)
if (check_screen 'desktop_package_tool_update_download') {
if (grep { $_ eq 'desktop_package_tool_update_download' } @$tags && check_screen 'desktop_package_tool_update_download') {
wait_screen_change { click_lastmatch; };
$n -= 1 if ($n > 1);
if (get_var("TAG") || get_var("COPR")) {