Fix wrong handling of GitHub repository URLs with "git"

When you have a URL like 'https://github.com/suse-edge/suse-edge.github.io.git'
the splitting mechanism splitted at `.git` which gave wrong results.

Using `.rsplit('.git', 1)` instead of `.split('.git')` fixed the issue.

Author: tomschr

src/docbuild/models/repo.py

@@ -79,7 +79,7 @@ def __init__(self, value: str) -> None:
 
         if 'https://' in value or 'http://' in value:
             parsed_original = urlparse(value)
-            name = parsed_original.path.strip('/').lower().split('.git')[0]
+            name = parsed_original.path.strip('/').lower().rsplit('.git', 1)[0]
             name = name.rstrip('/')
             url = f'{parsed_original.scheme}://{parsed_original.netloc}/{name}.git'
             host = f'{parsed_original.scheme}://{parsed_original.netloc}'
@@ -103,7 +103,7 @@ def __init__(self, value: str) -> None:
 
         elif '/' in value:
             value = value.lower()
-            name = value.split('.git')[0].rstrip('/')
+            name = value.rsplit('.git', 1)[0].rstrip('/')
             url = f'{self.DEFAULT_HOST}/{name}.git'
             surl = f'gh://{name}'
 

tests/models/test_repo.py

@@ -23,6 +23,12 @@ def repo_url() -> str:
         ('https://github.com/org/repo', 'org/repo', 'https://github.com/org/repo.git'),
         # 4
         ('https://github.com/org/repo/', 'org/repo', 'https://github.com/org/repo.git'),
+        # 5
+        (
+            'https://github.com/ORG/repo_git.git/',
+            'org/repo_git',
+            'https://github.com/org/repo_git.git',
+        ),
     ],
 )
 def test_repo_https(input_value, name, url):
@@ -42,6 +48,8 @@ def test_repo_https(input_value, name, url):
         ('[email protected]:org/repo', 'org/repo', 'https://github.com/org/repo.git'),
         # 4
         ('[email protected]:org/repo/', 'org/repo', 'https://github.com/org/repo.git'),
+        # 5
+        ('[email protected]:org/repo_git/', 'org/repo_git', 'https://github.com/org/repo_git.git'),
     ],
 )
 def test_repo_ssh(input_value, name, url):
@@ -61,6 +69,8 @@ def test_repo_ssh(input_value, name, url):
         ('gh://org/repo/', 'org/repo', 'https://github.com/org/repo.git'),
         # 4
         ('gh://org/repo.git', 'org/repo', 'https://github.com/org/repo.git'),
+        # 5
+        ('gh://ORG/repo_git.git', 'org/repo_git', 'https://github.com/org/repo_git.git'),
     ],
 )
 def test_repo_abbreviation(input_value, name, url):
@@ -86,6 +96,8 @@ def test_repo_abbreviation(input_value, name, url):
         ('gh://org/repo/', 'gh://org/repo'),
         # 7
         ('gh://org/repo.git', 'gh://org/repo'),
+        # 8
+        ('gh://ORG/repo_git.git', 'gh://org/repo_git'),
     ],
 )
 def test_repo_with_surl(input_value, surl):
@@ -105,6 +117,9 @@ def test_repo_with_unknown_abbreviation():
         ('org/repo', 'org/repo', 'https://github.com/org/repo.git'),
         # 2
         ('ORG/repo', 'org/repo', 'https://github.com/org/repo.git'),
+        # 3
+        ('org/repo_git.git', 'org/repo_git', 'https://github.com/org/repo_git.git'),
+        # 4
     ],
 )
 def test_repo_abbreviated(input_value, name, url):