Feature #5330: Test suite: identify and document race conditions
Accessing systray icons in the test suite is fragile
Wait for systray icons to finish loading
When restoring snapshots in the test suite we'll wait for the for the
Florence icon to be displayed in the system tray to ensure that the
icons won't shift positions because another application icon was added
to the system tray.
Once the icon for Florence Virtual Keyboard has loaded, the only icon
left to load is Vidalia. Vidalia's loading will not cause the other
systray icons to shift positions.
#3 Updated by kytv over 4 years ago
We want to wait until all of the systray icons have loaded before trying to interact with any of the icons in the systray. Otherwise, Sikuli may try clicking
gpgApplet, for example, but if the systray is still being filled it could end up clicking something else entirely such as the notification icon or the volume control.
#8 Updated by anonym over 4 years ago
- Assignee changed from anonym to kytv
- % Done changed from 0 to 20
- QA Check changed from Ready for QA to Dev Needed
IMHO the correct place to put the code isn't
restore_background(). It's conceivable that systray interaction is done before saving/restoring the background snapshot, and scenarios without any snapshot at all are already existing.
I think we should replace the
GNOME has started step with with
the Tails desktop is ready step, which does what
GNOME has started does and what you added to
restore_background(). And more things, later on, as we come up with them. So I wouldn't mind a
sed -i 's,GNOME has started,the Tails desktop is ready,g' $(git grep -l "GNOME has started").