mirror of
https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git
synced 2024-11-21 21:43:08 +00:00
Fix new logic in Clocks test
Whoops, we can't just use a straight match_has_tag there as we did another assert in the middle... Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
parent
5332b99076
commit
cc832a4ad8
@ -35,10 +35,11 @@ sub run {
|
|||||||
# assume it starts at 'inactive' once we are no longer testing
|
# assume it starts at 'inactive' once we are no longer testing
|
||||||
# < 46.0 anywhere
|
# < 46.0 anywhere
|
||||||
assert_screen(["clocks_alarm_active", "clocks_alarm_inactive"]);
|
assert_screen(["clocks_alarm_active", "clocks_alarm_inactive"]);
|
||||||
|
my $wasactive = match_has_tag("clocks_alarm_active");
|
||||||
# Now toggle the switch to change its state
|
# Now toggle the switch to change its state
|
||||||
assert_and_click("gnome_button_toggle");
|
assert_and_click("gnome_button_toggle");
|
||||||
# whichever state it was in, check it's now in the other
|
# whichever state it was in, check it's now in the other
|
||||||
if (match_has_tag("clocks_alarm_active")) {
|
if ($wasactive) {
|
||||||
assert_screen("clocks_alarm_inactive");
|
assert_screen("clocks_alarm_inactive");
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
|
Loading…
Reference in New Issue
Block a user