Rename and return a list from lookup_actions

Author: Dexterp37

jbi/models.py

@@ -238,7 +238,7 @@ class RunnerContext(Context, extra="forbid"):
 
     operation: Operation
     event: WebhookEvent
-    action: Optional[Action] = None
+    actions: Optional[list[Action]] = None
     bug: BugId | Bug
 
 

jbi/runner.py

@@ -58,9 +58,9 @@ def groups2operation(steps: ActionSteps):
     return by_operation
 
 
-def lookup_action(bug: bugzilla_models.Bug, actions: Actions) -> Action:
+def lookup_actions(bug: bugzilla_models.Bug, actions: Actions) -> list[Action]:
     """
-    Find first matching action from bug's whiteboard field.
+    Find matching actions from bug's whiteboard field.
 
     Tags are strings between brackets and can have prefixes/suffixes
     using dashes (eg. ``[project]``, ``[project-moco]``, ``[project-moco-sprint1]``).
@@ -71,7 +71,7 @@ def lookup_action(bug: bugzilla_models.Bug, actions: Actions) -> Action:
             # [tag-word], [tag-], [tag], but not [word-tag] or [tagword]
             search_string = r"\[" + tag + r"(-[^\]]*)*\]"
             if re.search(search_string, bug.whiteboard, flags=re.IGNORECASE):
-                return action
+                return [action]
 
     raise ActionNotFoundError(", ".join(actions.by_tag.keys()))
 
@@ -221,7 +221,7 @@ def execute_action(
             raise IgnoreInvalidRequestError("private bugs are not supported")
 
         try:
-            action = lookup_action(bug, actions)
+            action = lookup_actions(bug, actions)[0]
         except ActionNotFoundError as err:
             raise IgnoreInvalidRequestError(
                 f"no bug whiteboard matching action tags: {err}"
@@ -240,7 +240,7 @@ def execute_action(
 
         runner_context = runner_context.update(bug=bug)
 
-        runner_context = runner_context.update(action=action)
+        runner_context = runner_context.update(actions=[action])
 
         linked_issue_key: Optional[str] = bug.extract_from_see_also(
             project_key=action.jira_project_key

tests/unit/test_runner.py

@@ -15,7 +15,7 @@
     Executor,
     execute_action,
     execute_or_queue,
-    lookup_action,
+    lookup_actions,
 )
 
 
@@ -187,7 +187,7 @@ def test_action_is_logged_as_success_if_returns_true(
         ("Action 'devtest' executed successfully for Bug 654321", Operation.SUCCESS),
     ]
     assert capturelogs.records[-1].bug["id"] == 654321
-    assert capturelogs.records[-1].action["whiteboard_tag"] == "devtest"
+    assert capturelogs.records[-1].actions[0]["whiteboard_tag"] == "devtest"
 
 
 def test_action_is_logged_as_ignore_if_returns_false(
@@ -619,7 +619,7 @@ def test_counter_is_incremented_for_attachment(
 )
 def test_lookup_action_found(whiteboard, actions, bug_factory):
     bug = bug_factory(id=1234, whiteboard=whiteboard)
-    action = lookup_action(bug, actions)
+    action = lookup_actions(bug, actions)[0]
     assert action.whiteboard_tag == "devtest"
     assert "test config" in action.description
 
@@ -649,5 +649,5 @@ def test_lookup_action_found(whiteboard, actions, bug_factory):
 def test_lookup_action_not_found(whiteboard, actions, bug_factory):
     bug = bug_factory(id=1234, whiteboard=whiteboard)
     with pytest.raises(ActionNotFoundError) as exc_info:
-        lookup_action(bug, actions)
+        lookup_actions(bug, actions)[0]
     assert str(exc_info.value) == "devtest"