@matheusbsbs / matheus-bagliano / commits / 11d54b8b9a
Fork 0

test: reword the final message of tests with known breakages

When we have known breakages, we still said "passed all N test(s)", which was a bit funny. This rewords it to read "passed all remaining N test(s)" in such a case. Signed-off-by: Junio C Hamano <gitster@pobox.com>

Junio C Hamano committed Feb 3, 2008 at 00:23 UTC 11d54b8b9a78d7236916883c09e165fdc3768f00
1 file changed +5 -2
t/test-lib.sh
+5 -2
@@ -295,6 +295,9 @@ test_done () {
295 if test "$test_broken" != 0
296 then
297 say_color error "still have $test_broken known breakage(s)"
298 + msg="remaining $(($test_count-$test_broken)) test(s)"
299 + else
300 + msg="$test_count test(s)"
301 fi
302 case "$test_failure" in
303 0)
@@ -306,11 +309,11 @@ test_done () {
309 # The Makefile provided will clean this test area so
310 # we will leave things as they are.
311
309 - say_color pass "passed all $test_count test(s)"
312 + say_color pass "passed all $msg"
313 exit 0 ;;
314
315 *)
313 - say_color error "failed $test_failure among $test_count test(s)"
316 + say_color error "failed $test_failure among $msg"
317 exit 1 ;;
318
319 esac