[10/12] t5520: don't put git in upstream of pipe
diff mbox series

Message ID 148d05cacda5fcfe686e787a6ed180c48239a21a.1571354136.git.liu.denton@gmail.com
State New
Headers show
  • t5520: various test cleanup
Related show

Commit Message

Denton Liu Oct. 17, 2019, 11:17 p.m. UTC
Before, if the invocation of git failed, it would be masked by the pipe
since only the return code of the last element of a pipe is used.
Rewrite the test to put the Git command on its own line so its return
code is not masked.

Signed-off-by: Denton Liu <liu.denton@gmail.com>
 t/t5520-pull.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff mbox series

diff --git a/t/t5520-pull.sh b/t/t5520-pull.sh
index 0eebab7d86..e611d414c8 100755
--- a/t/t5520-pull.sh
+++ b/t/t5520-pull.sh
@@ -668,7 +668,8 @@  test_expect_success 'pull --rebase fails on corrupt HEAD' '
 		cd corrupt &&
 		test_commit one &&
-		obj=$(git rev-parse --verify HEAD | sed "s#^..#&/#") &&
+		git rev-parse --verify HEAD >head &&
+		obj=$(sed "s#^..#&/#" head) &&
 		rm -f .git/objects/$obj &&
 		test_must_fail git pull --rebase