srbala/os-autoinst-distri-fedora
1
0
Fork 0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2024-11-22 14:03:09 +00:00
Code Issues Releases Activity

Try and reduce timing failures in desktop_browser new tab

Browse Source 
The way this works at present, with a check_screen and then an
assert_and_click, there's a window where the check has passed
so we're committed to the assert, but it takes a half second or
so for the assert to actually complete (checking for a needle
is a somewhat heavy operation). During that half second the
'new update!' notification can...and quite often does...appear.
Changing the assert_and_click to a click_lastmatch should (I
hope) tighten this window; click_lastmatch should fire faster
than assert_and_click so there'll be less of a window for the
update notification to appear and break stuff.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2019-10-19 10:30:50 -07:00
parent e1030118f9
commit b12c529e34
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/desktop_browser.pm
View File

@ -9,7 +9,7 @@ sub _open_new_tab {
# always visible because GNOME might pop up a notification that # always visible because GNOME might pop up a notification that
# blocks it. so, we try both. # blocks it. so, we try both.
if (check_screen 'browser_new_tab') { if (check_screen 'browser_new_tab') {
assert_and_click 'browser_new_tab'; click_lastmatch;
} }
else { else {
send_key 'ctrl-t'; send_key 'ctrl-t';