Unskip include-tag related fetch tests

nulltoken · nulltoken · commit bc4897d84f82 · 2015-03-13T20:49:06.000+01:00
Due to a recent GitHub policy change, those tests now pass
diff --git a/LibGit2Sharp.Tests/FetchFixture.cs b/LibGit2Sharp.Tests/FetchFixture.cs
@@ -12,7 +12,7 @@ public class FetchFixture : BaseFixture
     {
         private const string remoteName = "testRemote";
 
-        [Theory(Skip = "Skipping due to recent github handling modification of --include-tag.")]
+        [Theory]
         [InlineData("http://github.com/libgit2/TestGitRepository")]
         [InlineData("https://github.com/libgit2/TestGitRepository")]
         [InlineData("git://github.com/libgit2/TestGitRepository.git")]
@@ -152,7 +152,7 @@ public void CanFetchCustomRefSpecsIntoAnEmptyRepository(string url, string local
         [Theory]
         [InlineData(TagFetchMode.All, 4)]
         [InlineData(TagFetchMode.None, 0)]
-        //[InlineData(TagFetchMode.Auto, 3)] // TODO: Skipping due to github modification of --include-tag handling."
+        [InlineData(TagFetchMode.Auto, 3)]
         public void FetchRespectsConfiguredAutoTagSetting(TagFetchMode tagFetchMode, int expectedTagCount)
         {
             string url = "http://github.com/libgit2/TestGitRepository";
diff --git a/LibGit2Sharp.Tests/RepositoryFixture.cs b/LibGit2Sharp.Tests/RepositoryFixture.cs
@@ -150,7 +150,7 @@ private static void AssertIsHidden(string repoPath)
             Assert.Equal(FileAttributes.Hidden, (attribs & FileAttributes.Hidden));
         }
 
-        [Fact(Skip = "Skipping due to recent github handling modification of --include-tag.")]
+        [Fact]
         public void CanFetchFromRemoteByName()
         {
             string remoteName = "testRemote";
diff --git a/LibGit2Sharp.Tests/SmartSubtransportFixture.cs b/LibGit2Sharp.Tests/SmartSubtransportFixture.cs
@@ -55,7 +55,7 @@ public void CustomSmartSubtransportTest(string scheme, string url)
                     }
 
                     // Add the expected tags
-                    string[] expectedTagNames = { "blob", "commit_tree" };
+                    string[] expectedTagNames = { "blob", "commit_tree", "annotated_tag" };
                     foreach (string tagName in expectedTagNames)
                     {
                         TestRemoteInfo.ExpectedTagInfo expectedTagInfo = expectedResults.Tags[tagName];

Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,7 @@ private static void AssertIsHidden(string repoPath)`
`150`	`150`	`Assert.Equal(FileAttributes.Hidden, (attribs & FileAttributes.Hidden));`
`151`	`151`	`}`
`152`	`152`
`153`		`- [Fact(Skip = "Skipping due to recent github handling modification of --include-tag.")]`
	`153`	`+ [Fact]`
`154`	`154`	`public void CanFetchFromRemoteByName()`
`155`	`155`	`{`
`156`	`156`	`string remoteName = "testRemote";`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public void CustomSmartSubtransportTest(string scheme, string url)`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`// Add the expected tags`
`58`		`- string[] expectedTagNames = { "blob", "commit_tree" };`
	`58`	`+ string[] expectedTagNames = { "blob", "commit_tree", "annotated_tag" };`
`59`	`59`	`foreach (string tagName in expectedTagNames)`
`60`	`60`	`{`
`61`	`61`	`TestRemoteInfo.ExpectedTagInfo expectedTagInfo = expectedResults.Tags[tagName];`