threads / patch / 66613

patcht2104: avoid suppressing git's exit code

Subject: [Outreachy PATCH] t2104: avoid suppressing git's exit code

## tl;dr

4 messages between Oct 21, 2026 and Oct 23, 2026. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Cynthia Uche· Oct 21, 2026, 03:37 UTC · lore

The exit code of the upstream command in a pipeline is lost, so a crash in 'git ls-files' would go unnoticed. Write the output to a file and compare that instead.

Signed-off-by: Cynthia Uche <herroyalpriestess@gmail.com>
---
 t/t2104-update-index-skip-worktree.sh | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)
Show changes to t/t2104-update-index-skip-worktree.sh +8 −4
diff --git a/t/t2104-update-index-skip-worktree.sh b/t/t2104-update-index-skip-worktree.sh
index 7a0778ed98..68a4339e24 100755
--- a/t/t2104-update-index-skip-worktree.sh
+++ b/t/t2104-update-index-skip-worktree.sh
@@ -34,7 +34,8 @@ test_expect_success 'setup' '
 	mkdir sub &&
 	touch ./1 ./2 sub/1 sub/2 &&
 	git add 1 2 sub/1 sub/2 &&
-	git ls-files -t | test_cmp expect.full -
+	git ls-files -t >actual &&
+	test_cmp expect.full actual
 '
 
 test_expect_success 'index is at version 2' '
@@ -43,7 +44,8 @@ test_expect_success 'index is at version 2' '
 
 test_expect_success 'update-index --skip-worktree' '
 	git update-index --skip-worktree 1 sub/1 &&
-	git ls-files -t | test_cmp expect.skip -
+	git ls-files -t >actual &&
+	test_cmp expect.skip actual
 '
 
 test_expect_success 'index is at version 3 after having some skip-worktree entries' '
@@ -51,12 +53,14 @@ test_expect_success 'index is at version 3 after having some skip-worktree entri
 '
 
 test_expect_success 'ls-files -t' '
-	git ls-files -t | test_cmp expect.skip -
+	git ls-files -t >actual &&
+	test_cmp expect.skip actual
 '
 
 test_expect_success 'update-index --no-skip-worktree' '
 	git update-index --no-skip-worktree 1 sub/1 &&
-	git ls-files -t | test_cmp expect.full -
+	git ls-files -t >actual &&
+	test_cmp expect.full actual
 '
 
 test_expect_success 'index version is back to 2 when there is no skip-worktree entry' '
-- 
2.56.GIT
Patrick Steinhardt· Oct 21, 2026, 08:00 UTC · re: Cynthia Uche · lore

Re: [Outreachy PATCH] t2104: avoid suppressing git's exit code

On Wed, Oct 21, 2026 at 04:37:23AM +0100, Cynthia Uche wrote:
> The exit code of the upstream command in a pipeline is lost, so a
> crash in 'git ls-files' would go unnoticed. Write the output to a
> file and compare that instead.
Makes sense, and the patch looks obviously good to me, too. Thanks!
Patrick
Junio C Hamano· Oct 23, 2026, 17:56 UTC · re: Priestess · lore

Re: [Outreachy PATCH] t2104: avoid suppressing git's exit code

Priestess <herroyalpriestess@gmail.com> writes:
>> Makes sense, and the patch looks obviously good to me, too. Thanks!
>
> Thanks for the review, Patrick.
Thanks, both.  Queued.

← back to recent threads