Feat(jobs): Add retriggered Bodhi handler to jobs matching event

This is due to different job_config_trigger_types (pull_request/commit) in retrigerring bodhi via dist-git PR.
packit · Sep 14, 2022 · efa550d · efa550d
1 parent 8089789
commit efa550d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/packit_service/worker/jobs.py b/packit_service/worker/jobs.py
@@ -467,7 +467,7 @@ def is_project_public_or_enabled_private(self) -> bool:
 
         return True
 
-    def check_explicit_matching(self):
+    def check_explicit_matching(self) -> List[JobConfig]:
         """Force explicit event/jobs matching for triggers
 
         Returns:
@@ -477,7 +477,7 @@ def check_explicit_matching(self):
         if isinstance(self.event, PullRequestCommentPagureEvent):
             for job in self.event.package_config.jobs:
                 if (
-                    job.type == JobType.koji_build
+                    job.type in [JobType.koji_build, JobType.bodhi_update]
                     and job.trigger == JobConfigTriggerType.commit
                     and self.event.job_config_trigger_type
                     == JobConfigTriggerType.pull_request

diff --git a/tests/unit/test_jobs.py b/tests/unit/test_jobs.py
@@ -2399,6 +2399,22 @@ def __init__(self):
                 ),
             ],
         ),
+        pytest.param(
+            PullRequestCommentPagureEvent,
+            JobConfigTriggerType.pull_request,
+            [
+                JobConfig(
+                    type=JobType.bodhi_update,
+                    trigger=JobConfigTriggerType.commit,
+                ),
+            ],
+            [
+                JobConfig(
+                    type=JobType.bodhi_update,
+                    trigger=JobConfigTriggerType.commit,
+                ),
+            ],
+        ),
     ],
 )
 def test_get_jobs_matching_trigger(event_kls, job_config_trigger_type, jobs, result):