Merge lp://staging/~gerboland/unity-2d/test-launcher-reveal-tweak into lp://staging/unity-2d

Proposed by Gerry Boland
Status: Work in progress
Proposed branch: lp://staging/~gerboland/unity-2d/test-launcher-reveal-tweak
Merge into: lp://staging/unity-2d
Diff against target: 29 lines (+2/-6)
2 files modified
tests/launcher/autohide_show_tests.rb (+1/-3)
tests/launcher/autohide_show_tests_rtl.rb (+1/-3)
To merge this branch: bzr merge lp://staging/~gerboland/unity-2d/test-launcher-reveal-tweak
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Needs Fixing
Review via email: mp+97354@code.staging.launchpad.net

Description of the change

[tests] Use bash loop to push mouse pointer into pointerBarrier

This should make revealing the launcher more reliable, as repeatedly sending commands to qttasserver can be subject to transmission delays.

To post a comment you must log in.
Revision history for this message
Albert Astals Cid (aacid) wrote :

This actually makes it not work for me :-/

review: Needs Fixing
986. By Gerry Boland

Increase loop duration so reveal works at default settings

Revision history for this message
Albert Astals Cid (aacid) wrote :

Still doesn't work here, invoking the current shell command but with bash -c "" does work for me though

review: Needs Fixing

Unmerged revisions

986. By Gerry Boland

Increase loop duration so reveal works at default settings

985. By Gerry Boland

[test] Use bash loop to push mouse into pointerBarrier

This should make revealing the launcher more reliable, as repeatedly sending
commands to qttasserver can be subject to transmission delays.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches