Merge lp:~aacid/unity-2d/unity-2d_test_launcher_show_desktop into lp:unity-2d

Proposed by Albert Astals Cid
Status: Merged
Approved by: Gerry Boland
Approved revision: 852
Merged at revision: 850
Proposed branch: lp:~aacid/unity-2d/unity-2d_test_launcher_show_desktop
Merge into: lp:unity-2d
Diff against target: 42 lines (+17/-6)
2 files modified
tests/launcher/autohide_show_tests.rb (+17/-0)
tests/manual-tests/launcher.txt (+0/-6)
To merge this branch: bzr merge lp:~aacid/unity-2d/unity-2d_test_launcher_show_desktop
Reviewer Review Type Date Requested Status
Gerry Boland (community) Needs Fixing
Review via email: mp+88843@code.launchpad.net

Description of the change

[test] Check that showing the desktop shows also the launcher

To post a comment you must log in.
Revision history for this message
Gerry Boland (gerboland) wrote :

Ctrl+Alt+D is not the correct keystroke for this, see the Google Doc on Shortcuts.

I suggest you use Xdo::Window.toggle_minimize_all which pretty much does the same thing

I'd also prefer a mode descriptive name like "check launcher shows when show-desktop triggered"

review: Needs Fixing
850. By Albert Astals Cid

Use XDo::XWindow.toggle_minimize_all instead ctrl+alt+d

According to Gerry it's the same, just more portable to different configs

851. By Albert Astals Cid

Eeeek, forgot to remove this

852. By Albert Astals Cid

This has a testability test now

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/launcher/autohide_show_tests.rb'
2--- tests/launcher/autohide_show_tests.rb 2012-01-17 11:25:32 +0000
3+++ tests/launcher/autohide_show_tests.rb 2012-01-17 15:30:38 +0000
4@@ -213,5 +213,22 @@
5 'terminal does not have focus when it should' )
6 xid.close!
7 end
8+
9+ test "Launcher visible on show-desktop" do
10+ xid = open_window_at(10,100)
11+ sleep 0.5
12+ assert_equal( Integer(@app.Unity2dPanel()['x_absolute']), -WIDTH, \
13+ 'Launcher visible with window in the way, should be hidden' )
14+ XDo::XWindow.toggle_minimize_all # This is effectively the show-desktop shortcut
15+ sleep 2
16+ assert_equal( Integer(@app.Unity2dPanel()['x_absolute']), 0, \
17+ 'Launcher hiding after triggering show-desktop, should be visible' )
18+
19+ XDo::XWindow.toggle_minimize_all
20+ sleep 2
21+ assert_equal( Integer(@app.Unity2dPanel()['x_absolute']), -WIDTH, \
22+ 'Launcher visible with window in the way, should be hidden' )
23+ xid.close!
24+ end
25
26 end
27
28=== modified file 'tests/manual-tests/launcher.txt'
29--- tests/manual-tests/launcher.txt 2011-12-07 15:01:18 +0000
30+++ tests/manual-tests/launcher.txt 2012-01-17 15:30:38 +0000
31@@ -77,12 +77,6 @@
32 -> Launcher is hidden - The launcher should not still be visible when you have the mouse on the top panel (lp:892004)
33
34 ----
35- * Open an application & maximize the same
36- * Hit show-desktop shortcut [Ctrl+Alt+D?] key to see the Desktop
37-
38---> Launcher is revealed - Showing the Desktop should reveal launcher (lp:898161)
39-
40-----
41
42 * Drag a file(example .txt) onto a tile(gedit) in the launcher
43

Subscribers

People subscribed via source and target branches