Add: recursive option on test command

changelog/recursive-test.dd

@@ -0,0 +1,3 @@
+Added dependency recursive test
+
+Dub now supports recursive test run. This feature can be activated by option `-r` or `--recursive`.

source/dub/commandline.d

@@ -966,6 +966,7 @@
 		bool m_combined = false;
 		bool m_parallel = false;
 		bool m_force = false;
+		bool m_recursive = false;
 	}
 
 	this()
@@ -1009,6 +1010,9 @@
 		args.getopt("f|force", &m_force, [
 			"Forces a recompilation even if the target is up to date"
 		]);
+		args.getopt("r|recursive", &m_recursive, [
+			"Runs test including dependent packages."
+		]);
 		bool coverage = false;
 		args.getopt("coverage", &coverage, [
 			"Enables code coverage statistics to be generated."
@@ -1024,6 +1028,15 @@
 		enforceUsage(free_args.length <= 1, "Expected one or zero arguments.");
 		if (free_args.length >= 1) package_name = free_args[0];
 
+		string[] tested_packages_cache;
+		return test(dub, package_name, app_args, tested_packages_cache);
+	}
+
+	int test(Dub dub, string package_name, string[] app_args, ref string[] tested_packages)
+	{
+		if (tested_packages.canFind(package_name)) return 0; // prevent duplicate runs
+		tested_packages ~= package_name;
+
 		setupPackage(dub, package_name, "unittest");
 
 		GeneratorSettings settings;
@@ -1041,6 +1054,13 @@
 		settings.runArgs = app_args;
 
 		dub.testProject(settings, m_buildConfig, NativePath(m_mainFile));
+		if (m_recursive) {
+			foreach (d; dub.project.dependencies.map!(d => d.name).array.sort) {
+				auto status = this.test(dub, d, app_args, tested_packages);
+				if (status > 0) return status;
+			}
+		}
+
 		return 0;
 	}
 }

test/test-recursive.sh

@@ -0,0 +1,25 @@
+#!/usr/bin/env bash
+
+. $(dirname "${BASH_SOURCE[0]}")/common.sh
+
+should_contain () {
+	for c in ${@:2}; do
+		if ! { echo $1; } | grep "$c"; then
+			die $LINENO "$c was not tested."
+			exit 1
+		fi
+	done
+}
+
+should_not_contain () {
+	for c in ${@:2}; do
+		if   { echo $1; } | grep "$c"; then
+			die $LINENO "Unexpected line '$c' was detected."
+			exit 1
+		fi
+	done
+}
+
+should_contain     "$($DUB test -r --root=$(dirname "${BASH_SOURCE[0]}")/test-recursive 2>&1)" "rootPackage" "libraryA" "libraryB" "libraryC"
+should_contain     "$($DUB test    --root=$(dirname "${BASH_SOURCE[0]}")/test-recursive 2>&1)" "rootPackage"
+should_not_contain "$($DUB test    --root=$(dirname "${BASH_SOURCE[0]}")/test-recursive 2>&1)" "libraryA" "libraryB" "libraryC"

test/test-recursive/dub.sdl

@@ -0,0 +1,4 @@
+name "test-recursive"
+dependency "library-a" path="./libraryA"
+dependency "library-b" path="./libraryB"
+dependency "library-c" path="./libraryC"

test/test-recursive/libraryA/dub.sdl

@@ -0,0 +1,3 @@
+name "library-a"
+dependency "library-b" path="../libraryB"
+dependency "library-c" path="../libraryC"

test/test-recursive/libraryA/source/app.d

@@ -0,0 +1,7 @@
+void main() {}
+
+unittest
+{
+	import std.stdio : writeln;
+	writeln("Testing libraryA...");
+}

test/test-recursive/libraryB/dub.sdl

@@ -0,0 +1,2 @@
+name "library-b"
+dependency "library-c" path="../libraryC"

test/test-recursive/libraryB/source/app.d

@@ -0,0 +1,7 @@
+void main() {}
+
+unittest
+{
+	import std.stdio : writeln;
+	writeln("Testing libraryB...");
+}

test/test-recursive/libraryC/dub.sdl

@@ -0,0 +1 @@
+name "library-c"

test/test-recursive/libraryC/source/app.d

@@ -0,0 +1,7 @@
+void main() {}
+
+unittest
+{
+	import std.stdio : writeln;
+	writeln("Testing libraryC...");
+}

test/test-recursive/source/app.d

@@ -0,0 +1,7 @@
+void main() {}
+
+unittest
+{
+	import std.stdio : writeln;
+	writeln("Testing rootPackage...");
+}