ultralytics-actions 0.0.36__tar.gz → 0.0.37__tar.gz

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.37}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.36
+Version: 0.0.37
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.37}/actions/__init__.py

@@ -22,4 +22,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.0.36"
+__version__ = "0.0.37"

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.37}/actions/utils/common_utils.py

@@ -50,7 +50,8 @@ def is_url(url, check=True, max_attempts=3, timeout=2):
 
         # Check structure
         result = parse.urlparse(url)
-        if not all([result.scheme, result.netloc]):
+        partition = result.netloc.partition(".")  # i.e. netloc = "github.com" -> ("github", ".", "com")
+        if not result.scheme or not partition[0] or not partition[2]:
             return False
 
         # Check response
@@ -82,7 +83,7 @@ def check_links_in_string(text, verbose=True, return_bad=False):
         r"("  # Start capturing group for plaintext URLs
         r"(?:https?://)?"  # Optional http:// or https://
         r"(?:www\.)?"  # Optional www.
-        r"[\w.-]+"  # Domain name and subdomains
+        r"(?:[\w.-]+)?"  # Optional domain name and subdomains
         r"\.[a-zA-Z]{2,}"  # TLD
         r"(?:/[^\s\"')\]]*)?"  # Optional path
         r")"

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.37}/pyproject.toml

@@ -95,6 +95,10 @@ packages = { find = { where = ["."], include = ["actions", "actions.*"] } }
 [tool.setuptools.dynamic]
 version = { attr = "actions.__version__" }
 
+[tool.pytest.ini_options]
+addopts = "--doctest-modules --durations=30 --color=yes"
+norecursedirs = [".git", "dist", "build"]
+
 [tool.ruff]
 line-length = 120
 

ultralytics_actions-0.0.37/tests/test_urls.py

@@ -0,0 +1,98 @@
+# Ultralytics Actions 🚀, AGPL-3.0 license
+# Continuous Integration (CI) GitHub Actions tests
+
+import pytest
+
+from actions.utils.common_utils import check_links_in_string, is_url
+
+URLS = [
+    "https://docs.ultralytics.com/help/CLA/",
+    "https://docs.ultralytics.com/help/contributing",
+    "https://docs.ultralytics.com",
+    "https://ultralytics.com",
+    "https://github.com/ultralytics/ultralytics",
+]
+
+
+@pytest.fixture
+def verbose():
+    """Fixture that provides a verbose logging utility for detailed output during testing and debugging."""
+    return False  # Set False to suppress print statements during tests
+
+
+def test_is_url():
+    """Test each URL using is_url function."""
+    for url in URLS:
+        assert is_url(url), f"URL check failed: {url}"
+
+
+def test_html_links(verbose):
+    """Tests the validity of URLs within HTML anchor tags and returns any invalid URLs found."""
+    text = "Visit <a href='https://err.com'>our site</a> or <a href=\"http://test.org\">test site</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_markdown_links(verbose):
+    """Validates URLs in markdown links within a given text using check_links_in_string."""
+    text = "Check [Example](https://err.com) or [Test](http://test.org)"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_mixed_formats(verbose):
+    """Tests URL detection in mixed text formats (HTML, Markdown, plain text) using check_links_in_string."""
+    text = "A <a href='https://1.com'>link</a> and [markdown](https://2.org) and https://3.net"
+    result, urls = check_links_in_string(text, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://1.com", "https://2.org", "https://3.net"}
+
+
+def test_duplicate_urls(verbose):
+    """Tests detection of duplicate URLs in various text formats using the check_links_in_string function."""
+    text = "Same URL: https://err.com and <a href='https://err.com'>link</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}
+
+
+def test_no_urls(verbose):
+    """Tests that a string with no URLs returns True when checked using the check_links_in_string function."""
+    text = "This text contains no URLs."
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is True
+    assert not set(urls)
+
+
+def test_invalid_urls(verbose):
+    """Test invalid URLs."""
+    text = "Invalid URL: http://.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"http://.com"}
+
+
+def test_urls_with_paths_and_queries(verbose):
+    """Test URLs with paths and query parameters to ensure they are correctly identified and validated."""
+    text = "Complex URL: https://err.com/path?query=value#fragment"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com/path?query=value#fragment"}
+
+
+def test_urls_with_different_tlds(verbose):
+    """Test URLs with various top-level domains (TLDs) to ensure correct identification and handling."""
+    text = "Different TLDs: https://err.ml https://err.org https://err.net https://err.io https://err.ai"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.ml", "https://err.org", "https://err.net", "https://err.io", "https://err.ai"}
+
+
+def test_case_sensitivity(verbose):
+    """Tests URL case sensitivity by verifying that URLs with different cases are correctly identified and handled."""
+    text = "Case test: HTTPS://err.com and https://err.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}

{ultralytics_actions-0.0.36 → ultralytics_actions-0.0.37}/ultralytics_actions.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.36
+Version: 0.0.37
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>

ultralytics_actions-0.0.36/tests/test_urls.py

@@ -1,15 +0,0 @@
-from actions.utils.common_utils import is_url
-
-URLS = [
-    "https://docs.ultralytics.com/help/CLA/",
-    "https://docs.ultralytics.com/help/contributing",
-    "https://docs.ultralytics.com",
-    "https://ultralytics.com",
-    "https://github.com/ultralytics/ultralytics",
-]
-
-
-def test_urls():
-    """Test each URL using is_url function."""
-    for url in URLS:
-        assert is_url(url), f"URL check failed: {url}"