Merge lp:~elopio/ubuntu-filemanager-app/cmake_autopilot_task into lp:ubuntu-filemanager-app

[Leo Arias]

[lp:~elopio/ubuntu-filemanager-app/cmake_autopilot_task]

170. By Leo Arias on 2014-04-30

Rename.

[David Planella (dpm)]

Thanks Leo. Could you add some more context on the MP's description? E.g.

- How is one now supposed to run the autopilot tests?
- Does this enable reusing Qt Creator's build, and thus supersedes https://code.launchpad.net/~dpm/ubuntu-filemanager-app/run-ap-from-out-of-source-build/+merge/217510 ?

review: Needs Information

[Nicholas Skaggs (nskaggs)]

David, this is one of the options we discussed, and yes it would supersede https://code.launchpad.net/~dpm/ubuntu-filemanager-app/run-ap-from-out-of-source-build/+merge/217510.

Essentially you would run the tests now using the target.

cmake .
make test-autopilot-build

One of the issues currently with the apps that have plugins that need to be built is the requirement you build the binary before running tests. Making it part of the build seems like a good idea, which is what I believe Leo is exploring here.

[Leo Arias (elopio)]

Right, that's the idea.
I would update the README to explain how to run them. But I'm missing a couple of things to finish this MP.

First, do you know what's the cmake variable name of the directory where the binary will be installed?
Second, how do we update the jenkins job to run the tests with this command instead of autopilot directly?

review: Needs Information

Unmerged revisions

170. By Leo Arias on 2014-04-30

Rename.

169. By Leo Arias on 2014-04-30

Added a CMake target for the autopilot tests.

=== modified file 'tests/autopilot/CMakeLists.txt'
--- tests/autopilot/CMakeLists.txt	2014-01-31 20:03:12 +0000
+++ tests/autopilot/CMakeLists.txt	2014-04-30 16:54:08 +0000
@@ -6,3 +6,8 @@
     DESTINATION ${PYTHON_PACKAGE_DIR}
     )
 endif(INSTALL_TESTS)
+
+add_custom_target(test-autopilot-build
+    COMMAND BINARY_PATH=${CMAKE_CURRENT_BINARY_DIR}/${EXEC} autopilot3 run ${AUTOPILOT_DIR}
+    WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+)

=== modified file 'tests/autopilot/filemanager/tests/__init__.py'
--- tests/autopilot/filemanager/tests/__init__.py	2014-04-28 21:06:30 +0000
+++ tests/autopilot/filemanager/tests/__init__.py	2014-04-30 16:54:08 +0000
@@ -59,8 +59,8 @@
         self.source_dir = os.path.dirname(
             os.path.dirname(os.path.abspath('.')))
         self.build_dir = self._get_build_dir()
-        self.local_location_binary = os.path.join(self.build_dir,
-                                                  'src', 'app', self.EXEC)
+
+        self.local_location_binary = os.environ.get('BINARY_PATH', None)
         self.installed_location_binary = os.path.join('/usr/bin/', self.EXEC)
         self.installed_location_qml = \
             '/usr/share/filemanager/qml/filemanager.qml'