chore: sate clippy

Author: CardboardTurkey

src/lib.rs

@@ -107,7 +107,7 @@ impl fmt::Display for GitUrl {
                     format!(":{}", &self.path)
                 }
             }
-            _ => (&self.path).to_string(),
+            _ => self.path.to_string(),
         };
 
         let git_url_str = format!("{}{}{}{}{}", scheme_prefix, auth_info, host, port, path);
@@ -211,7 +211,7 @@ impl GitUrl {
                 let mut fullname: Vec<&str> = Vec::new();
 
                 // TODO: Add support for parsing out orgs from these urls
-                let hosts_w_organization_in_path = vec!["dev.azure.com", "ssh.dev.azure.com"];
+                let hosts_w_organization_in_path = ["dev.azure.com", "ssh.dev.azure.com"];
                 //vec!["dev.azure.com", "ssh.dev.azure.com", "visualstudio.com"];
 
                 let host_str = if let Some(host) = normalized.host_str() {
@@ -362,7 +362,7 @@ fn normalize_file_path(filepath: &str) -> Result<Url, GitUrlParseError> {
             if let Ok(file_url) = normalize_url(&format!("file://{}", filepath)) {
                 Ok(file_url)
             } else {
-                return Err(GitUrlParseError::FileUrlNormalizeFailedSchemeAdded);
+                Err(GitUrlParseError::FileUrlNormalizeFailedSchemeAdded)
             }
         }
     }
@@ -463,11 +463,7 @@ fn is_ssh_url(url: &str) -> bool {
 
         // Make sure we provided a username, and not just `@`
         let parts: Vec<&str> = url.split('@').collect();
-        if parts.len() != 2 && !parts[0].is_empty() {
-            return false;
-        } else {
-            return true;
-        }
+        return parts.len() == 2 || parts[0].is_empty();
     }
 
     // it's an ssh url if we have a domain:path pattern
@@ -481,11 +477,7 @@ fn is_ssh_url(url: &str) -> bool {
     // This should also handle if a port is specified
     // no port example: ssh://user@domain:path/to/repo.git
     // port example: ssh://user@domain:port/path/to/repo.git
-    if parts.len() != 2 && !parts[0].is_empty() && !parts[1].is_empty() {
-        return false;
-    } else {
-        return true;
-    }
+    parts.len() == 2 && parts[0].is_empty() && parts[1].is_empty()
 }
 
 #[derive(Error, Debug, PartialEq, Eq)]

tests/normalize.rs

@@ -169,7 +169,7 @@ fn null_in_input2() {
 // From https://github.com/tjtelan/git-url-parse-rs/issues/51
 #[test]
 fn large_bad_input1() {
-    let test_url = std::iter::repeat("g@1::::").take(10000).collect::<String>();
+    let test_url = "g@1::::".repeat(10000);
     let normalized = normalize_url(&test_url);
 
     assert!(normalized.is_err());
@@ -178,7 +178,7 @@ fn large_bad_input1() {
 // From https://github.com/tjtelan/git-url-parse-rs/issues/51
 #[test]
 fn large_bad_input2() {
-    let test_url = std::iter::repeat(":").take(10000).collect::<String>();
+    let test_url = ":".repeat(10000);
     let normalized = normalize_url(&test_url);
 
     assert!(normalized.is_err());