Remote.PushUrl defaults to Remote.Url

Author: Ryan Roden-Corrent

LibGit2Sharp.Tests/RemoteFixture.cs

@@ -91,6 +91,8 @@ public void CanSetRemoteUrl()
                     r => r.Url = newUrl);
 
                 Assert.Equal(newUrl, updatedremote.Url);
+                // with no push url set, PushUrl defaults to the fetch url
+                Assert.Equal(newUrl, updatedremote.PushUrl);
             }
         }
 
@@ -111,6 +113,8 @@ public void CanSetRemotePushUrl()
                 Remote updatedremote = repo.Network.Remotes.Update(remote,
                     r => r.PushUrl = pushurl);
 
+                // url should not change, push url should be set to new value
+                Assert.Equal(url, updatedremote.Url);
                 Assert.Equal(pushurl, updatedremote.PushUrl);
             }
         }

LibGit2Sharp/Remote.cs

@@ -20,6 +20,7 @@ public class Remote : IEquatable<Remote>, IBelongToARepository
         internal readonly Repository repository;
 
         private readonly RefSpecCollection refSpecs;
+        private string pushUrl;
 
         /// <summary>
         /// Needed for mocking purposes.
@@ -56,9 +57,18 @@ internal static Remote BuildFromPtr(RemoteSafeHandle handle, Repository repo)
 
         /// <summary>
         /// Gets the distinct push url for this remote repository, if set.
-        /// If no separate push url is specified, PushUrl is null.
+        /// Defaults to the fetch url (<see cref="Url"/>) if not set.
         /// </summary>
-        public virtual string PushUrl { get; private set; }
+        public virtual string PushUrl {
+            get
+            {
+                return pushUrl ?? Url;
+            }
+            private set
+            {
+                pushUrl = value;
+            }
+        }
 
         /// <summary>
         /// Gets the Tag Fetch Mode of the remote - indicating how tags are fetched.