Unsetting the varibale for good can have unwanted effects for new tests added in the future It also meant we needed to hardcode the value for "user.signingkey". Move the unset into a subshell, get rid of the hardcoded "[email protected]", and switch the GPG variant to using test_config just like GPGSM.
Signed-off-by: Henning Schild <[email protected]> --- t/t5534-push-signed.sh | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/t/t5534-push-signed.sh b/t/t5534-push-signed.sh index 0cb88aa6f..f6d674156 100755 --- a/t/t5534-push-signed.sh +++ b/t/t5534-push-signed.sh @@ -194,10 +194,12 @@ test_expect_success GPG 'fail without key and heed user.signingkey' ' EOF - unset GIT_COMMITTER_EMAIL && - git config user.email [email protected] && - test_must_fail git push --signed dst noop ff +noff && - git config user.signingkey [email protected] && + test_config user.email [email protected] && + ( + unset GIT_COMMITTER_EMAIL && + test_must_fail git push --signed dst noop ff +noff + ) && + test_config user.signingkey $GIT_COMMITTER_EMAIL && git push --signed dst noop ff +noff && ( @@ -241,11 +243,14 @@ test_expect_success GPGSM 'fail without key and heed user.signingkey x509' ' E_O_F EOF - unset GIT_COMMITTER_EMAIL && + test_config user.email [email protected] && test_config user.signingkey "" && - test_must_fail git push --signed dst noop ff +noff && - test_config user.signingkey [email protected] && + ( + unset GIT_COMMITTER_EMAIL && + test_must_fail git push --signed dst noop ff +noff + ) && + test_config user.signingkey $GIT_COMMITTER_EMAIL && git push --signed dst noop ff +noff && ( -- 2.16.4
