Re: [PATCH 1/1] t1403: prefer test_path_exists helper function
From: Patrick Steinhardt <hidden>
Date: 2025-03-03 10:26:36
On Sat, Mar 01, 2025 at 04:28:38PM +0530, Mahendra Dani wrote:
quoted hunk ↗ jump to hunk
test -e does not provide a nice error message when we hit test failures, so use test_path_exists instead. Signed-off-by: Mahendra Dani <redacted> --- t/t1403-show-ref.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/t/t1403-show-ref.sh b/t/t1403-show-ref.sh index 9d698b3cc3..12f7b60024 100755 --- a/t/t1403-show-ref.sh +++ b/t/t1403-show-ref.sh@@ -196,7 +196,7 @@ test_expect_success 'show-ref --verify with dangling ref' ' remove_object() { file=$(sha1_file "$*") && - test -e "$file" && + test_path_exists "$file" && rm -f "$file" } &&
The refactoring is true to the original spirit of the preimage indeed. But we could also improve it even further if we verified that the path not only exists, but exists and is a file via `test_path_is_file()`. If we decide to do that we should also explain the change in the commit message. Thanks! Patrick