Merge lp:~canonical-platform-qa/autopilot/tests_for_known_apps into lp:autopilot

[Leo Arias]

Added tests to open all the known apps.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:521
http://jenkins.qa.ubuntu.com/job/autopilot-ci/948/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-amd64-ci/11
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-amd64-ci/11/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-armhf-ci/11
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-armhf-ci/11/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-i386-ci/11
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-i386-ci/11/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-vivid-autopilot/15
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-vivid-autopilot/22
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/407
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/407/artifact/work/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

[Richard Huddie (rhuddie)]

I've tried to run the new test, using updated ProcessManager.KNOWN_APPS from lp:~canonical-platform-qa/autopilot/fix1411096-update_desktop. But I get failures for Calculator and System Settings, as seen in Jenkins: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-vivid-autopilot/15/artifact/results/autopilot/results/test_python3_autopilot.xml

[Richard Huddie (rhuddie)]

My environment was getting KNOWN_APPS from installed /usr/lib/python so was failing due to using the old values for desktop file. The test passes now it is using the updated version. So setting to approved.

review: Approve

[Nicholas Skaggs (nskaggs)]

Approved, with comment. I don't think it makes sense to invest time to improve this. Perhaps even if could be argued to drop it at some point.

review: Approve

[Thomi Richards (thomir)]

+1 from me, if CI passes.

review: Approve

[lp:~canonical-platform-qa/autopilot/tests_for_known_apps]

522. By Leo Arias on 2015-01-19

Used start app window instead.

523. By Leo Arias on 2015-01-19

Only check that the app is not none.

524. By Leo Arias on 2015-01-19

Also check the desktop file.

525. By Leo Arias on 2015-01-19

Reverted the last commit.

[Leo Arias (elopio)]

thomi, CI will pass this on utopic, not on vivid.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:525
http://jenkins.qa.ubuntu.com/job/autopilot-ci/964/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-amd64-ci/27
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-amd64-ci/27/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-armhf-ci/27
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-armhf-ci/27/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-i386-ci/27
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-vivid-i386-ci/27/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-vivid-autopilot/31
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-vivid-autopilot/37
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/441
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/441/artifact/work/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

=== modified file 'autopilot/tests/functional/test_process_emulator.py'
--- autopilot/tests/functional/test_process_emulator.py	2014-10-22 17:23:02 +0000
+++ autopilot/tests/functional/test_process_emulator.py	2015-01-19 17:24:04 +0000
@@ -1,7 +1,7 @@
 # -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
 #
 # Autopilot Functional Test Tool
-# Copyright (C) 2012-2014 Canonical
+# Copyright (C) 2012, 2013, 2014, 2015 Canonical
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -119,6 +119,25 @@
         self.assertThat(list(window.geometry), Equals(proxy_window.geometry))
 
 
+@skipIf(model() != "Desktop", "Not suitable for device (ProcManager)")
+class StartKnowAppsTests(AutopilotTestCase):
+
+    scenarios = [
+        (app_name, {
+            'app_name': app_name,
+            'desktop_file': (
+                ProcessManager.KNOWN_APPS[app_name]['desktop-file'])
+        })
+        for app_name in ProcessManager.KNOWN_APPS
+    ]
+
+    def test_start_app_window(self):
+        """Ensure we can start all the known applications."""
+        app = self.process_manager.start_app_window(self.app_name, locale='C')
+
+        self.assertThat(app, NotEquals(None))
+
+
 class ProcessManagerApplicationNoCleanupTests(AutopilotTestCase):
     """Testing the process manager without the automated cleanup that running
     within as an AutopilotTestCase provides.