Check that status and resolution maps are provided

Author: leplatrem

config/config.nonprod.yaml

@@ -14,14 +14,12 @@
         - add_link_to_bugzilla
         - add_link_to_jira
         - maybe_assign_jira_user
-        - maybe_update_issue_status
         - maybe_update_issue_severity
         - sync_whiteboard_labels
         - sync_keywords_labels
       existing:
         - update_issue_summary
         - maybe_assign_jira_user
-        - maybe_update_issue_status
         - maybe_update_issue_severity
         - sync_whiteboard_labels
         - sync_keywords_labels

config/config.prod.yaml

@@ -539,12 +539,10 @@
         - add_link_to_jira
         - maybe_assign_jira_user
         - maybe_update_issue_status
-        - maybe_update_issue_resolution
       existing:
         - update_issue_summary
         - maybe_assign_jira_user
         - maybe_update_issue_status
-        - maybe_update_issue_resolution
       comment:
         - create_comment
     status_map:

jbi/models.py

@@ -203,6 +203,16 @@ def validate_actions(cls, actions: list[Action]):
                     f"Provide bugzilla_user_id data for `{action.whiteboard_tag}` action."
                 )
 
+            assert action.parameters.status_map or (
+                "maybe_update_issue_status" not in action.parameters.steps.new
+                and "maybe_update_issue_status" not in action.parameters.steps.existing
+            ), "`maybe_update_issue_status` was used without `status_map`"
+            assert action.parameters.resolution_map or (
+                "maybe_update_issue_resolution" not in action.parameters.steps.new
+                and "maybe_update_issue_resolution"
+                not in action.parameters.steps.existing
+            ), "`maybe_update_issue_resolution` was used without `resolution_map`"
+
         return actions
 
     model_config = ConfigDict(ignored_types=(functools.cached_property,))