Git development
 help / color / mirror / Atom feed
* [GSoC PATCH] t0410: use test_path_exists instead of test -e
@ 2026-10-29 14:15 Tilak Kumar
  2026-10-29 18:51 ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: Tilak Kumar @ 2026-10-29 14:15 UTC (permalink / raw)
  To: git; +Cc: Tilak Kumar

Use the test_path_exists helper so that a missing
promisor packfile is reported with a clear message
instead of failing silently.

Signed-off-by: Tilak Kumar <raaztilak07@gmail.com>
---
 t/t0410-partial-clone.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/t0410-partial-clone.sh b/t/t0410-partial-clone.sh
index a54685e3c7..c3c5fd6e33 100755
--- a/t/t0410-partial-clone.sh
+++ b/t/t0410-partial-clone.sh
@@ -627,7 +627,7 @@ test_expect_success 'gc stops traversal when a missing but promised object is re
 	git -C repo gc &&
 
 	# Ensure that the promisor packfile still exists, and remove it
-	test -e repo/.git/objects/pack/pack-$HASH.pack &&
+	test_path_exists repo/.git/objects/pack/pack-$HASH.pack &&
 	rm repo/.git/objects/pack/pack-$HASH.* &&
 
 	# Ensure that the single other pack contains the commit, but not the tree
-- 
2.55.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [GSoC PATCH] t0410: use test_path_exists instead of test -e
  2026-10-29 14:15 [GSoC PATCH] t0410: use test_path_exists instead of test -e Tilak Kumar
@ 2026-10-29 18:51 ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2026-10-29 18:51 UTC (permalink / raw)
  To: Tilak Kumar; +Cc: git

Tilak Kumar <raaztilak07@gmail.com> writes:

> Use the test_path_exists helper so that a missing
> promisor packfile is reported with a clear message
> instead of failing silently.
>
> Signed-off-by: Tilak Kumar <raaztilak07@gmail.com>
> ---
>  t/t0410-partial-clone.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Makes sense.  Will queue.

Thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-10-29 18:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-29 14:15 [GSoC PATCH] t0410: use test_path_exists instead of test -e Tilak Kumar
2026-10-29 18:51 ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox