# [GSoC PATCH] t3310: replace test -f/-d with test_path_is_file/test_path_is_dir 4 messages from 2026-02-27 to 2026-02-28. Participants: Francesco Paparatto, Junio C Hamano. Thread: https://gitlist.dev/t/65090 ## Francesco Paparatto, 2026-02-27 18:45 Subject: [GSoC PATCH] t3310: replace test -f/-d with test_path_is_file/test_path_is_dir Message-ID: <20260227184538.4718-1-francescopaparatto@gmail.com> URL: https://gitlist.dev/e/20260227184538.4718-1-francescopaparatto%40gmail.com ``` Replace old-style path assertions with modern helpers that provide clearer diagnostic messages on failure. That's because when test -f fails, the output gives no indication of what went wrong. These instances were found using: git grep "test -[efd]" t/ as suggested in the microproject ideas. Signed-off-by: Francesco Paparatto --- t/t3310-notes-merge-manual-resolve.sh | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t3310-notes-merge-manual-resolve.sh b/t/t3310-notes-merge-manual-resolve.sh index 597df5ebc0..f0054b0a39 100755 --- a/t/t3310-notes-merge-manual-resolve.sh +++ b/t/t3310-notes-merge-manual-resolve.sh @@ -260,7 +260,7 @@ test_expect_success 'change notes in z' ' ' test_expect_success 'cannot do merge w/conflicts when previous merge is unfinished' ' - test -d .git/NOTES_MERGE_WORKTREE && + test_path_is_dir .git/NOTES_MERGE_WORKTREE && test_must_fail git notes merge z >output 2>&1 && # Output should indicate what is wrong test_grep -q "\\.git/NOTES_MERGE_\\* exists" output @@ -320,7 +320,7 @@ w notes on 1st commit EOF test_expect_success 'can do merge without conflicts even if previous merge is unfinished (x => w)' ' - test -d .git/NOTES_MERGE_WORKTREE && + test_path_is_dir .git/NOTES_MERGE_WORKTREE && git notes merge x && verify_notes w && # Verify that other notes refs has not changed (x and y) @@ -564,10 +564,10 @@ EOF # NOTES_MERGE_* refs and .git/NOTES_MERGE_* state files must remain git rev-parse --verify NOTES_MERGE_PARTIAL && git rev-parse --verify NOTES_MERGE_REF && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha1 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha2 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha3 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha4 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha1 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha2 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha3 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha4 && # Refs are unchanged test "$(git rev-parse refs/notes/m)" = "$(git rev-parse refs/notes/w)" && test "$(git rev-parse refs/notes/y)" = "$(git rev-parse NOTES_MERGE_PARTIAL^1)" && -- 2.52.0 ``` ## Junio C Hamano, 2026-02-27 19:31 Subject: Re: [GSoC PATCH] t3310: replace test -f/-d with test_path_is_file/test_path_is_dir Message-ID: URL: https://gitlist.dev/e/xmqqv7fioueg.fsf%40gitster.g In-Reply-To: <20260227184538.4718-1-francescopaparatto@gmail.com> ``` Francesco Paparatto writes: > Replace old-style path assertions with modern helpers that provide > clearer diagnostic messages on failure. That's because when test -f fails, the > output gives no indication of what went wrong. > > These instances were found using: git grep "test -[efd]" t/ as suggested in the microproject ideas. An overly long line. Keep the display columns for a line in a patch e-mail below ~70 columns, so that after getting quoted "> " for a few times, the result will still fit in 80-column terminals. > test_expect_success 'cannot do merge w/conflicts when previous merge is unfinished' ' > - test -d .git/NOTES_MERGE_WORKTREE && > + test_path_is_dir .git/NOTES_MERGE_WORKTREE && OK. > @@ -320,7 +320,7 @@ w notes on 1st commit > EOF > > test_expect_success 'can do merge without conflicts even if previous merge is unfinished (x => w)' ' > - test -d .git/NOTES_MERGE_WORKTREE && > + test_path_is_dir .git/NOTES_MERGE_WORKTREE && OK. > @@ -564,10 +564,10 @@ EOF > # NOTES_MERGE_* refs and .git/NOTES_MERGE_* state files must remain > git rev-parse --verify NOTES_MERGE_PARTIAL && > git rev-parse --verify NOTES_MERGE_REF && > - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha1 && > - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha2 && > - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha3 && > - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha4 && > + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha1 && > + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha2 && > + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha3 && > + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha4 && OK. > # Refs are unchanged > test "$(git rev-parse refs/notes/m)" = "$(git rev-parse refs/notes/w)" && > test "$(git rev-parse refs/notes/y)" = "$(git rev-parse NOTES_MERGE_PARTIAL^1)" && Here is a pair of opportunities for others that want more microproject exercises to make sure we do not ignore failing "git" invocations. You do not want to address them, of course. You've had your share already ;-) Thanks. ``` ## Francesco Paparatto, 2026-02-28 00:52 Subject: Re: [GSoC PATCH] t3310: replace test -f/-d with test_path_is_file/test_path_is_dir Message-ID: URL: https://gitlist.dev/e/CAEaT9_-Uc11uW%3D-APEhqLUWeD0-cXoyi95wYrdazdnL1CKVt4g%40mail.gmail.com In-Reply-To: ``` Junio C Hamano writes: > An overly long line. Keep the display columns for a line in a patch > e-mail below ~70 columns, so that after getting quoted "> " for a > few times, the result will still fit in 80-column terminals. Thanks for the review. I'll resend with the commit message line wrapped. Francesco ``` ## Francesco Paparatto, 2026-02-28 00:59 Subject: [GSoC PATCH v2] t3310: replace test -f/-d with test_path_is_file/test_path_is_dir Message-ID: <20260228005939.9012-1-francescopaparatto@gmail.com> URL: https://gitlist.dev/e/20260228005939.9012-1-francescopaparatto%40gmail.com In-Reply-To: ``` Replace old-style path assertions with modern helpers that provide clearer diagnostic messages on failure. When test -f fails, the output gives no indication of what went wrong. These instances were found using: git grep "test -[efd]" t/ as suggested in the microproject ideas. Signed-off-by: Francesco Paparatto --- t/t3310-notes-merge-manual-resolve.sh | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t3310-notes-merge-manual-resolve.sh b/t/t3310-notes-merge-manual-resolve.sh index 597df5ebc0..f0054b0a39 100755 --- a/t/t3310-notes-merge-manual-resolve.sh +++ b/t/t3310-notes-merge-manual-resolve.sh @@ -260,7 +260,7 @@ test_expect_success 'change notes in z' ' ' test_expect_success 'cannot do merge w/conflicts when previous merge is unfinished' ' - test -d .git/NOTES_MERGE_WORKTREE && + test_path_is_dir .git/NOTES_MERGE_WORKTREE && test_must_fail git notes merge z >output 2>&1 && # Output should indicate what is wrong test_grep -q "\\.git/NOTES_MERGE_\\* exists" output @@ -320,7 +320,7 @@ w notes on 1st commit EOF test_expect_success 'can do merge without conflicts even if previous merge is unfinished (x => w)' ' - test -d .git/NOTES_MERGE_WORKTREE && + test_path_is_dir .git/NOTES_MERGE_WORKTREE && git notes merge x && verify_notes w && # Verify that other notes refs has not changed (x and y) @@ -564,10 +564,10 @@ EOF # NOTES_MERGE_* refs and .git/NOTES_MERGE_* state files must remain git rev-parse --verify NOTES_MERGE_PARTIAL && git rev-parse --verify NOTES_MERGE_REF && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha1 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha2 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha3 && - test -f .git/NOTES_MERGE_WORKTREE/$commit_sha4 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha1 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha2 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha3 && + test_path_is_file .git/NOTES_MERGE_WORKTREE/$commit_sha4 && # Refs are unchanged test "$(git rev-parse refs/notes/m)" = "$(git rev-parse refs/notes/w)" && test "$(git rev-parse refs/notes/y)" = "$(git rev-parse NOTES_MERGE_PARTIAL^1)" && -- 2.52.0 ```