Fix Koji artifact URLs to use build version and release (#5067)

Fixes #4887

Author: bajertom

tests/unit/artifact/__init__.py

@@ -40,7 +40,12 @@ def mock_api(method, *args, **kwargs):
         if method == "listBuildRPMs":
             return mock_rpms
         if method == "getBuild":
-            return {"id": MOCK_BUILD_ID_KOJI_BREW, "package_name": "test-package"}
+            return {
+                "id": MOCK_BUILD_ID_KOJI_BREW,
+                "package_name": "test-package",
+                "version": "1.0",
+                "release": "1.fc43",
+            }
         return None
 
     mock_call_api.side_effect = mock_api
@@ -85,7 +90,12 @@ def mock_api(method, *args, **kwargs):
         if method == "listBuildRPMs":
             return mock_rpms
         if method == "getBuild":
-            return {"id": MOCK_BUILD_ID_KOJI_BREW, "package_name": "test-package"}
+            return {
+                "id": MOCK_BUILD_ID_KOJI_BREW,
+                "package_name": "test-package",
+                "version": "1.0",
+                "release": "1.fc43",
+            }
         if method == "getTaskDescendents":
             task_id = args[0] if args else kwargs.get('taskID')
             return {str(task_id): None, str(task_id + 1): None}

tests/unit/artifact/test_brew.py

@@ -39,7 +39,7 @@ def test_brew_valid_draft_build(mock_brew, mock_call_api, artifact_provider):
     mock_call_api.side_effect = lambda method, *a, **kw: (
         mock_rpms
         if method == "listBuildRPMs"
-        else {"id": draft_id, "package_name": "test-package"}
+        else {"id": draft_id, "package_name": "test-package", "version": "1.0", "release": "1.el9"}
     )
 
     provider = artifact_provider(f"brew.build:{draft_id}")

tmt/steps/prepare/artifact/providers/koji.py

@@ -196,11 +196,13 @@ def make_rpm_artifact(self, rpm_meta: dict[str, Any]) -> ArtifactInfo:
         """
         assert self.build_info is not None
         package_name = self.build_info["package_name"]
+        build_version = self.build_info["version"]
+        build_release = self.build_info["release"]
         version_info = RpmVersion.from_rpm_meta(rpm_meta)
         path = (
             f"packages/{package_name}/"
-            f"{version_info.version}/"
-            f"{version_info.release}/"
+            f"{build_version}/"
+            f"{build_release}/"
             f"{version_info.arch}/"
             f"{version_info.nvra}.rpm"
         )