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

[Albert Astals Cid]

[test] Check the Launcher does not push itself

[Michał Sawicz (saviq)]

review: Approve (functional)

[Gerry Boland (gerboland)]

Please don't use "system", use $SUT.execute_shell_command

Otherwise good. We need to figure out simpler way to write these nested verify calls :)

review: Needs Fixing

[Albert Astals Cid (aacid)]

Fixed, sorry copy&pasted from wrong branch

[lp:~aacid/unity-2d/unity-2d_test_launcher_pushes_itself]

869. By Albert Astals Cid on 2012-01-25

do not use system but execute_shell_command

[Gerry Boland (gerboland)]

Thank you!

=== modified file 'tests/launcher/always_visible_tests.rb'
--- tests/launcher/always_visible_tests.rb	2012-01-24 14:54:32 +0000
+++ tests/launcher/always_visible_tests.rb	2012-01-25 16:14:24 +0000
@@ -95,4 +95,30 @@
       }
       xid.close!
   end
+
+  # Test case objectives:
+  #   * Check the Launcher does not push itself
+  # Pre-conditions
+  #   * Desktop with no running applications
+  # Test steps
+  #   * Set the visibility behaviour to intellihide
+  #   * Verify launcher x is 0
+  #   * Set the visibility behaviour to always visible
+  #   * Verify launcher x is not different than 0 for 1 continued second
+  # Post-conditions
+  #   * None
+  # References
+  #   * None
+  test "Launcher does not push itself" do
+      $SUT.execute_shell_command 'gsettings set com.canonical.Unity2d.Launcher hide-mode 2'
+      verify_equal( 0, TIMEOUT, 'Launcher hiding on empty desktop, should be visible' ) {
+        @app.Launcher()['x_absolute'].to_i
+      }
+      $SUT.execute_shell_command 'gsettings set com.canonical.Unity2d.Launcher hide-mode 0'
+      verify_not(0, 'Launcher x should be 0 after setting the always visible mode') {
+        verify_true( 1 ) {
+          @app.Launcher()['x_absolute'].to_i != 0
+        }
+      }
+  end
 end