feat: ability to change the fetch url of a remote

The remote has a couple of "builder" methods to change
is fields, e.g. `push_url` for setting the push url.

A builder method for changing the fetch url of a remote
was missing. This makes it impossible to fully replicate 
the functionality of `git remote set-url`.

Author: metlos

gix/src/remote/build.rs

@@ -2,6 +2,31 @@ use crate::{bstr::BStr, remote, Remote};
 
 /// Builder methods
 impl Remote<'_> {
+    /// Set the `url` to be used when fetching data from a remote.
+    pub fn with_url<Url, E>(self, url: Url) -> Result<Self, remote::init::Error>
+    where
+        Url: TryInto<gix_url::Url, Error = E>,
+        gix_url::parse::Error: From<E>,
+    {
+        self.url_inner(
+            url.try_into().map_err(|err| remote::init::Error::Url(err.into()))?,
+            true,
+        )
+    }
+
+    /// Set the `url` to be used when fetching data from a remote, without applying rewrite rules in case these could be faulty,
+    /// eliminating one failure mode.
+    pub fn with_url_without_url_rewrite<Url, E>(self, url: Url) -> Result<Self, remote::init::Error>
+    where
+        Url: TryInto<gix_url::Url, Error = E>,
+        gix_url::parse::Error: From<E>,
+    {
+        self.url_inner(
+            url.try_into().map_err(|err| remote::init::Error::Url(err.into()))?,
+            false,
+        )
+    }
+
     /// Set the `url` to be used when pushing data to a remote.
     pub fn push_url<Url, E>(self, url: Url) -> Result<Self, remote::init::Error>
     where
@@ -50,6 +75,19 @@ impl Remote<'_> {
         Ok(self)
     }
 
+    fn url_inner(mut self, url: gix_url::Url, should_rewrite_urls: bool) -> Result<Self, remote::init::Error> {
+        self.url = url.into();
+
+        let (fetch_url_alias, _) = if should_rewrite_urls {
+            remote::init::rewrite_urls(&self.repo.config, self.url.as_ref(), None)
+        } else {
+            Ok((None, None))
+        }?;
+        self.url_alias = fetch_url_alias;
+
+        Ok(self)
+    }
+
     /// Add `specs` as refspecs for `direction` to our list if they are unique, or ignore them otherwise.
     pub fn with_refspecs<Spec>(
         mut self,

gix/tests/gix/repository/remote.rs

@@ -20,6 +20,10 @@ mod remote_at {
         );
         assert_eq!(remote.url(Direction::Fetch).unwrap().to_bstring(), fetch_url);
 
+        let new_fetch_url = "https://host.xz/byron/gitoxide";
+        remote = remote.with_url(new_fetch_url)?;
+        assert_eq!(remote.url(Direction::Fetch).unwrap().to_bstring(), new_fetch_url);
+
         for (spec, direction) in [
             ("refs/heads/push", Direction::Push),
             ("refs/heads/fetch", Direction::Fetch),
@@ -57,6 +61,18 @@ mod remote_at {
             "push is the same as fetch was rewritten"
         );
 
+        let remote = remote.with_url("https://github.com/foobar/gitoxide")?;
+        assert_eq!(
+            remote.url(Direction::Fetch).unwrap().to_bstring(),
+            rewritten_fetch_url,
+            "fetch was rewritten"
+        );
+        assert_eq!(
+            remote.url(Direction::Push).unwrap().to_bstring(),
+            rewritten_fetch_url,
+            "push is the same as fetch was rewritten"
+        );
+
         let remote = repo
             .remote_at("https://github.com/foobar/gitoxide".to_owned())?
             .push_url("file://dev/null".to_owned())?;
@@ -87,6 +103,18 @@ mod remote_at {
             "push is the same as fetch was rewritten"
         );
 
+        let remote = remote.with_url_without_url_rewrite("https://github.com/foobaz/gitoxide")?;
+        assert_eq!(
+            remote.url(Direction::Fetch).unwrap().to_bstring(),
+            "https://github.com/foobaz/gitoxide",
+            "fetch was rewritten"
+        );
+        assert_eq!(
+            remote.url(Direction::Push).unwrap().to_bstring(),
+            "https://github.com/foobaz/gitoxide",
+            "push is the same as fetch was rewritten"
+        );
+
         let remote = repo
             .remote_at_without_url_rewrite("https://github.com/foobar/gitoxide".to_owned())?
             .push_url_without_url_rewrite("file://dev/null".to_owned())?;
@@ -96,6 +124,19 @@ mod remote_at {
             "file://dev/null",
             "push-url rewrite rules are not applied"
         );
+
+        let remote = remote
+            .with_url_without_url_rewrite("https://github.com/foobaz/gitoxide".to_owned())?
+            .push_url_without_url_rewrite("file://dev/null".to_owned())?;
+        assert_eq!(
+            remote.url(Direction::Fetch).unwrap().to_bstring(),
+            "https://github.com/foobaz/gitoxide"
+        );
+        assert_eq!(
+            remote.url(Direction::Push).unwrap().to_bstring(),
+            "file://dev/null",
+            "push-url rewrite rules are not applied"
+        );
         Ok(())
     }
 }