chore: handle git@github.com url format (#153)

Author: sileht

pytest_mergify/utils.py

@@ -36,8 +36,20 @@ def get_ci_provider() -> typing.Optional[CIProviderT]:
 
 
 def get_repository_name_from_url(repository_url: str) -> typing.Optional[str]:
+    # Handle SSH Git URLs like git@github.com:owner/repo.git
     if match := re.match(
-        r"(https?://[\w.-]+/)?(?P<full_name>[\w.-]+/[\w.-]+)/?$",
+        r"git@[\w.-]+:(?P<full_name>[\w.-]+/[\w.-]+)(?:\.git)?/?$",
+        repository_url,
+    ):
+        full_name = match.group("full_name")
+        # Remove .git suffix if present
+        if full_name.endswith(".git"):
+            full_name = full_name[:-4]
+        return full_name
+
+    # Handle HTTPS/HTTP URLs like https://github.com/owner/repo (with optional port)
+    if match := re.match(
+        r"(https?://[\w.-]+(?::\d+)?/)?(?P<full_name>[\w.-]+/[\w.-]+)/?$",
         repository_url,
     ):
         return match.group("full_name")

tests/test_utils.py

@@ -0,0 +1,47 @@
+import pytest
+
+from pytest_mergify.utils import get_repository_name_from_url
+
+
+@pytest.mark.parametrize(
+    "url,expected",
+    [
+        ("https://github.com/owner/repo", "owner/repo"),
+        ("https://github.com/owner/repo/", "owner/repo"),
+        ("http://github.com/owner/repo", "owner/repo"),
+        ("https://gitlab.com/owner/repo", "owner/repo"),
+        ("https://git.example.com/owner/repo", "owner/repo"),
+        ("owner/repo", "owner/repo"),
+        ("https://github.com/my-org.name/my-repo.name", "my-org.name/my-repo.name"),
+        ("https://git.example.com:8080/owner/repo", "owner/repo"),
+        ("https://github.com/owner123/repo456", "owner123/repo456"),
+        ("git@github.com:owner/repo.git", "owner/repo"),
+        ("git@github.com:owner/repo", "owner/repo"),
+        ("git@gitlab.com:owner/repo.git", "owner/repo"),
+        (
+            "git@git.example.com:my-org.name/my-repo.name.git",
+            "my-org.name/my-repo.name",
+        ),
+        ("git@bitbucket.org:owner123/repo456.git", "owner123/repo456"),
+    ],
+)
+def test_get_repository_name_from_url_valid(url: str, expected: str) -> None:
+    """Test valid URL formats that should extract repository names."""
+    result = get_repository_name_from_url(url)
+    assert result == expected
+
+
+@pytest.mark.parametrize(
+    "url",
+    [
+        "https://github.com/owner/repo/issues",
+        "https://github.com/owner",
+        "",
+        "not-a-url",
+        "https://github.com/owner/repo?tab=readme",
+    ],
+)
+def test_get_repository_name_from_url_invalid(url: str) -> None:
+    """Test invalid URL formats that should return None."""
+    result = get_repository_name_from_url(url)
+    assert result is None