]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t7700: s/test -f/test_path_is_file/
authorDenton Liu <liu.denton@gmail.com>
Wed, 27 Nov 2019 19:53:52 +0000 (11:53 -0800)
committerJunio C Hamano <gitster@pobox.com>
Fri, 29 Nov 2019 21:20:15 +0000 (13:20 -0800)
Since we have debugging-friendly alternatives to `test -f`, replace
instances of `test -f` with `test_path_is_file` so that if a command
ever fails, we get better debugging information.

Signed-off-by: Denton Liu <liu.denton@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t7700-repack.sh

index a96e876c4e6d6d66586f3dfee1f77e15cefc46df..1d14ddcbdb991d7f30bd7abb4907a0bab37b465a 100755 (executable)
@@ -106,7 +106,7 @@ test_expect_success 'packed obs in alt ODB are repacked even when local repo is
        mv .git/objects/pack/* alt_objects/pack &&
        git repack -a &&
        myidx=$(ls -1 .git/objects/pack/*.idx) &&
-       test -f "$myidx" &&
+       test_path_is_file "$myidx" &&
        for p in alt_objects/pack/*.idx
        do
                git verify-pack -v $p | sed -n -e "/^[0-9a-f]\{40\}/p"
@@ -129,7 +129,7 @@ test_expect_success 'packed obs in alt ODB are repacked when local repo has pack
        git repack &&
        git repack -a -d &&
        myidx=$(ls -1 .git/objects/pack/*.idx) &&
-       test -f "$myidx" &&
+       test_path_is_file "$myidx" &&
        for p in alt_objects/pack/*.idx
        do
                git verify-pack -v $p | sed -n -e "/^[0-9a-f]\{40\}/p"
@@ -148,7 +148,7 @@ test_expect_success 'packed obs in alternate ODB kept pack are repacked' '
        for p in alt_objects/pack/*.pack
        do
                base_name=$(basename $p .pack) &&
-               if test -f alt_objects/pack/$base_name.keep
+               if test_path_is_file alt_objects/pack/$base_name.keep
                then
                        rm alt_objects/pack/$base_name.keep
                else
@@ -157,7 +157,7 @@ test_expect_success 'packed obs in alternate ODB kept pack are repacked' '
        done &&
        git repack -a -d &&
        myidx=$(ls -1 .git/objects/pack/*.idx) &&
-       test -f "$myidx" &&
+       test_path_is_file "$myidx" &&
        for p in alt_objects/pack/*.idx
        do
                git verify-pack -v $p | sed -n -e "/^[0-9a-f]\{40\}/p"