1
0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2025-05-02 07:31:33 +00:00

navigation: fix details check logic

The current check never fails - if we don't see the details after
30 seconds, we never actually assert them. We may or may not
soft fail, but we'll never fail.

This simplifies the check (there's no need to specifically look
for the 'loading' screen) and makes it actually fail if the
details don't show up in 90 seconds total.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2025-03-20 15:09:58 -07:00
parent 4d4a0cf312
commit 9715d39928
3 changed files with 1 additions and 27 deletions

View File

@ -1,22 +0,0 @@
{
"area": [
{
"ypos": 723,
"width": 103,
"height": 27,
"type": "match",
"xpos": 534
},
{
"type": "match",
"height": 34,
"xpos": 561,
"width": 40,
"ypos": 424
}
],
"properties": [],
"tags": [
"navigation_details_notloaded"
]
}

Binary file not shown.

Before

Width:  |  Height:  |  Size: 318 KiB

View File

@ -31,11 +31,7 @@ sub run {
# Sometimes, the details take a time to load,
# if that happens, fail softly.
unless (check_screen('navigation_details_shown', timeout => 30)) {
if (check_screen('navigation_details_notloaded')) {
record_soft_failure('Window details not loaded in time.');
}
}
else {
record_soft_failure('Window details not loaded in time.');
assert_screen("navigation_details_shown", timeout => 60);
}
release_key("alt");