Make dcop tests run without X

Signed-off-by: Alexander Golubev <fatzer2@gmail.com>
(cherry picked from commit 7f6c0b1c7d39628a7c1f3cc4271f92054fe151eb)

3 files changed, 24 insertions, 10 deletions

diff --git a/dcop/tests/driver.cpp b/dcop/tests/driver.cpp
index 5ff057649..cb9ec11e9 100644
--- a/dcop/tests/driver.cpp
+++ b/dcop/tests/driver.cpp
@@ -53,8 +53,8 @@ int main(int argc, char** argv)
 	if ( argc < 2 ) { qWarning("Usage: driver <appid>"); return 1; }
 	const char* appname = strdup( argv[ 1 ] );
 	argv[ 1 ] = 0; // sue me
-	KCmdLineArgs::init( argc, argv, "TestAppDriver", "Tests the dcop familly of tools + libraries", "1.0" ); // FIXME
-	KApplication app;
+	KCmdLineArgs::init( argc, argv, argv[1], "TestAppDriver", "Tests the dcop familly of tools + libraries", "1.0" ); // FIXME
+	KApplication app (/*stylesEnabled=*/ false, /*GUIEnabled=*/ false);
 	app.dcopClient()->attach(  );
 	app.dcopClient()->registerAs( "TestAppDriver" );
 	Driver * object = new Driver( appname );
diff --git a/dcop/tests/run-tests.sh b/dcop/tests/run-tests.sh
index b088905b0..1645b0696 100644
--- a/dcop/tests/run-tests.sh
+++ b/dcop/tests/run-tests.sh
@@ -5,30 +5,44 @@ clean_up() {
 
 clean_up
 
+../dcopserver --nofork &
+DCOP_SERVER_PID=$!
+
+die() {
+    kill $DCOP_SERVER_PID
+    echo "$1"
+    exit 1;
+}
+
 echo '* Running batch mode'
-./dcop_test --batch >batch.stdout
+./dcop_test --batch >batch.stdout || die "Failed to run dcop_test"
 
 echo -n '* Starting test app '
 ./dcop_test >shell.stdout &
+DCOP_TEST_PID=$!
 
-while ! dcop | grep -q TestApp; do echo -n '.'; sleep 2; done
+while ! ../client/dcop | grep -q "TestApp-$DCOP_TEST_PID"; do
+    echo -n '.'
+    sleep 2
+    kill -0 "$DCOP_TEST_PID" || die "dcop_test died unexpectadly"
+done
 
 echo ' started'
 
 echo '* Running driver mode'
-./driver `dcop 'TestApp-*'` >driver.stdout
+./driver "TestApp-$DCOP_TEST_PID" >driver.stdout || die "Failed to start driver"
 
 echo '* Running shell mode'
 source ./shell.generated >shell.returns
 
 echo -n '* Comparing ... '
 
-compare() 
+compare()
 {
 if ! diff -q --strip-trailing-cr $1 $2; then
 	echo "FAILED:"
 	diff -u $1 $2
-	exit 1;
+	die "$1 and $2 are different";
 fi
 }
 
@@ -39,6 +53,6 @@ compare batch.returns driver.returns
 
 clean_up
 
+kill $DCOP_SERVER_PID
 echo "Passed"
 exit 0;
-
diff --git a/dcop/tests/test.cpp b/dcop/tests/test.cpp
index 7f5114071..a224dc1fc 100644
--- a/dcop/tests/test.cpp
+++ b/dcop/tests/test.cpp
@@ -31,8 +31,8 @@ int main(int argc, char** argv)
 		batch();
 		return 0;
 	}
-	KCmdLineArgs::init( argc, argv, "TestApp", "Tests the dcop familly of tools + libraries", "1.0" ); // FIXME
-	KApplication app;
+	KCmdLineArgs::init( argc, argv, argv[0], "TestApp", "Tests the dcop familly of tools + libraries", "1.0" ); // FIXME
+	KApplication app (/*stylesEnabled=*/ false, /*GUIEnabled=*/ false);
 	if(!app.dcopClient()->attach(  ))
 		return 1;