t5613: use test_must_fail
authorJeff King <peff@peff.net>
Mon, 3 Oct 2016 20:34:01 +0000 (16:34 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 3 Oct 2016 21:25:43 +0000 (14:25 -0700)
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>
t/t5613-info-alternate.sh
index 4548fb0ab93c180c653454b911fdadd67d2c7969..65074ddc1301b1ce9cf64d77b219b195586428d2 100755 (executable)
@@ -46,10 +46,9 @@ git clone -l -s F G &&
 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"
 
@@ -75,7 +74,8 @@ cd "$base_dir"
 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"
 
@@ -89,7 +89,7 @@ cd "$base_dir"
 test_expect_success \
     'that relative alternate is only possible for current dir' '
     cd D &&
-    ! (git fsck)
+    test_must_fail git fsck
 '
 
 cd "$base_dir"