Merge lp:~chris.gagnon/unity8/autopilot-helper-to-background-and-signal-app into lp:unity8

[Chris Gagnon]

add autopilot helpers to restart unity8 if it's not started with testability, put an app in the background and send a signal to an app

[lp:~chris.gagnon/unity8/autopilot-helper-to-background-and-signal-app]

914. By Chris Gagnon on 2014-05-21

moved signal to it's own function

915. By Chris Gagnon on 2014-05-21

get shell for restarted unity8

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:915
http://jenkins.qa.ubuntu.com/job/unity8-ci/3020/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/382
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/55
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/114
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/114
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/114/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/114
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/853
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/748
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/748/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/7459

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3020/rebuild

review: Approve (continuous-integration)

[Albert Astals Cid (aacid)]

Text conflict in tests/autopilot/unity8/process_helpers.py
1 conflicts encountered.

review: Needs Fixing

[Albert Astals Cid (aacid)]

Interestingly seems it doesn't conflict anymore

review: Abstain

[Chris Gagnon (chris.gagnon)]

I was using currentFocusedAppID but that doesn't exist for the shell anymore

[Leo Arias (elopio)]

I'm more interested in this one than in the rest:
restart_unity_if_not_testable()

Can you put it in a Fixture? Something called like: UnityWithTestability.

About send_app_to_background, is it better to call the slots, or would it be better to tell autopilot to click the home button? I have a branch that opens the launcher and clicks, in case you prefer that.

Instead of currentFocusedAppID, use focusedApplicationId. Even better if you use the method from MainWindow: get_current_focused_app_id

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:915
http://jenkins.qa.ubuntu.com/job/unity8-ci/3629/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/2555
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/631
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/723
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/723
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/723/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/723
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2700
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3798
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3798/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/10497

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3629/rebuild

review: Needs Fixing (continuous-integration)

[Michał Sawicz (saviq)]

Bump?

[Chris Gagnon (chris.gagnon)]

some of the internals of the shell has changed, I need to rework this mp

Unmerged revisions

915. By Chris Gagnon on 2014-05-21

get shell for restarted unity8

914. By Chris Gagnon on 2014-05-21

moved signal to it's own function

913. By Chris Gagnon on 2014-05-21

add helpers to restart unity, background app, send a signal to an app.

=== modified file 'tests/autopilot/unity8/process_helpers.py'
--- tests/autopilot/unity8/process_helpers.py	2014-05-14 22:48:45 +0000
+++ tests/autopilot/unity8/process_helpers.py	2014-05-21 12:17:26 +0000
@@ -29,6 +29,7 @@
 else:
     from autopilot.introspection import ProcessSearchError
 from autopilot.introspection import get_proxy_object_for_existing_process
+from testtools.matchers import NotEquals
 
 from unity8.shell import emulators
 from unity8.shell.emulators import main_window as main_window_emulator
@@ -75,7 +76,7 @@
             main_window_emulator.QQuickView)
 
     greeter = main_window.get_greeter()
-    if greeter.created == False:
+    if greeter.created is False:
         raise RuntimeWarning("Greeter appears to be already unlocked.")
 
     # Because of potential input jerkiness under heavy load,
@@ -98,7 +99,8 @@
 
 def lock_unity(unity_proxy_obj=None):
     """Helper function that attempts to lock the unity greeter."""
-    import evdev, time
+    import evdev
+    import time
     uinput = evdev.UInput(name='unity8-autopilot-power-button',
                           devnode='/dev/autopilot-uinput')
     # One press and release to turn screen off (locking unity)
@@ -113,7 +115,7 @@
 
 
 def restart_unity_with_testability(*args):
-    """Restarts (or starts) unity with testability enabled.
+    """Restart (or starts) unity with testability enabled.
 
     Passes *args arguments to the launched process.
 
@@ -122,6 +124,36 @@
     return restart_unity(*args)
 
 
+def restart_unity_if_not_testable():
+    """Return unity shell, restart unity if needed."""
+    try:
+        pid = _get_unity_pid()
+        unity8 = get_proxy_object_for_existing_process(pid)
+        shell = unity8.select_single("Shell")
+    except:
+        pid = restart_unity_with_testability()
+        unity8 = get_proxy_object_for_existing_process(pid)
+        shell = unity8.select_single("Shell")
+    return shell
+
+
+def send_app_to_background(shell, app_name):
+    """Put app in background app
+    :param shell: The unity8 shell
+    :param app_name: The name of the app to put in the background
+    """
+    shell.slots.showHome()
+    shell.currentFocusedAppId.wait_for(NotEquals(app_name))
+
+
+def signal_app(app_proxy_object, signal):
+    """Send signal to app
+    :param app_proxy_object: Proxy object of app you want to send a signal to
+    :param signal: The signal to send the app
+    """
+    app_proxy_object.process.send_signal(signal)
+
+
 def restart_unity(*args):
     """Restarts (or starts) unity8 using the provided arguments.