From 114803bc9d8ca0947ed4e15b3da05f4fd846b5c0 Mon Sep 17 00:00:00 2001 From: Dannon Baker Date: Mon, 22 Apr 2024 21:32:31 -0400 Subject: [PATCH] Force MessageExceptions used in workflow modules to have a string error message. --- lib/galaxy/workflow/modules.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/galaxy/workflow/modules.py b/lib/galaxy/workflow/modules.py index ba0692c77d5..6113473c95f 100644 --- a/lib/galaxy/workflow/modules.py +++ b/lib/galaxy/workflow/modules.py @@ -2557,11 +2557,13 @@ def populate_module_and_state( step_args = param_map.get(step.id, {}) step_errors = module_injector.compute_runtime_state(step, step_args=step_args) if step_errors: - raise exceptions.MessageException(step_errors, err_data={step.order_index: step_errors}) + raise exceptions.MessageException( + "Error computing workflow step runtime state", err_data={step.order_index: step_errors} + ) if step.upgrade_messages: if allow_tool_state_corrections: log.debug('Workflow step "%i" had upgrade messages: %s', step.id, step.upgrade_messages) else: raise exceptions.MessageException( - step.upgrade_messages, err_data={step.order_index: step.upgrade_messages} + "Workflow step has upgrade messages", err_data={step.order_index: step.upgrade_messages} )