python-semantic-release 8.0.6__tar.gz → 8.0.8__tar.gz

{python-semantic-release-8.0.6/python_semantic_release.egg-info → python-semantic-release-8.0.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: python-semantic-release
-Version: 8.0.6
+Version: 8.0.8
 Summary: Automatic Semantic Versioning for Python projects
 Author-email: Rolf Erik Lekang <me@rolflekang.com>
 License: MIT

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-semantic-release"
-version = "8.0.6"
+version = "8.0.8"
 description = "Automatic Semantic Versioning for Python projects"
 requires-python = ">=3.7"
 license = { text = "MIT" }
@@ -58,11 +58,12 @@ test = [
   "pytest-mock>=3,<4",
   "pytest-lazy-fixture~=0.6.3",
   "pytest-cov>=4,<5",
+  "pytest-pretty>=1.2.0,<2",
   "responses==0.23.3",
   "requests-mock>=1.10.0,<2",
   "types-pytest-lazy-fixture>=0.6.3.3",
 ]
-dev = ["pre-commit", "tox", "black", "ruff>=0.0.282"]
+dev = ["pre-commit", "tox", "black", "ruff==0.0.286"]
 mypy = ["mypy", "types-requests"]
 
 [tool.pytest.ini_options]

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8/python_semantic_release.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: python-semantic-release
-Version: 8.0.6
+Version: 8.0.8
 Summary: Automatic Semantic Versioning for Python projects
 Author-email: Rolf Erik Lekang <me@rolflekang.com>
 License: MIT

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/python_semantic_release.egg-info/requires.txt

@@ -14,7 +14,7 @@ shellingham>=1.5.0.post1
 pre-commit
 tox
 black
-ruff>=0.0.282
+ruff==0.0.286
 
 [docs]
 Sphinx<=6.0.0
@@ -33,6 +33,7 @@ pytest-xdist<4,>=2
 pytest-mock<4,>=3
 pytest-lazy-fixture~=0.6.3
 pytest-cov<5,>=4
+pytest-pretty<2,>=1.2.0
 responses==0.23.3
 requests-mock<2,>=1.10.0
 types-pytest-lazy-fixture>=0.6.3.3

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/semantic_release/__init__.py

@@ -23,7 +23,7 @@ from semantic_release.version import (
     tags_and_versions,
 )
 
-__version__ = "8.0.6"
+__version__ = "8.0.8"
 
 
 def setup_hook(argv: list[str]) -> None:

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/semantic_release/cli/config.py

@@ -84,7 +84,7 @@ class ChangelogConfig(BaseModel):
 
 class BranchConfig(BaseModel):
     match: str = "(main|master)"
-    prerelease_token = "rc"  # noqa: S105
+    prerelease_token: str = "rc"  # noqa: S105
     prerelease: bool = False
 
 

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/semantic_release/helpers.py

@@ -67,7 +67,7 @@ def dynamic_import(import_path: str) -> Any:
     string
     """
     log.debug("Trying to import %s", import_path)
-    module_name, _, attr = import_path.split(":", maxsplit=1)
+    module_name, attr = import_path.split(":", maxsplit=1)
     module = importlib.import_module(module_name)
     return getattr(module, attr)
 

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/semantic_release/hvcs/github.py

@@ -36,6 +36,7 @@ class Github(HvcsBase):
 
     DEFAULT_DOMAIN = "github.com"
     DEFAULT_API_DOMAIN = "api.github.com"
+    DEFAULT_UPLOAD_DOMAIN = "uploads.github.com"
 
     def __init__(
         self,
@@ -59,6 +60,7 @@ class Github(HvcsBase):
         ).replace("https://", "")
 
         self.api_url = f"https://{self.hvcs_api_domain}"
+        self.upload_url = f"https://{self.DEFAULT_UPLOAD_DOMAIN}"
 
         self.token = token
         auth = None if not self.token else TokenAuth(self.token)
@@ -179,15 +181,19 @@ class Github(HvcsBase):
         return self.edit_release_notes(release_id, release_notes)
 
     @logged_function(log)
-    def asset_upload_url(self, release_id: str) -> str:
+    @suppress_not_found
+    def asset_upload_url(self, release_id: str) -> str | None:
         """
         Get the correct upload url for a release
         https://docs.github.com/en/enterprise-server@3.5/rest/releases/releases#get-a-release
         :param release_id: ID of the release to upload to
-        :return: URL found to upload for a release
+        :return: URL to upload for a release if found, else None
         """
         # https://docs.github.com/en/enterprise-server@3.5/rest/releases/assets#upload-a-release-asset
-        return f"{self.api_url}/repos/{self.owner}/{self.repo_name}/releases/{release_id}/assets"  # noqa: E501
+        response = self.session.get(
+            f"{self.api_url}/repos/{self.owner}/{self.repo_name}/releases/{release_id}",
+        )
+        return response.json().get("upload_url").replace("{?name,label}", "")
 
     @logged_function(log)
     def upload_asset(
@@ -202,6 +208,12 @@ class Github(HvcsBase):
         :return: The status of the request
         """
         url = self.asset_upload_url(release_id)
+        if url is None:
+            raise HTTPError(
+                "There is no associated url for uploading asset for release "
+                f"{release_id}. Release url: "
+                f"{self.api_url}/repos/{self.owner}/{self.repo_name}/releases/{release_id}"
+            )
         content_type = (
             mimetypes.guess_type(file, strict=False)[0] or "application/octet-stream"
         )

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/tests/unit/semantic_release/changelog/test_template.py

@@ -30,12 +30,12 @@ EXAMPLE_TEMPLATE_FORMAT_STR = """
             "comment_end_string": "#}",
         },
         {
-            "block_start_string": "{%",
-            "block_end_string": "%}",
+            "block_start_string": "{[",
+            "block_end_string": "]}",
             "variable_start_string": "{{",
             "variable_end_string": "}}",
-            "comment_start_string": "{#",
-            "comment_end_string": "#}",
+            "comment_start_string": "/*",
+            "comment_end_string": "*/",
         },
     ],
 )

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/tests/unit/semantic_release/hvcs/test_gitea.py

@@ -127,18 +127,6 @@ def test_gitea_get_repository_owner_and_name(default_gitea_client):
             "git@gitea.com:custom/example.git",
             "https://aabbcc@gitea.com/custom/example.git",
         ),
-        (
-            False,
-            "aabbcc",
-            "git@gitea.com:custom/example.git",
-            "git@gitea.com:custom/example.git",
-        ),
-        (
-            True,
-            "aabbcc",
-            "git@gitea.com:custom/example.git",
-            "https://aabbcc@gitea.com/custom/example.git",
-        ),
     ],
 )
 def test_remote_url(

{python-semantic-release-8.0.6 → python-semantic-release-8.0.8}/tests/unit/semantic_release/hvcs/test_github.py

@@ -213,17 +213,6 @@ def test_pull_request_url(default_gh_client, pr_number):
     )
 
 
-def test_asset_upload_url(default_gh_client):
-    assert default_gh_client.asset_upload_url(
-        release_id=420
-    ) == "https://{domain}/repos/{owner}/{repo}/releases/{release_id}/assets".format(
-        domain=default_gh_client.hvcs_api_domain,
-        owner=default_gh_client.owner,
-        repo=default_gh_client.repo_name,
-        release_id=420,
-    )
-
-
 ############
 # Tests which need http response mocking
 ############
@@ -231,6 +220,7 @@ def test_asset_upload_url(default_gh_client):
 
 github_matcher = re.compile(rf"^https://{Github.DEFAULT_DOMAIN}")
 github_api_matcher = re.compile(rf"^https://{Github.DEFAULT_API_DOMAIN}")
+github_upload_matcher = re.compile(rf"^https://{Github.DEFAULT_UPLOAD_DOMAIN}")
 
 
 @pytest.mark.parametrize("status_code", (200, 201))
@@ -533,6 +523,43 @@ def test_create_or_update_release_when_create_fails_and_no_release_for_tag(
         mock_edit_release_notes.assert_not_called()
 
 
+def test_asset_upload_url(default_gh_client):
+    release_id = 1
+    # '{?name,label}' are added by github.com at least, maybe custom too
+    # https://docs.github.com/en/rest/releases/releases?apiVersion=2022-11-28#get-a-release
+    resp_payload = {
+        "upload_url": (
+            f"{default_gh_client.upload_url}/repos/"
+            f"{default_gh_client.owner}/{default_gh_client.repo_name}/"
+            f"releases/{release_id}/"
+            "assets{?name,label}"
+        ),
+        "status": "success",
+    }
+    with requests_mock.Mocker(session=default_gh_client.session) as m:
+        m.register_uri("GET", github_api_matcher, json=resp_payload, status_code=200)
+        assert default_gh_client.asset_upload_url(
+            release_id
+        ) == "https://{domain}/repos/{owner}/{repo}/releases/{release_id}/assets".format(
+            domain=default_gh_client.DEFAULT_UPLOAD_DOMAIN,
+            owner=default_gh_client.owner,
+            repo=default_gh_client.repo_name,
+            release_id=release_id,
+        )
+        assert m.called
+        assert len(m.request_history) == 1
+        assert m.last_request.method == "GET"
+        assert (
+            m.last_request.url
+            == "{api_url}/repos/{owner}/{repo_name}/releases/{release_id}".format(
+                api_url=default_gh_client.api_url,
+                owner=default_gh_client.owner,
+                repo_name=default_gh_client.repo_name,
+                release_id=1,
+            )
+        )
+
+
 @pytest.mark.parametrize("status_code", (200, 201))
 @pytest.mark.parametrize("mock_release_id", range(3))
 def test_upload_asset_succeeds(
@@ -540,9 +567,21 @@ def test_upload_asset_succeeds(
 ):
     label = "abc123"
     urlparams = {"name": example_changelog_md.name, "label": label}
+    expected_upload_url = (
+        f"{default_gh_client.upload_url}/repos/{default_gh_client.owner}/"
+        f"{default_gh_client.repo_name}/releases/{mock_release_id}/"
+        r"assets{?name,label}"
+    )
+    json_get_up_url = {"status": "ok", "upload_url": expected_upload_url}
     with requests_mock.Mocker(session=default_gh_client.session) as m:
         m.register_uri(
-            "POST", github_api_matcher, json={"status": "ok"}, status_code=status_code
+            "POST",
+            github_upload_matcher,
+            json={"status": "ok"},
+            status_code=status_code,
+        )
+        m.register_uri(
+            "GET", github_api_matcher, json=json_get_up_url, status_code=status_code
         )
         assert (
             default_gh_client.upload_asset(
@@ -553,13 +592,17 @@ def test_upload_asset_succeeds(
             is True
         )
         assert m.called
-        assert len(m.request_history) == 1
-        assert m.last_request.method == "POST"
-        assert m.last_request.url == "{url}?{params}".format(
-            url=default_gh_client.asset_upload_url(mock_release_id),
+        assert len(m.request_history) == 2
+        get_req, post_req = m.request_history
+        assert isinstance(get_req, requests_mock.request._RequestObjectProxy)
+        assert isinstance(post_req, requests_mock.request._RequestObjectProxy)
+        assert get_req.method == "GET"
+
+        assert post_req.method == "POST"
+        assert post_req.url == "{url}?{params}".format(
+            url=expected_upload_url.replace(r"{?name,label}", ""),
             params=urlencode(urlparams),
         )
-
         # Check if content-type header was correctly set according to
         # mimetypes - not retesting guessing functionality
         assert {
@@ -567,8 +610,8 @@ def test_upload_asset_succeeds(
                 example_changelog_md.resolve(), strict=False
             )[0]
             or "application/octet-stream"
-        }.items() <= m.last_request.headers.items()
-        assert m.last_request.body == example_changelog_md.read_bytes()
+        }.items() <= post_req.headers.items()
+        assert post_req.body == example_changelog_md.read_bytes()
 
 
 @pytest.mark.parametrize("status_code", (400, 404, 429, 500, 503))
@@ -578,14 +621,23 @@ def test_upload_asset_fails(
 ):
     label = "abc123"
     urlparams = {"name": example_changelog_md.name, "label": label}
+    json_get_up_url = {
+        "status": "ok",
+        "upload_url": "{up_url}/repos/{owner}/{repo_name}/releases/{release_id}".format(
+            up_url=default_gh_client.upload_url,
+            owner=default_gh_client.owner,
+            repo_name=default_gh_client.repo_name,
+            release_id=mock_release_id,
+        ),
+    }
     with requests_mock.Mocker(session=default_gh_client.session) as m:
         m.register_uri(
             "POST",
-            github_api_matcher,
+            github_upload_matcher,
             json={"message": "error"},
             status_code=status_code,
         )
-
+        m.register_uri("GET", github_api_matcher, json=json_get_up_url, status_code=200)
         with pytest.raises(HTTPError):
             default_gh_client.upload_asset(
                 release_id=mock_release_id,
@@ -594,9 +646,10 @@ def test_upload_asset_fails(
             )
 
         assert m.called
-        assert len(m.request_history) == 1
-        assert m.last_request.method == "POST"
-        assert m.last_request.url == "{url}?{params}".format(
+        assert len(m.request_history) == 2
+        post_req = m.last_request.copy()
+        assert post_req.method == "POST"
+        assert post_req.url == "{url}?{params}".format(
             url=default_gh_client.asset_upload_url(mock_release_id),
             params=urlencode(urlparams),
         )
@@ -608,8 +661,8 @@ def test_upload_asset_fails(
                 example_changelog_md.resolve(), strict=False
             )[0]
             or "application/octet-stream"
-        }.items() <= m.last_request.headers.items()
-        assert m.last_request.body == example_changelog_md.read_bytes()
+        }.items() <= post_req.headers.items()
+        assert post_req.body == example_changelog_md.read_bytes()
 
 
 # Note - mocking as the logic for uploading an asset