Code review comment for lp:~dholbach/ubuntu-clock-app/reboot-packaging

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Daniel, I got the "make autopilot" to work. The first fix needed was already done by you in rev 51 (adjusting CMakeList.txt file to point at the new test path). The second fix needed is to rename ubuntu-clock-app to ubuntu_clock_app in the tests/autopilot/run file.

=== modified file 'tests/autopilot/run' (properties changed: -x to +x)
--- tests/autopilot/run 2014-07-31 16:58:56 +0000
+++ tests/autopilot/run 2014-08-05 10:13:21 +0000
@@ -7,5 +7,5 @@

 SCRIPTPATH=`dirname $0`
 pushd ${SCRIPTPATH}
-autopilot run ubuntu-clock-app
+autopilot run ubuntu_clock_app

After which I get it passing,

nik90@nik90-devel:~/Clock/add-alarm-test/builddir$ make autopilot
[ 0%] Built target alarmsettings-qmldir
[ 7%] Automoc for target timezone
[ 7%] Built target timezone_automoc
[ 42%] Built target timezone
[ 50%] Automoc for target datetime
[ 50%] Built target datetime_automoc
[ 71%] Built target datetime
[ 78%] Automoc for target alarmsettings
[ 78%] Built target alarmsettings_automoc
[100%] Built target alarmsettings
[100%] Built target timezone-qmldir
[100%] Built target datetime-qmldir
~/Clock/add-alarm-test/tests/autopilot ~/Clock/add-alarm-test/builddir/app
Loading tests from: /home/nik90/Clock/add-alarm-test/tests/autopilot

Tests running...

Ran 1 test in 48.013s
OK
~/Clock/add-alarm-test/builddir/app
[100%] Built target autopilot

« Back to merge proposal