[tracker-miners/sam/ci-image-url: 1/2] tests: Use dbus-run-session instead of dbus-launch



commit acd0f55a492418e5ebc1db3e9972799e97b093ad
Author: Sam Thursfield <sam afuera me uk>
Date:   Sat Jan 5 18:24:42 2019 +0100

    tests: Use dbus-run-session instead of dbus-launch
    
    See https://gitlab.gnome.org/GNOME/tracker-miners/issues/44

 .gitlab-ci.yml                        |  2 +-
 tests/functional-tests/test-runner.sh | 10 +---------
 2 files changed, 2 insertions(+), 10 deletions(-)
---
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index e3dc0e842..906db941a 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -27,4 +27,4 @@ test-fedora-latest:
       # screenful of junk each time unless we strip these.
       unset $(env|grep -o '^CI_[^=]*')
 
-      su tracker -c 'cd build; LANG=en_US.UTF8 dbus-launch meson test --suite tracker-miners 
--print-errorlogs'
+      su tracker -c 'cd build; LANG=en_US.UTF8 meson test --suite tracker-miners --print-errorlogs'
diff --git a/tests/functional-tests/test-runner.sh b/tests/functional-tests/test-runner.sh
index 29eed46b3..12fdb0538 100755
--- a/tests/functional-tests/test-runner.sh
+++ b/tests/functional-tests/test-runner.sh
@@ -6,8 +6,6 @@ set -e
 
 SCRIPT=$1
 
-DBUS_SESSION_BUS_PID=
-
 export TEMP_DIR=`mktemp --tmpdir -d tracker-test-XXXX`
 
 # We need to use the actual home directory for some tests because
@@ -18,13 +16,7 @@ export REAL_HOME=`echo ~`
 # in the filesystem
 HOME=$TEMP_DIR
 
-eval `dbus-launch --sh-syntax`
-
-trap "/bin/kill $DBUS_SESSION_BUS_PID; exit" INT ERR
-
 echo "Running $@"
-$@
-
-kill $DBUS_SESSION_BUS_PID
+dbus-run-session $@
 
 rm -R $TEMP_DIR


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]