t1301: use modern test_* helpers

This shortens the code and fixes some &&-chaining.

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

diff --git a/t/t1301-shared-repo.sh b/t/t1301-shared-repo.sh
index 7eecfb836a05cac56b8eca1ac56f67e8b22b1fbd..ac108754088d34758a75f14943516c4a701f0ca8 100755 (executable)
--- a/t/t1301-shared-repo.sh
+++ b/t/t1301-shared-repo.sh
@@ -12,12 +12,11 @@ setfacl -k . 2>/dev/null
 
 # User must have read permissions to the repo -> failure on --shared=0400
 test_expect_success 'shared = 0400 (faulty permission u-w)' '
 
 # User must have read permissions to the repo -> failure on --shared=0400
 test_expect_success 'shared = 0400 (faulty permission u-w)' '

+       test_when_finished "rm -rf sub" &&

        mkdir sub && (
        mkdir sub && (

-               cd sub && git init --shared=0400
+               cd sub &&
+               test_must_fail git init --shared=0400

        )
        )

-       ret="$?"
-       rm -rf sub
-       test $ret != "0"

 '
 
 modebits () {
 '
 
 modebits () {


@@ -33,7 +32,7 @@ do
                        git init --shared=1 &&
                        test 1 = "$(git config core.sharedrepository)"
                ) &&
                        git init --shared=1 &&
                        test 1 = "$(git config core.sharedrepository)"
                ) &&

-               actual=$(ls -l sub/.git/HEAD)
+               actual=$(ls -l sub/.git/HEAD) &&

                case "$actual" in
                -rw-rw-r--*)
                        : happy
                case "$actual" in
                -rw-rw-r--*)
                        : happy


@@ -90,10 +89,8 @@ do
                rm -f .git/info/refs &&
                git update-server-info &&
                actual="$(modebits .git/info/refs)" &&
                rm -f .git/info/refs &&
                git update-server-info &&
                actual="$(modebits .git/info/refs)" &&

-               test "x$actual" = "x-$y" || {
-                       ls -lt .git/info
-                       false
-               }
+               verbose test "x$actual" = "x-$y"
+

        '
 
        umask 077 &&
        '
 
        umask 077 &&


@@ -102,10 +99,7 @@ do
                rm -f .git/info/refs &&
                git update-server-info &&
                actual="$(modebits .git/info/refs)" &&
                rm -f .git/info/refs &&
                git update-server-info &&
                actual="$(modebits .git/info/refs)" &&

-               test "x$actual" = "x-$x" || {
-                       ls -lt .git/info
-                       false
-               }
+               verbose test "x$actual" = "x-$x"

 
        '
 
 
        '