summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Sunshine <sunshine@sunshineco.com>2018-07-02 00:23:50 (GMT)
committerJunio C Hamano <gitster@pobox.com>2018-07-03 19:38:05 (GMT)
commite7eb15faca767cd00acf722ce0382538b7add29a (patch)
treee68b64d61b46392b0fb3435f4161d953e45f5cf3
parentf1e123981177311d5038b033e5b1e89927e213db (diff)
downloadgit-e7eb15faca767cd00acf722ce0382538b7add29a.zip
git-e7eb15faca767cd00acf722ce0382538b7add29a.tar.gz
git-e7eb15faca767cd00acf722ce0382538b7add29a.tar.bz2
t7201: drop pointless "exit 0" at end of subshell
This test employs a for-loop inside a subshell and correctly aborts the loop and fails the test overall (via "exit 1") if any iteration of the for-loop fails. Otherwise, it exits the subshell with an explicit but entirely unnecessary "exit 0", presumably to indicate that all iterations of the loop succeeded. The &&-chain is broken between the for-loop and the "exit 0". Rather than fixing the &&-chain, just drop the pointless "exit 0". Signed-off-by: Eric Sunshine <sunshine@sunshineco.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rwxr-xr-xt/t7201-co.sh1
1 files changed, 0 insertions, 1 deletions
diff --git a/t/t7201-co.sh b/t/t7201-co.sh
index ab9da61..8d8a63a 100755
--- a/t/t7201-co.sh
+++ b/t/t7201-co.sh
@@ -673,7 +673,6 @@ test_expect_success 'custom merge driver with checkout -m' '
do
grep $t arm || exit 1
done
- exit 0
) &&
mv arm expect &&