[gentoo-portage-dev] [PATCH] Output uniform status message before and after src_test()

[Michał Górny]

Before, the status message indicating src_test() run was output only if
the default EAPI implementation was used. If an ebuild overrode test
phase, the test phase output lacked delimitation. Instead, output
uniform status messages before and after running src_test() alike for
other phase functions.
---
 bin/phase-functions.sh | 4 ++++
 bin/phase-helpers.sh   | 4 ----
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/bin/phase-functions.sh b/bin/phase-functions.sh
index f39a024..5795aac 100644
--- a/bin/phase-functions.sh
+++ b/bin/phase-functions.sh
@@ -489,7 +489,11 @@ __dyn_test() {
 		local save_sp=${SANDBOX_PREDICT}
 		addpredict /
 		__ebuild_phase pre_src_test
+
+		__vecho ">>> Test phase: ${CATEGORY}/${PF}"
 		__ebuild_phase src_test
+		__vecho ">>> Completed testing ${CATEGORY}/${PF}"
+
 		>> "$PORTAGE_BUILDDIR/.tested" || \
 			die "Failed to create $PORTAGE_BUILDDIR/.tested"
 		__ebuild_phase post_src_test
diff --git a/bin/phase-helpers.sh b/bin/phase-helpers.sh
index 47bd843..1be550d 100644
--- a/bin/phase-helpers.sh
+++ b/bin/phase-helpers.sh
@@ -667,15 +667,11 @@ __eapi0_src_test() {
 		internal_opts+=" -j1"
 	fi
 	if $emake_cmd ${internal_opts} check -n &> /dev/null; then
-		__vecho ">>> Test phase [check]: ${CATEGORY}/${PF}"
 		$emake_cmd ${internal_opts} check || \
 			die "Make check failed. See above for details."
 	elif $emake_cmd ${internal_opts} test -n &> /dev/null; then
-		__vecho ">>> Test phase [test]: ${CATEGORY}/${PF}"
 		$emake_cmd ${internal_opts} test || \
 			die "Make test failed. See above for details."
-	else
-		__vecho ">>> Test phase [none]: ${CATEGORY}/${PF}"
 	fi
 }
 
-- 
2.0.4