t5613: use test_must_fail

Besides being our normal style, this correctly checks for an
error exit() versus signal death.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/t/t5613-info-alternate.sh b/t/t5613-info-alternate.sh
index 4548fb0ab93c180c653454b911fdadd67d2c7969..65074ddc1301b1ce9cf64d77b219b195586428d2 100755 (executable)
--- a/t/t5613-info-alternate.sh
+++ b/t/t5613-info-alternate.sh
@@ -46,10 +46,9 @@ git clone -l -s F G &&
 git clone --bare -l -s G H'
 
 test_expect_success 'invalidity of deepest repository' \
 git clone --bare -l -s G H'
 
 test_expect_success 'invalidity of deepest repository' \

-'cd H && {
-       git fsck
-       test $? -ne 0
-}'
+'cd H &&
+test_must_fail git fsck
+'

 
 cd "$base_dir"
 
 
 cd "$base_dir"
 


@@ -75,7 +74,8 @@ cd "$base_dir"
 test_expect_success 'that info/alternates is necessary' \
 'cd C &&
 rm -f .git/objects/info/alternates &&
 test_expect_success 'that info/alternates is necessary' \
 'cd C &&
 rm -f .git/objects/info/alternates &&

-! (git fsck)'
+test_must_fail git fsck
+'

 
 cd "$base_dir"
 
 
 cd "$base_dir"
 


@@ -89,7 +89,7 @@ cd "$base_dir"
 test_expect_success \
     'that relative alternate is only possible for current dir' '
     cd D &&
 test_expect_success \
     'that relative alternate is only possible for current dir' '
     cd D &&

-    ! (git fsck)
+    test_must_fail git fsck

 '
 
 cd "$base_dir"
 '
 
 cd "$base_dir"