Add test for DescribeOptions.OnlyFollowFirstParent

This commit adds a test that sets up a branch where there is a less recent tag on the same branch and a more recent tag on a merged branch. When Describe is called with OnlyFollowFirstParent = false, the latter tag should be returned, and vice versa.

Author: ninjeff

LibGit2Sharp.Tests/DescribeFixture.cs

@@ -1,6 +1,7 @@
 using System.Linq;
 using LibGit2Sharp.Tests.TestHelpers;
 using Xunit;
+using System;
 
 namespace LibGit2Sharp.Tests
 {
@@ -49,5 +50,38 @@ public void CanDescribeACommit()
                     new DescribeOptions{ AlwaysRenderLongFormat = true }));
             }
         }
+
+        [Fact]
+        public void CanFollowFirstParent()
+        {
+            string path = SandboxStandardTestRepo();
+            string configPath = CreateConfigurationWithDummyUser(Constants.Identity);
+            var options = new RepositoryOptions { GlobalConfigurationLocation = configPath };
+
+            using (var repo = new Repository(path, options))
+            {
+                var branch = repo.CreateBranch("branch");
+
+                // Make an earlier tag on master
+                repo.Commit("A", repo.Config.BuildSignature(DateTimeOffset.Now), repo.Config.BuildSignature(DateTimeOffset.Now), new CommitOptions { AllowEmptyCommit = true });
+                repo.ApplyTag("firstParentTag");
+
+                // Make a later tag on branch
+                repo.Checkout(branch);
+                repo.Commit("B", repo.Config.BuildSignature(DateTimeOffset.Now), repo.Config.BuildSignature(DateTimeOffset.Now), new CommitOptions { AllowEmptyCommit = true });
+                repo.ApplyTag("mostRecentTag");
+
+                repo.Checkout("master");
+                repo.Commit("C", repo.Config.BuildSignature(DateTimeOffset.Now), repo.Config.BuildSignature(DateTimeOffset.Now), new CommitOptions { AllowEmptyCommit = true });
+                repo.Merge(branch, repo.Config.BuildSignature(DateTimeOffset.Now), new MergeOptions() { FastForwardStrategy = FastForwardStrategy.NoFastForward });
+
+                // With OnlyFollowFirstParent = false, the most recent tag reachable should be returned
+                Assert.Equal("mostRecentTag", repo.Describe(repo.Head.Tip, new DescribeOptions { OnlyFollowFirstParent = false, MinimumCommitIdAbbreviatedSize = 0, Strategy = DescribeStrategy.Tags }));
+
+                // With OnlyFollowFirstParent = true, the most recent tag on the current branch should be returned
+                Assert.Equal("firstParentTag", repo.Describe(repo.Head.Tip, new DescribeOptions { OnlyFollowFirstParent = true, MinimumCommitIdAbbreviatedSize = 0, Strategy = DescribeStrategy.Tags }));
+
+            }
+        }
     }
 }