Merge lp:~ken-vandine/content-hub/home_dir_for_tests into lp:content-hub

[Ken VanDine]

Set $HOME when running tests

Set $HOME when running tests

[Christopher Townsend (townsend)]

Seems reasonable as a workaround for the missing HOME env var on random builds.

review: Approve

[system-apps-ci-bot (system-apps-ci-bot)]

FAILED: Continuous integration, rev:336
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/12/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2424/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2424
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2235/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/2235/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2235/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/2235/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/2235/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/2235/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/12/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ken-vandine/content-hub/home_dir_for_tests]

337. By Ken VanDine on 2017-04-04

Run dbus-test-runner with --keep-env to ensure the environment is not cleared

Unmerged revisions

337. By Ken VanDine on 2017-04-04

Run dbus-test-runner with --keep-env to ensure the environment is not cleared

336. By Ken VanDine on 2017-04-04

Set $HOME when running tests

=== modified file 'debian/rules'
--- debian/rules	2017-03-15 13:30:11 +0000
+++ debian/rules	2017-04-04 19:04:35 +0000
@@ -14,7 +14,8 @@
 	dh $@ --with translations --fail-missing -- -B build
 
 override_dh_auto_test:
-	make -C build/tests/acceptance-tests test
+	TEST_HOME=`mktemp -d`
+	HOME=$(TEST_HOME) make -C build/tests/acceptance-tests test
 
 override_dh_auto_install:
 	dh_auto_install

=== modified file 'tests/acceptance-tests/CMakeLists.txt'
--- tests/acceptance-tests/CMakeLists.txt	2017-03-15 12:54:42 +0000
+++ tests/acceptance-tests/CMakeLists.txt	2017-04-04 19:04:35 +0000
@@ -64,7 +64,7 @@
     ${TEST_LIBS}
   )
 
-  add_test(NAME ${test} COMMAND dbus-test-runner --task ${CMAKE_CURRENT_BINARY_DIR}/${test})
+  add_test(NAME ${test} COMMAND dbus-test-runner --keep-env --task ${CMAKE_CURRENT_BINARY_DIR}/${test})
 
   SET_TESTS_PROPERTIES(${test}
     PROPERTIES ENVIRONMENT "CONTENT_HUB_TESTING=1")