Skip to content

Commit 8e92c2c

Browse files
Michael J Grubergitster
authored andcommitted
t7510: test verify-commit
This mixes the "git verify-commit" tests in with the "git show --show-signature" tests, to keep the tests more readable. The tests already mix in the "call show" tests with the "verify" tests. So in case of a test beakage, a '-v' run would be needed to reveal the exact point of breakage anyway. Additionally, test the actual output of "git verify-commit" and "git show --show-signature" and compare to "git cat-file". Signed-off-by: Michael J Gruber <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 0f109c9 commit 8e92c2c

File tree

1 file changed

+19
-1
lines changed

1 file changed

+19
-1
lines changed

t/t7510-signed-commit.sh

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,11 @@ test_expect_success GPG 'create signed commits' '
4545
git tag seventh-signed
4646
'
4747

48-
test_expect_success GPG 'show signatures' '
48+
test_expect_success GPG 'verify and show signatures' '
4949
(
5050
for commit in initial second merge fourth-signed fifth-signed sixth-signed master
5151
do
52+
git verify-commit $commit &&
5253
git show --pretty=short --show-signature $commit >actual &&
5354
grep "Good signature from" actual &&
5455
! grep "BAD signature from" actual || exit 1
@@ -58,6 +59,7 @@ test_expect_success GPG 'show signatures' '
5859
(
5960
for commit in merge^2 fourth-unsigned sixth-unsigned seventh-unsigned
6061
do
62+
test_must_fail git verify-commit $commit &&
6163
git show --pretty=short --show-signature $commit >actual &&
6264
! grep "Good signature from" actual &&
6365
! grep "BAD signature from" actual || exit 1
@@ -66,11 +68,25 @@ test_expect_success GPG 'show signatures' '
6668
)
6769
'
6870

71+
test_expect_success GPG 'show signed commit with signature' '
72+
git show -s initial >commit &&
73+
git show -s --show-signature initial >show &&
74+
git verify-commit -v initial >verify.1 2>verify.2 &&
75+
git cat-file commit initial >cat &&
76+
grep -v "gpg: " show >show.commit &&
77+
grep "gpg: " show >show.gpg &&
78+
grep -v "^ " cat | grep -v "^gpgsig " >cat.commit &&
79+
test_cmp show.commit commit &&
80+
test_cmp show.gpg verify.2 &&
81+
test_cmp cat.commit verify.1
82+
'
83+
6984
test_expect_success GPG 'detect fudged signature' '
7085
git cat-file commit master >raw &&
7186
7287
sed -e "s/seventh/7th forged/" raw >forged1 &&
7388
git hash-object -w -t commit forged1 >forged1.commit &&
89+
! git verify-commit $(cat forged1.commit) &&
7490
git show --pretty=short --show-signature $(cat forged1.commit) >actual1 &&
7591
grep "BAD signature from" actual1 &&
7692
! grep "Good signature from" actual1
@@ -81,6 +97,7 @@ test_expect_success GPG 'detect fudged signature with NUL' '
8197
cat raw >forged2 &&
8298
echo Qwik | tr "Q" "\000" >>forged2 &&
8399
git hash-object -w -t commit forged2 >forged2.commit &&
100+
! git verify-commit $(cat forged2.commit) &&
84101
git show --pretty=short --show-signature $(cat forged2.commit) >actual2 &&
85102
grep "BAD signature from" actual2 &&
86103
! grep "Good signature from" actual2
@@ -89,6 +106,7 @@ test_expect_success GPG 'detect fudged signature with NUL' '
89106
test_expect_success GPG 'amending already signed commit' '
90107
git checkout fourth-signed^0 &&
91108
git commit --amend -S --no-edit &&
109+
git verify-commit HEAD &&
92110
git show -s --show-signature HEAD >actual &&
93111
grep "Good signature from" actual &&
94112
! grep "BAD signature from" actual

0 commit comments

Comments
 (0)