[PATCHv2 4/6] t7510: exit for loop with test result
- From
Michael J Gruber <git@drmicha.warpmail.net>
- Date
- Jun 13, 2014, 10:42 UTC
- Message-ID
- <99893263f1819646a3a324b2fff0bcd0d56f3818.1402655839.git.git@drmicha.warpmail.net>
- In-Reply-To
- <cover.1402063795.git.git@drmicha.warpmail.net>
When t7510 was introduced, the author made sure that a for loop in a subshell would return with the appropriate error code.
Make sure this is true also the for the first line in each loop, which was missed.
Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> --- t/t7510-signed-commit.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/t/t7510-signed-commit.sh b/t/t7510-signed-commit.sh index 5ddac1a..a5ba48e 100755 --- a/t/t7510-signed-commit.sh +++ b/t/t7510-signed-commit.sh @@ -49,7 +49,7 @@ test_expect_success GPG 'show signatures' ' ( for commit in initial second merge fourth-signed fifth-signed sixth-signed master do - git show --pretty=short --show-signature $commit >actual && + git show --pretty=short --show-signature $commit >actual || exit 1 grep "Good signature from" actual || exit 1 ! grep "BAD signature from" actual || exit 1 echo $commit OK @@ -58,7 +58,7 @@ test_expect_success GPG 'show signatures' ' ( for commit in merge^2 fourth-unsigned sixth-unsigned seventh-unsigned do - git show --pretty=short --show-signature $commit >actual && + git show --pretty=short --show-signature $commit >actual || exit 1 grep "Good signature from" actual && exit 1 ! grep "BAD signature from" actual || exit 1 echo $commit OK
-- 2.0.0.426.g37dbf84