diff options
author | Rasmus Jonsson <wasmus@zom.bi> | 2020-02-23 00:50:49 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-02-24 17:01:24 (GMT) |
commit | a51d9e8f07652ded830cb50187be74c1709816d8 (patch) | |
tree | 366fb019399ad062f210a90a91cfc053953aef05 | |
parent | d0654dc308b0ba76dd8ed7bbb33c8d8f7aacd783 (diff) | |
download | git-a51d9e8f07652ded830cb50187be74c1709816d8.zip git-a51d9e8f07652ded830cb50187be74c1709816d8.tar.gz git-a51d9e8f07652ded830cb50187be74c1709816d8.tar.bz2 |
t1050: replace test -f with test_path_is_file
Use test_path_is_file() instead of 'test -f' for better debugging
information.
Signed-off-by: Rasmus Jonsson <wasmus@zom.bi>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rwxr-xr-x | t/t1050-large.sh | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/t/t1050-large.sh b/t/t1050-large.sh index d3b2adb..184b479 100755 --- a/t/t1050-large.sh +++ b/t/t1050-large.sh @@ -53,7 +53,8 @@ test_expect_success 'add a large file or two' ' for p in .git/objects/pack/pack-*.pack do count=$(( $count + 1 )) - if test -f "$p" && idx=${p%.pack}.idx && test -f "$idx" + if test_path_is_file "$p" && + idx=${p%.pack}.idx && test_path_is_file "$idx" then continue fi @@ -65,7 +66,7 @@ test_expect_success 'add a large file or two' ' test $cnt = 2 && for l in .git/objects/??/?????????????????????????????????????? do - test -f "$l" || continue + test_path_is_file "$l" || continue bad=t done && test -z "$bad" && @@ -76,7 +77,8 @@ test_expect_success 'add a large file or two' ' for p in .git/objects/pack/pack-*.pack do count=$(( $count + 1 )) - if test -f "$p" && idx=${p%.pack}.idx && test -f "$idx" + if test_path_is_file "$p" && + idx=${p%.pack}.idx && test_path_is_file "$idx" then continue fi @@ -111,7 +113,7 @@ test_expect_success 'packsize limit' ' count=0 && for pi in .git/objects/pack/pack-*.idx do - test -f "$pi" && count=$(( $count + 1 )) + test_path_is_file "$pi" && count=$(( $count + 1 )) done && test $count = 2 && |