Merge lp:~renatofilho/mediaplayer-app/mediaplayer-autopilot-plus into lp:mediaplayer-app

[Renato Araujo Oliveira Filho]

Created CMAKE options used by autopilot targed.
 * ENABLE_AUTOPILOT - Enable or Disable autopilot tests
 * AUTOPILOT_RECORD - Enable or Disable autopilot record tests
 * AUTOPILOT_RECORD_PATH - Directory to put recorded tests

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:52
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/94/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/./build=panda-pbuilder,distribution=quantal,flavor=armhf/94/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/94//rebuild/?

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

16 + message(STATUS "Autopilot tests disable: autopilot binary not found")

typo: should be "disabled", with a "d"

[lp:~renatofilho/mediaplayer-app/mediaplayer-autopilot-plus]

53. By Renato Araujo Oliveira Filho on 2013-03-05

Fixed typo.

[Renato Araujo Oliveira Filho (renatofilho)]

> 16 + message(STATUS "Autopilot tests disable: autopilot binary not
> found")
>
> typo: should be "disabled", with a "d"
Fixed in rev. 53

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:53
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/95/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/./build=panda-pbuilder,distribution=quantal,flavor=armhf/95/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/mediaplayer-app-ci/95//rebuild/?

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

lgtm

review: Approve

=== modified file 'cmake/autopilot.cmake'
--- cmake/autopilot.cmake	2013-02-21 17:35:48 +0000
+++ cmake/autopilot.cmake	2013-03-05 19:20:26 +0000
@@ -1,8 +1,38 @@
 add_custom_target(autopilot)
 
+option(ENABLE_AUTOPILOT "Enable or Disable autopilot tests" On)
+option(AUTOPILOT_RECORD "Enable or Disable autopilot record tests" OFF)
+option(AUTOPILOT_RECORD_PATH "Directory to put recorded tests" OFF)
+
+if(ENABLE_AUTOPILOT)
+    find_program(AUTOPILOT_BIN autopilot)
+    if(AUTOPILOT_BIN)
+        message(STATUS "Autopilot tests enabled.")
+    else()
+        message(STATUS "Autopilot tests disabled: autopilot binary not found")
+    endif()
+endif()
+
+if(AUTOPILOT_RECORD OR AUTOPILOT_RECORD_PATH)
+    find_program(AUTOPILOT_REC_BIN recordmydesktop)
+    if(AUTOPILOT_REC_BIN)
+        message(STATUS "Record autopilot enabled")
+        if(AUTOPILOT_RECORD_PATH)
+            message(STATUS "Save autopilot tests video in: ${AUTOPILOT_RECORD_PATH}")
+            set(AUTOPILOT_TESTS_ARGS -r -rd ${AUTOPILOT_RECORD_PATH})
+        else()
+            set(AUTOPILOT_TESTS_ARGS -r)
+        endif()
+    else()
+        message(STATUS "recordmydesktop necessary for record autopilot tests not found.")
+        set(AUTOPILOT_TESTS_ARGS "")
+    endif()
+endif()
+
 function(declare_autopilot_test ENVIROMENT TEST_NAME WORKING_DIR)
-
-    add_custom_command(TARGET autopilot
-        COMMAND ${ENVIROMENT} autopilot run ${TEST_NAME}
-        WORKING_DIRECTORY ${WORKING_DIR}) 
+    if(AUTOPILOT_BIN)
+        add_custom_command(TARGET autopilot
+            COMMAND ${ENVIROMENT} autopilot run ${TEST_NAME} ${AUTOPILOT_TESTS_ARGS}
+            WORKING_DIRECTORY ${WORKING_DIR}) 
+    endif()
 endfunction()