diff options
author | Chandra Pratap <chandrapratap3519@gmail.com> | 2024-01-10 12:54:17 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-01-10 17:24:56 (GMT) |
commit | 1260914190ad737029de6f4f61851180a4ccce56 (patch) | |
tree | 71aa981819c1f9df9fe0bde90031750d584a2595 /t/t4129-apply-samemode.sh | |
parent | a54a84b333adbecf7bc4483c0e36ed5878cac17b (diff) | |
download | git-1260914190ad737029de6f4f61851180a4ccce56.zip git-1260914190ad737029de6f4f61851180a4ccce56.tar.gz git-1260914190ad737029de6f4f61851180a4ccce56.tar.bz2 |
t4129: prevent loss of exit code due to the use of pipes
Piping the output of git commands like git-ls-files to another
command (grep in this case) hides the exit code returned by
these commands. Prevent this by storing the output of git-ls-files
to a temporary file and then "grep-ping" from that file. Replace
grep with test_grep as the latter is more verbose when it fails.
Signed-off-by: Chandra Pratap <chandrapratap3519@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4129-apply-samemode.sh')
-rwxr-xr-x | t/t4129-apply-samemode.sh | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/t/t4129-apply-samemode.sh b/t/t4129-apply-samemode.sh index e7a7295..2775bfa 100755 --- a/t/t4129-apply-samemode.sh +++ b/t/t4129-apply-samemode.sh @@ -41,7 +41,8 @@ test_expect_success FILEMODE 'same mode (index only)' ' chmod +x file && git add file && git apply --cached patch-0.txt && - git ls-files -s file | grep "^100755" + git ls-files -s file >ls-files-output && + test_grep "^100755" ls-files-output ' test_expect_success FILEMODE 'mode update (no index)' ' @@ -60,7 +61,8 @@ test_expect_success FILEMODE 'mode update (with index)' ' test_expect_success FILEMODE 'mode update (index only)' ' git reset --hard && git apply --cached patch-1.txt && - git ls-files -s file | grep "^100755" + git ls-files -s file >ls-files-output && + test_grep "^100755" ls-files-output ' test_expect_success FILEMODE 'empty mode is rejected' ' |