Merge lp:~elopio/ubuntu-filemanager-app/cmake_autopilot_task into lp:ubuntu-filemanager-app
Status: | Work in progress |
---|---|
Proposed branch: | lp:~elopio/ubuntu-filemanager-app/cmake_autopilot_task |
Merge into: | lp:ubuntu-filemanager-app |
Diff against target: |
27 lines (+7/-2) 2 files modified
tests/autopilot/CMakeLists.txt (+5/-0) tests/autopilot/filemanager/tests/__init__.py (+2/-2) |
To merge this branch: | bzr merge lp:~elopio/ubuntu-filemanager-app/cmake_autopilot_task |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Leo Arias (community) | Needs Information on 2014-05-02 | ||
David Planella | 2014-04-30 | Needs Information on 2014-04-30 | |
Review via email:
|
Nicholas Skaggs (nskaggs) wrote : | # |
David, this is one of the options we discussed, and yes it would supersede https:/
Essentially you would run the tests now using the target.
cmake .
make test-autopilot-
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) wrote : | # |
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?
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? /code.launchpad .net/~dpm/ ubuntu- filemanager- app/run- ap-from- out-of- source- build/+ merge/217510 ?
- Does this enable reusing Qt Creator's build, and thus supersedes https:/