Merge lp:~dinko-metalac/sudoku-app/bug-fix-1227340 into lp:sudoku-app

[Dinko Osmankovic]

Bug fix for #1227340

Fix for bug #1227340

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:114
http://91.189.93.70:8080/job/sudoku-app-ci/21/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests/414
    SUCCESS: http://91.189.93.70:8080/job/sudoku-app-precise-amd64-ci/21
    SUCCESS: http://91.189.93.70:8080/job/sudoku-app-quantal-amd64-ci/21
    SUCCESS: http://91.189.93.70:8080/job/sudoku-app-raring-amd64-ci/21
    SUCCESS: http://91.189.93.70:8080/job/sudoku-app-saucy-amd64-ci/21

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/sudoku-app-ci/21/rebuild

review: Approve (continuous-integration)

[David Planella (dpm)]

Nice work Dinko! Just a small tip: if you use the --fixes lp:1227340 argument when you're doing the bzr commit, Launchpad will take care of marking the bug as Fix Committed automatically for you.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

review: Approve (continuous-integration)

[Dinko Osmankovic (dinko-metalac)]

Hehehe, I have no idea if its fixed now. It worked before on my side :D

=== modified file 'tests/autopilot/sudoku_app/tests/__init__.py'
--- tests/autopilot/sudoku_app/tests/__init__.py	2013-08-30 16:16:40 +0000
+++ tests/autopilot/sudoku_app/tests/__init__.py	2013-09-19 07:59:49 +0000
@@ -71,6 +71,18 @@
                                 pass
         return None
 
+    def launch_and_quit_app(self):
+        self.launch_app()
+        self.main_window.get_qml_view().visible.wait_for(True)
+        
+        # When calling launch_app an instance of the spawned process
+        # control object will be stored in self.app.process, and a cleanup
+        # handler will be registered that essentially kills the process.
+        # Therefore, by triggering the cleanup handler here we're killing the
+        # process and removing the handler, which allows a clean launch of
+        # the process during regular test setup.
+        self.doCleanups()
+
     def clean_db(self):
         path = self.find_db()
         if path is None: