diff --git a/templates/webapps/tool_shed/repository/display_tool_functional_test_results.mako b/templates/webapps/tool_shed/repository/display_tool_functional_test_results.mako
index 2102fa1c872..b2b13894b37 100644
--- a/templates/webapps/tool_shed/repository/display_tool_functional_test_results.mako
+++ b/templates/webapps/tool_shed/repository/display_tool_functional_test_results.mako
@@ -3,9 +3,37 @@
<%namespace file="/webapps/tool_shed/common/common.mako" import="*" />
<%namespace file="/webapps/tool_shed/repository/common.mako" import="*" />
+<%!
+ def inherit(context):
+ if context.get('use_panels'):
+ return '/webapps/tool_shed/base_panels.mako'
+ else:
+ return '/base.mako'
+%>
+
+<%inherit file="${inherit(context)}"/>
+
+<%def name="render_functional_test_text( functional_test_text )">
+ <%
+ from tool_shed.util.shed_util_common import to_safe_string
+ %>
+
+
+ | ${ to_safe_string( functional_test_text, to_html=True ) } |
+
+%def>
+
<%
from galaxy.web.framework.helpers import time_ago
- from tool_shed.util.shed_util_common import to_safe_string
changeset_revision = repository_metadata.changeset_revision
has_metadata = repository.metadata_revisions
@@ -44,15 +72,6 @@
browse_label = 'Browse repository tip files'
%>
-<%!
- def inherit(context):
- if context.get('use_panels'):
- return '/webapps/tool_shed/base_panels.mako'
- else:
- return '/base.mako'
-%>
-<%inherit file="${inherit(context)}"/>
-
%if is_new:
@@ -198,13 +217,8 @@
tool_id = test_results_dict.get( 'tool_id', 'unknown' )
tool_version = test_results_dict.get( 'tool_version', 'unknown' )
test_status = 'Test failed'
-
stderr = test_results_dict.get( 'stderr', '' )
- if stderr:
- stderr = to_safe_string( stderr, to_html=True )
traceback = test_results_dict.get( 'traceback', '' )
- if traceback:
- traceback = to_safe_string( traceback, to_html=True )
%>
| Tool id: ${tool_id} version: ${tool_version} |
@@ -219,11 +233,11 @@
| Stderr |
- ${stderr} |
+ ${render_functional_test_text( stderr )} |
| Traceback |
- ${traceback} |
+ ${render_functional_test_text( traceback )} |
%endfor
@@ -274,8 +288,6 @@
tool_id = test_results_dict.get( 'tool_id', None )
tool_version = test_results_dict.get( 'tool_version', None )
reason_test_is_invalid = test_results_dict.get( 'reason_test_is_invalid', None )
- if reason_test_is_invalid:
- reason_test_is_invalid = to_safe_string( reason_test_is_invalid, to_html=True )
%>
%if tool_id or tool_version:
@@ -285,7 +297,7 @@
%if reason_test_is_invalid:
| Reason test is invalid |
- ${reason_test_is_invalid} |
+ ${render_functional_test_text( reason_test_is_invalid )} |
%endif
%endfor