Add identifier to tests target overrides created in copr end handler (#…

…2661) Add identifier to tests target overrides created in copr end handler Reviewed-by: Laura Barcziová
packit · Nov 28, 2024 · 6fb1e2f · 6fb1e2f
2 parents daca72a + 03c0f11
commit 6fb1e2f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/packit_service/worker/handlers/copr.py b/packit_service/worker/handlers/copr.py
@@ -479,12 +479,13 @@ def handle_testing_farm(self):
                 and self.copr_event.chroot
                 in self.copr_build_helper.build_targets_for_test_job(job_config)
             ):
-                event_dict["tests_targets_override"] = list(
-                    self.copr_build_helper.build_target2test_targets_for_test_job(
+                event_dict["tests_targets_override"] = [
+                    (target, job_config.identifier or "")
+                    for target in self.copr_build_helper.build_target2test_targets_for_test_job(
                         self.copr_event.chroot,
                         job_config,
-                    ),
-                )
+                    )
+                ]
                 signature(
                     TaskName.testing_farm.value,
                     kwargs={

diff --git a/packit_service/worker/handlers/mixin.py b/packit_service/worker/handlers/mixin.py
@@ -432,7 +432,7 @@ def copr_build_helper(self) -> CoprBuildJobHelper:
         # when reporting state of SRPM build built in Copr
         build_targets_override = (
             {
-                (build.target, build.identifier)
+                (build.target, build.identifier or "")
                 for build in CoprBuildTargetModel.get_all_by_build_id(
                     str(self.copr_event.build_id),
                 )