Move Push related tests to a separate file

Author: yorah

LibGit2Sharp.Tests/CloneFixture.cs

@@ -50,51 +50,6 @@ private void AssertLocalClone(string path)
             }
         }
 
-        [Fact]
-        public void CanLocallyCloneAndCommitAndPush()
-        {
-            var scd = BuildSelfCleaningDirectory();
-            using (var originalRepo = new Repository(CloneBareTestRepo()))
-            using (Repository clonedRepo = Repository.Clone(originalRepo.Info.Path, scd.RootedDirectoryPath))
-            {
-                Remote remote = clonedRepo.Network.Remotes["origin"];
-
-                // Compare before
-                Assert.Equal(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
-                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
-                Assert.Equal(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
-                    clonedRepo.Refs.Head.ResolveToDirectReference());
-
-                // Change local state (commit)
-                const string relativeFilepath = "new_file.txt";
-                string filePath = Path.Combine(clonedRepo.Info.WorkingDirectory, relativeFilepath);
-                File.WriteAllText(filePath, "__content__");
-                clonedRepo.Index.Stage(relativeFilepath);
-                clonedRepo.Commit("__commit_message__", DummySignature, DummySignature);
-
-                // Assert local state has changed
-                Assert.NotEqual(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
-                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
-                Assert.NotEqual(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
-                    clonedRepo.Refs.Head.ResolveToDirectReference());
-
-                // Push the change upstream (remote state is supposed to change)
-                clonedRepo.Network.Push(remote, "HEAD", @"refs/heads/master", OnPushStatusError);
-
-                // Assert that both local and remote repos are in sync
-                Assert.Equal(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
-                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
-                Assert.Equal(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
-                    clonedRepo.Refs.Head.ResolveToDirectReference());
-            }
-        }
-
-        private void OnPushStatusError(PushStatusError pushStatusErrors)
-        {
-            Assert.True(false, string.Format("Failed to update reference '{0}': {1}",
-                pushStatusErrors.Reference, pushStatusErrors.Message));
-        }
-
         [Fact]
         public void CanCloneALocalRepositoryFromALocalUri()
         {

LibGit2Sharp.Tests/LibGit2Sharp.Tests.csproj

@@ -60,6 +60,7 @@
   <ItemGroup>
     <Compile Include="CheckoutFixture.cs" />
     <Compile Include="RemoteFixture.cs" />
+    <Compile Include="PushFixture.cs" />
     <Compile Include="StageFixture.cs" />
     <Compile Include="StashFixture.cs" />
     <Compile Include="CloneFixture.cs" />

LibGit2Sharp.Tests/PushFixture.cs

@@ -0,0 +1,55 @@
+using System.IO;
+using System.Linq;
+using LibGit2Sharp.Tests.TestHelpers;
+using Xunit;
+
+namespace LibGit2Sharp.Tests
+{
+    public class PushFixture : BaseFixture
+    {
+        private void OnPushStatusError(PushStatusError pushStatusErrors)
+        {
+            Assert.True(false, string.Format("Failed to update reference '{0}': {1}",
+                pushStatusErrors.Reference, pushStatusErrors.Message));
+        }
+
+        [Fact]
+        public void CanLocallyCloneAndCommitAndPush()
+        {
+            var scd = BuildSelfCleaningDirectory();
+            using (var originalRepo = new Repository(CloneBareTestRepo()))
+            using (Repository clonedRepo = Repository.Clone(originalRepo.Info.Path, scd.RootedDirectoryPath))
+            {
+                Remote remote = clonedRepo.Network.Remotes["origin"];
+
+                // Compare before
+                Assert.Equal(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
+                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
+                Assert.Equal(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
+                    clonedRepo.Refs.Head.ResolveToDirectReference());
+
+                // Change local state (commit)
+                const string relativeFilepath = "new_file.txt";
+                string filePath = Path.Combine(clonedRepo.Info.WorkingDirectory, relativeFilepath);
+                File.WriteAllText(filePath, "__content__");
+                clonedRepo.Index.Stage(relativeFilepath);
+                clonedRepo.Commit("__commit_message__", DummySignature, DummySignature);
+
+                // Assert local state has changed
+                Assert.NotEqual(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
+                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
+                Assert.NotEqual(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
+                    clonedRepo.Refs.Head.ResolveToDirectReference());
+
+                // Push the change upstream (remote state is supposed to change)
+                clonedRepo.Network.Push(remote, "HEAD", @"refs/heads/master", OnPushStatusError);
+
+                // Assert that both local and remote repos are in sync
+                Assert.Equal(originalRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier,
+                    clonedRepo.Refs["HEAD"].ResolveToDirectReference().TargetIdentifier);
+                Assert.Equal(clonedRepo.Network.ListReferences(remote).Single(r => r.CanonicalName == "refs/heads/master"),
+                    clonedRepo.Refs.Head.ResolveToDirectReference());
+            }
+        }
+    }
+}