Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] One more attempt to fix gui tests #3911

Merged
merged 1 commit into from
May 17, 2023

Conversation

vodorok
Copy link
Collaborator

@vodorok vodorok commented May 16, 2023

The order of two testcases was swapped, and some of the click() calls
got a 100 msec pause inserted before them.
I've yet to discover the reason why these momentary pauses fix the
shadowing, but it looks like it is working.

@vodorok vodorok requested a review from bruntib as a code owner May 16, 2023 16:30
@vodorok vodorok force-pushed the gui_fix_vol_432 branch 2 times, most recently from 26b5da2 to 2f6a3e6 Compare May 16, 2023 17:13
@vodorok vodorok added GUI 🎨 bugfix 🔨 test ☑️ Adding or refactoring tests labels May 16, 2023
@vodorok vodorok force-pushed the gui_fix_vol_432 branch 2 times, most recently from 9e1a61f to 1817693 Compare May 16, 2023 22:19
@Szelethus Szelethus added this to the release 6.22.2 milestone May 17, 2023
@vodorok vodorok force-pushed the gui_fix_vol_432 branch 3 times, most recently from 7e0e2fd to 5768d0e Compare May 17, 2023 12:06
The order of two testcases was swapped, and some of the click() calls
got a 100 msec pause inserted before them.
I've yet to discover the reason why these momentary pauses fix the
shadowing, but it looks like it is working.
@bruntib bruntib merged commit 6362ae6 into Ericsson:master May 17, 2023
@vodorok vodorok deleted the gui_fix_vol_432 branch May 22, 2023 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix 🔨 GUI 🎨 test ☑️ Adding or refactoring tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants