fix: failure to parse provider info if .git suffix is missing (#70)

Author: mrcjkb

src/types/provider/mod.rs

@@ -90,8 +90,11 @@ impl<'a> GenericProvider<'a> {
     /// Parse the most common form of git url by offered by git providers
     fn parse_path(input: &str) -> Result<(&str, GenericProvider), GitUrlParseError> {
         let (input, _) = opt(tag("/")).parse(input)?;
-        let (input, (user, repo)) =
-            separated_pair(is_not("/"), tag("/"), take_until(".git")).parse(input)?;
+        let (input, (user, repo)) = if input.ends_with(".git") {
+            separated_pair(is_not("/"), tag("/"), take_until(".git")).parse(input)?
+        } else {
+            separated_pair(is_not("/"), tag("/"), is_not("/")).parse(input)?
+        };
         Ok((input, GenericProvider { owner: user, repo }))
     }
 

tests/provider.rs

@@ -209,6 +209,25 @@ fn ssh_gitlab_subgroups() {
     assert_eq!(provider_info.fullname(), full);
 }
 
+#[test]
+fn url_without_git_suffix() {
+    let _ = env_logger::try_init();
+    let test_url = "http://git.example.com:3000/user/repo";
+    let parsed = GitUrl::parse(test_url).expect("URL parse failed");
+    debug!("{:#?}", parsed);
+
+    let provider_info: GenericProvider = parsed.provider_info().unwrap();
+    debug!("{:#?}", provider_info);
+
+    let owner = "user";
+    let repo = "repo";
+    let full = format!("{owner}/{repo}");
+
+    assert_eq!(provider_info.owner(), owner);
+    assert_eq!(provider_info.repo(), repo);
+    assert_eq!(provider_info.fullname(), full);
+}
+
 #[test]
 fn filepath() {
     let _ = env_logger::try_init();