]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t7030: stop using invalid tag name
authorJeff King <peff@peff.net>
Wed, 18 Jan 2023 20:36:22 +0000 (15:36 -0500)
committerJunio C Hamano <gitster@pobox.com>
Wed, 18 Jan 2023 20:59:44 +0000 (12:59 -0800)
We intentionally invalidate the signature of a tag by switching its tag
name from "seventh" to "7th forged". However, the latter is not a valid
tag name because it contains a space. This doesn't currently affect the
test, but we're better off using something syntactically valid. That
reduces the number of possible failure modes in the test, and
future-proofs us if git hash-object gets more picky about its input.

The t7031 script, which was mostly copied from t7030, has the same
problem, so we'll fix it, too.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t7030-verify-tag.sh
t/t7031-verify-tag-signed-ssh.sh

index 10faa645157ea4521e370abdfd470c84a24f6051..6f526c37c2776e6288a1abe5860b6a9efed25183 100755 (executable)
@@ -115,7 +115,7 @@ test_expect_success GPGSM 'verify and show signatures x509 with high minTrustLev
 
 test_expect_success GPG 'detect fudged signature' '
        git cat-file tag seventh-signed >raw &&
-       sed -e "/^tag / s/seventh/7th forged/" raw >forged1 &&
+       sed -e "/^tag / s/seventh/7th-forged/" raw >forged1 &&
        git hash-object -w -t tag forged1 >forged1.tag &&
        test_must_fail git verify-tag $(cat forged1.tag) 2>actual1 &&
        grep "BAD signature from" actual1 &&
index 1cb36b9ab83cdba01a554a94c982c968cd572f8b..36eb86a4b19f3c758b2830fe5157a1667069321a 100755 (executable)
@@ -125,7 +125,7 @@ test_expect_success GPGSSH,GPGSSH_VERIFYTIME 'verify-tag failes with tag date ou
 test_expect_success GPGSSH 'detect fudged ssh signature' '
        test_config gpg.ssh.allowedSignersFile "${GPGSSH_ALLOWED_SIGNERS}" &&
        git cat-file tag seventh-signed >raw &&
-       sed -e "/^tag / s/seventh/7th forged/" raw >forged1 &&
+       sed -e "/^tag / s/seventh/7th-forged/" raw >forged1 &&
        git hash-object -w -t tag forged1 >forged1.tag &&
        test_must_fail git verify-tag $(cat forged1.tag) 2>actual1 &&
        grep "${GPGSSH_BAD_SIGNATURE}" actual1 &&