From ef802adfb4afe44ae2ef529275aadc2629d7170f Mon Sep 17 00:00:00 2001 From: Ratchanan Srirattanamet Date: Mon, 27 Apr 2020 17:58:04 +0700 Subject: integration-test: provide a temporary HOME directory to the test A couple of things in the integration-test rely on having a HOME directory. Provide a temporary HOME for the test and its child, which both prevents polluting the building user's HOME and allow the test to run where HOME=/nonexistent (i.e. autobuilder). Also remove the exception not to run the integration test inside the (Canonical's) Jenkins, as it's not relevant anymore. --- tests/integration/CMakeLists.txt | 13 ------------- tests/integration/main.cpp | 17 +++++++++++++++++ 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/tests/integration/CMakeLists.txt b/tests/integration/CMakeLists.txt index 74b9c23..e803a87 100644 --- a/tests/integration/CMakeLists.txt +++ b/tests/integration/CMakeLists.txt @@ -87,19 +87,6 @@ target_link_libraries( gmenuharness-shared ) -#### -## When building under jenkins, pulseuadio fails to start as there is no -## /run/user tree, HOME directory is missing, and similar issues. So here -## we check that we are the jenkins user, and avoid the integration tests, -## until such time in future when we can run these tests in some different -## manner to avoid needing to run pulseaudio in this way. -#### -execute_process( - COMMAND whoami - OUTPUT_VARIABLE TESTS_USER - OUTPUT_STRIP_TRAILING_WHITESPACE -) - add_test( integration-tests integration-tests diff --git a/tests/integration/main.cpp b/tests/integration/main.cpp index a29b3b6..f42c90a 100644 --- a/tests/integration/main.cpp +++ b/tests/integration/main.cpp @@ -20,6 +20,7 @@ //#include #include +#include #include #include @@ -44,6 +45,22 @@ int main(int argc, char **argv) qputenv("LANG", "C.UTF-8"); unsetenv("LC_ALL"); + /* + * A couple of things rely on having a HOME directory: + * - The indicator itself relies on having a writable gsettings/dconf + * database, and the test relies on the functionality it provides. + * - The test starts Pulseaudio, which requires both a runtime and a + * state directory, both of which has a failback to the HOME. + * Provide a temporary HOME for the test and its child, which both prevents + * polluting the building user's HOME and allow the test to run where + * HOME=/nonexistent (i.e. autobuilder). + */ + QTemporaryDir tmpHome; + if (!tmpHome.isValid()) + qFatal("Cannot create a temporary HOME for the test."); + + setenv("HOME", tmpHome.path().toLocal8Bit().constData(), true); + QCoreApplication application(argc, argv); DBusMock::registerMetaTypes(); DBusTypes::registerMetaTypes(); -- cgit v1.2.3