From 95914c3484a4c9dc47cab02da6698cb8d0e0ea1c Mon Sep 17 00:00:00 2001
From: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: Mon, 6 Jun 2011 16:03:06 +0100
Subject: [PATCH 4/5] Run bus tests from test/

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=34405
---
 bus/Makefile.am  |   22 ++++++++++++----------
 test/Makefile.am |    5 +++++
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/bus/Makefile.am b/bus/Makefile.am
index dfb3bc3..ac41bf2 100644
--- a/bus/Makefile.am
+++ b/bus/Makefile.am
@@ -146,23 +146,25 @@ bus_test_launch_helper_CPPFLAGS= -DDBUS_STATIC_BUILD	\
 	-DACTIVATION_LAUNCHER_TEST	\
 	-DACTIVATION_LAUNCHER_DO_OOM
 
-## we use noinst_PROGRAMS not check_PROGRAMS so that we build
-## even when not doing "make check"
-noinst_PROGRAMS = $(TESTS)
+noinst_PROGRAMS =
 dbus_daemon_exec_PROGRAMS = dbus-daemon
 libexec_PROGRAMS = dbus-daemon-launch-helper
 
-## note that TESTS has special meaning (stuff to use in make check)
-## so if adding tests not to be run in make check, don't add them to
-## TESTS
-TESTS =
+## Note that TESTS has special meaning (stuff to use in make check).
+## We don't actually want to run any of these tests until test/ has been
+## compiled, so we don't put them in TESTS here; we run them in test/
+## instead.
 
 if DBUS_BUILD_TESTS
-TESTS_ENVIRONMENT=DBUS_TEST_DATA=$(top_builddir)/test/data DBUS_TEST_HOMEDIR=$(top_builddir)/dbus DBUS_FATAL_WARNINGS=1 DBUS_BLOCK_ON_ABORT=1
-TESTS += bus-test bus-test-system
+## we use noinst_PROGRAMS not check_PROGRAMS so that we build
+## even when not doing "make check"
+
+# run as a test by test/Makefile.am
+noinst_PROGRAMS += bus-test bus-test-system
 
 if DBUS_UNIX
-TESTS += bus-test-launch-helper
+# run as a test by test/Makefile.am
+noinst_PROGRAMS += bus-test-launch-helper
 # this is used by the tests but is not,itself, a test
 noinst_PROGRAMS += dbus-daemon-launch-helper-test
 endif DBUS_UNIX
diff --git a/test/Makefile.am b/test/Makefile.am
index 9a12e0b..a5ac540 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -34,6 +34,9 @@ TESTS = \
 ## order, but we don't want to run them til we arrive in this directory,
 ## since they depend on stuff from this directory
 TESTS += \
+	../bus/bus-test \
+	../bus/bus-test-system \
+	../bus/bus-test-launch-helper \
 	../dbus/dbus-test \
 	$(NULL)
 
@@ -45,6 +48,8 @@ TESTS=
 endif !DBUS_BUILD_TESTS
 
 TESTS_ENVIRONMENT = \
+	DBUS_BLOCK_ON_ABORT=1 \
+	DBUS_FATAL_WARNINGS=1 \
 	DBUS_TEST_DATA=$(top_builddir)/test/data \
 	DBUS_TEST_HOMEDIR=$(top_builddir)/dbus \
 	$(NULL)
-- 
1.7.5.3