threads / patch / 1311

patchTest framework: prettyprint the failed command.

Subject: [PATCH] Test framework: prettyprint the failed command.

## tl;dr

One message between Jul 23, 2005 and Jul 23, 2005. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Junio C Hamano· Jul 23, 2005, 02:09 UTC · lore

The output from a failure case had the test description message and the first line of the actual test script concatenated on the same line, which was ugly. Correct the output routine a bit to make it more readable.

Signed-off-by: Junio C Hamano <junkio@cox.net>
---
 t/test-lib.sh |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
ba7b8ae0b3af6040c8af2f4e381bd4c30ab9cd24
Show changes to t/test-lib.sh +3 −1
diff --git a/t/test-lib.sh b/t/test-lib.sh
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -84,7 +84,9 @@ test_ok_ () {
 test_failure_ () {
 	test_count=$(expr "$test_count" + 1)
 	test_failure=$(expr "$test_failure" + 1);
-	say "FAIL $test_count: $@"
+	say "FAIL $test_count: $1"
+	shift
+	echo "$@" | sed -e 's/^/	/'
 	test "$immediate" == "" || exit 1
 }
 

← back to recent threads