testprogs/blackbox: make sure subunit.sh always terminates DETAILS with '\n]\n'

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

diff --git a/testprogs/blackbox/subunit.sh b/testprogs/blackbox/subunit.sh
index bcc5bd6a928b1911f1f9a61768785b62b5025ef6..45183575bddb3a2d8acf9866f3ef62631a969577 100755 (executable)
--- a/testprogs/blackbox/subunit.sh
+++ b/testprogs/blackbox/subunit.sh
@@ -50,7 +50,7 @@ subunit_fail_test () {
   timestamp
   printf 'failure: %s [\n' "$1"
   cat -
-  echo "]"
+  printf '\n]\n'
 }
 
 
@@ -62,7 +62,7 @@ subunit_error_test () {
   timestamp
   printf 'error: %s [\n' "$1"
   cat -
-  echo "]"
+  printf '\n]\n'
 }
 
 subunit_skip_test () {
@@ -72,7 +72,7 @@ subunit_skip_test () {
   # makes it convenient to write in scripts (using <<END syntax.
   printf 'skip: %s [\n' "$1"
   cat -
-  echo "]"
+  printf '\n]\n'
 }
 
 testit () {