@@ -130,4 +130,150 @@ test_expect_success 'traverse unexpected non-blob tag (seen)' '
test_i18ngrep "not a blob" output
'
+test_expect_success 'setup unexpected non-tag tag' '
+ test_when_finished "git tag -d tag-commit tag-tag" &&
+
+ git tag -a -m"my tagged commit" tag-commit $commit &&
+ tag_commit=$(git rev-parse tag-commit) &&
+ git tag -a -m"my tagged tag" tag-tag tag-commit &&
+ tag_tag=$(git rev-parse tag-tag) &&
+
+ git cat-file tag tag-tag >good-tag-tag &&
+ git cat-file tag tag-commit >good-commit-tag &&
+
+ sed -e "s/$tag_commit/$commit/" <good-tag-tag >broken-tag-tag-commit &&
+ sed -e "s/$tag_commit/$tree/" <good-tag-tag >broken-tag-tag-tree &&
+ sed -e "s/$tag_commit/$blob/" <good-tag-tag >broken-tag-tag-blob &&
+
+ sed -e "s/$commit/$tag_commit/" <good-commit-tag >broken-commit-tag-tag &&
+ sed -e "s/$commit/$tree/" <good-commit-tag >broken-commit-tag-tree &&
+ sed -e "s/$commit/$blob/" <good-commit-tag >broken-commit-tag-blob &&
+
+ tag_tag_commit=$(git hash-object -w -t tag broken-tag-tag-commit) &&
+ tag_tag_tree=$(git hash-object -w -t tag broken-tag-tag-tree) &&
+ tag_tag_blob=$(git hash-object -w -t tag broken-tag-tag-blob) &&
+
+ git update-ref refs/tags/tag_tag_commit $tag_tag_commit &&
+ git update-ref refs/tags/tag_tag_tree $tag_tag_tree &&
+ git update-ref refs/tags/tag_tag_blob $tag_tag_blob &&
+
+ commit_tag_tag=$(git hash-object -w -t tag broken-commit-tag-tag) &&
+ commit_tag_tree=$(git hash-object -w -t tag broken-commit-tag-tree) &&
+ commit_tag_blob=$(git hash-object -w -t tag broken-commit-tag-blob) &&
+
+ git update-ref refs/tags/commit_tag_tag $commit_tag_tag &&
+ git update-ref refs/tags/commit_tag_tree $commit_tag_tree &&
+ git update-ref refs/tags/commit_tag_blob $commit_tag_blob
+'
+
+test_expect_failure 'traverse unexpected incorrectly typed tag (to commit & tag)' '
+ test_must_fail git rev-list --objects $tag_tag_commit 2>err &&
+ cat >expect <<-EOF &&
+ error: object $commit is a commit, not a tag
+ fatal: bad object $commit
+ EOF
+ test_cmp expect err &&
+
+ test_must_fail git rev-list --objects $commit_tag_tag 2>err &&
+ cat >expect <<-EOF &&
+ error: object $tag_commit is a tag, not a commit
+ fatal: bad object $tag_commit
+ EOF
+ test_cmp expect err
+'
+
+test_expect_failure 'traverse unexpected incorrectly typed tag (to tree)' '
+ test_must_fail git rev-list --objects $tag_tag_tree 2>err &&
+ cat >expect <<-EOF &&
+ error: object $tree is a tree, not a tag
+ fatal: bad object $tree
+ EOF
+ test_cmp expect err &&
+
+ test_must_fail git rev-list --objects $commit_tag_tree 2>err &&
+ cat >expect <<-EOF &&
+ error: object $tree is a tree, not a commit
+ fatal: bad object $tree
+ EOF
+ test_cmp expect err
+'
+
+test_expect_failure 'traverse unexpected incorrectly typed tag (to blob)' '
+ test_must_fail git rev-list --objects $tag_tag_blob 2>err &&
+ cat >expect <<-EOF &&
+ error: object $blob is a blob, not a tag
+ fatal: bad object $blob
+ EOF
+ test_cmp expect err &&
+
+ test_must_fail git rev-list --objects $commit_tag_blob 2>err &&
+ cat >expect <<-EOF &&
+ error: object $blob is a blob, not a commit
+ fatal: bad object $blob
+ EOF
+ test_cmp expect err
+'
+
+test_expect_failure 'traverse unexpected non-tag tag (tree seen to blob)' '
+ test_must_fail git rev-list --objects $tree $commit_tag_blob 2>err &&
+ cat >expect <<-EOF &&
+ error: object $blob is a blob, not a commit
+ fatal: bad object $blob
+ EOF
+ test_cmp expect err &&
+
+ test_must_fail git rev-list --objects $tree $tag_tag_blob 2>err &&
+ cat >expect <<-EOF &&
+ error: object $blob is a blob, not a tag
+ fatal: bad object $blob
+ EOF
+ test_cmp expect err
+'
+
+
+test_expect_failure 'traverse unexpected objects with for-each-ref' '
+ cat >expect <<-EOF &&
+ error: bad tag pointer to $tree in $tag_tag_tree
+ fatal: parse_object_buffer failed on $tag_tag_tree for refs/tags/tag_tag_tree
+ EOF
+ test_must_fail git for-each-ref --format="%(*objectname)" 2>actual &&
+ test_cmp expect actual
+'
+
+>fsck-object-isa
+test_expect_failure 'setup: unexpected objects with fsck' '
+ test_must_fail git fsck 2>err &&
+ sed -n -e "/^error: object .* is a .*, not a .*$/ {
+ s/^error: object \([0-9a-f]*\) is a \([a-z]*\), not a [a-z]*$/\\1 \\2/;
+ p;
+ }" <err >fsck-object-isa
+'
+
+while read oid type
+do
+ test_expect_failure "fsck knows unexpected object $oid is $type" '
+ git cat-file -t $oid >expect &&
+ echo $type >actual &&
+ test_cmp expect actual
+ '
+done <fsck-object-isa
+
+test_expect_success 'traverse unexpected non-tag tag (blob seen to blob)' '
+ test_must_fail git rev-list --objects $blob $commit_tag_blob 2>err &&
+ cat >expected <<-EOF &&
+ error: object $blob is a blob, not a commit
+ error: bad tag pointer to $blob in $commit_tag_blob
+ fatal: bad object $commit_tag_blob
+ EOF
+ test_cmp expected err &&
+
+ test_must_fail git rev-list --objects $blob $tag_tag_blob 2>err &&
+ cat >expected <<-EOF &&
+ error: object $blob is a blob, not a tag
+ error: bad tag pointer to $blob in $tag_tag_blob
+ fatal: bad object $tag_tag_blob
+ EOF
+ test_cmp expected err
+'
+
test_done
Fix a blind spot in the tests added in 0616617c7e1 (t: introduce tests for unexpected object types, 2019-04-09), there were no meaningful tests for checking how we reported on finding the incorrect object type in a tag, i.e. one that broke the "type" promise in the tag header. We'll report the wrong object type in these cases, and thus fail on the "test_cmp", e.g. for the first "error: " output being tested here we should say "$commit is a tag, not a commit", instead we say "$commit is a commit, not a tag". This will be fixed in a subsequent commit. See the discussion & notes in [1] and downthread of there for test snippets that are adapted here. In the case of "fsck" which objects we visit in what order, and if we report errors on them depends on their OIDs. So the test uses the technique of extracting the OID/type combinations that fsck does report, and asserting that those are correct (currently, it's far from correct). 1. https://lore.kernel.org/git/YGTGgFI19fS7Uv6I@coredump.intra.peff.net/ Helped-by: Jeff King <peff@peff.net> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- t/t6102-rev-list-unexpected-objects.sh | 146 +++++++++++++++++++++++++ 1 file changed, 146 insertions(+)