Re: [PATCH] test the commit.gpgsign config option
Junio C Hamano writes: > If any of the above fail, the next test will run with an unknown > random value in commit.gpgsign depending on where the sequence > failed. Use one test_when_finished with test_unconfig at the very > beginning, perhaps. In other words, I'll squash this in. diff --git a/t/t7510-signed-commit.sh b/t/t7510-signed-commit.sh index 537bfba..5ddac1a 100755 --- a/t/t7510-signed-commit.sh +++ b/t/t7510-signed-commit.sh @@ -5,6 +5,8 @@ test_description='signed commit tests' . "$TEST_DIRECTORY/lib-gpg.sh" test_expect_success GPG 'create signed commits' ' + test_when_finished "test_unconfig commit.gpgsign" && + echo 1 >file && git add file && test_tick && git commit -S -m initial && git tag initial && @@ -40,9 +42,7 @@ test_expect_success GPG 'create signed commits' ' git tag seventh-unsigned && test_tick && git rebase -f HEAD^^ && git tag sixth-signed HEAD^ && - git tag seventh-signed && - - git config --unset commit.gpgsign + git tag seventh-signed ' test_expect_success GPG 'show signatures' ' -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test the commit.gpgsign config option
Nicolas Vigier writes: > The tests are checking that : > > - when commit.gpgsign is true, "git commit" creates signed commits > > - when commit.gpgsign is false, "git commit" creates unsigned commits > > - when commit.gpgsign is true, "git commit --no-gpg-sign" creates > unsigned commits > > - when commit.gpgsign is true, "git rebase -f" creates signed commits > > Signed-off-by: Nicolas Vigier > --- > t/t7510-signed-commit.sh | 25 + > 1 file changed, 21 insertions(+), 4 deletions(-) > > diff --git a/t/t7510-signed-commit.sh b/t/t7510-signed-commit.sh > index 1d3c56fe61fa..537bfba76ecf 100755 > --- a/t/t7510-signed-commit.sh > +++ b/t/t7510-signed-commit.sh > @@ -25,12 +25,29 @@ test_expect_success GPG 'create signed commits' ' > git tag fourth-unsigned && > > test_tick && git commit --amend -S -m "fourth signed" && > - git tag fourth-signed > + git tag fourth-signed && > + > + git config commit.gpgsign true && > + echo 5 >file && test_tick && git commit -a -m "fifth signed" && > + git tag fifth-signed && > + > + git config commit.gpgsign false && > + echo 6 >file && test_tick && git commit -a -m "sixth" && > + git tag sixth-unsigned && > + > + git config commit.gpgsign true && > + echo 7 >file && test_tick && git commit -a -m "seventh" --no-gpg-sign && > + git tag seventh-unsigned && > + > + test_tick && git rebase -f HEAD^^ && git tag sixth-signed HEAD^ && > + git tag seventh-signed && > + > + git config --unset commit.gpgsign If any of the above fail, the next test will run with an unknown random value in commit.gpgsign depending on where the sequence failed. Use one test_when_finished with test_unconfig at the very beginning, perhaps. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] test the commit.gpgsign config option
The tests are checking that : - when commit.gpgsign is true, "git commit" creates signed commits - when commit.gpgsign is false, "git commit" creates unsigned commits - when commit.gpgsign is true, "git commit --no-gpg-sign" creates unsigned commits - when commit.gpgsign is true, "git rebase -f" creates signed commits Signed-off-by: Nicolas Vigier --- t/t7510-signed-commit.sh | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/t/t7510-signed-commit.sh b/t/t7510-signed-commit.sh index 1d3c56fe61fa..537bfba76ecf 100755 --- a/t/t7510-signed-commit.sh +++ b/t/t7510-signed-commit.sh @@ -25,12 +25,29 @@ test_expect_success GPG 'create signed commits' ' git tag fourth-unsigned && test_tick && git commit --amend -S -m "fourth signed" && - git tag fourth-signed + git tag fourth-signed && + + git config commit.gpgsign true && + echo 5 >file && test_tick && git commit -a -m "fifth signed" && + git tag fifth-signed && + + git config commit.gpgsign false && + echo 6 >file && test_tick && git commit -a -m "sixth" && + git tag sixth-unsigned && + + git config commit.gpgsign true && + echo 7 >file && test_tick && git commit -a -m "seventh" --no-gpg-sign && + git tag seventh-unsigned && + + test_tick && git rebase -f HEAD^^ && git tag sixth-signed HEAD^ && + git tag seventh-signed && + + git config --unset commit.gpgsign ' test_expect_success GPG 'show signatures' ' ( - for commit in initial second merge master + for commit in initial second merge fourth-signed fifth-signed sixth-signed master do git show --pretty=short --show-signature $commit >actual && grep "Good signature from" actual || exit 1 @@ -39,7 +56,7 @@ test_expect_success GPG 'show signatures' ' done ) && ( - for commit in merge^2 fourth-unsigned + for commit in merge^2 fourth-unsigned sixth-unsigned seventh-unsigned do git show --pretty=short --show-signature $commit >actual && grep "Good signature from" actual && exit 1 @@ -52,7 +69,7 @@ test_expect_success GPG 'show signatures' ' test_expect_success GPG 'detect fudged signature' ' git cat-file commit master >raw && - sed -e "s/fourth signed/4th forged/" raw >forged1 && + sed -e "s/seventh/7th forged/" raw >forged1 && git hash-object -w -t commit forged1 >forged1.commit && git show --pretty=short --show-signature $(cat forged1.commit) >actual1 && grep "BAD signature from" actual1 && -- 1.8.4.2 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html