Repo.rev_parse: Handle <tag>^{commit} correctly

This should resolve to commit object. Fixes: #1995 Signed-off-by: Frank Lichtenheld <[email protected]>

Author: flichtenheld

git/repo/fun.py

@@ -301,7 +301,13 @@ def rev_parse(repo: "Repo", rev: str) -> AnyGitObject:
 
 # Handle type.
 ifoutput_type=="commit":
-pass# Default.
+obj=cast("TagObject", obj)
+ifobjandobj.type=="tag":
+obj=deref_tag(obj)
+else:
+# Cannot do anything for non-tags.
+pass
+# END handle tag
 elifoutput_type=="tree":
 try:
 obj=cast(AnyGitObject, obj)

test/test_repo.py

@@ -1064,9 +1064,9 @@ def test_rev_parse(self):
 # TODO: Dereference tag into a blob 0.1.7^{blob} - quite a special one.
 # Needs a tag which points to a blob.
 
-# ref^0 returns commit being pointed to, same with ref~0, and ^{}
+# ref^0 returns commit being pointed to, same with ref~0, ^{}, and ^{commit}
 tag=rev_parse("0.1.4")
-fortokenin ("~0", "^0", "^{}"):
+fortokenin ("~0", "^0", "^{}", "^{commit}"):
 self.assertEqual(tag.object, rev_parse("0.1.4%s"%token))
 # END handle multiple tokens