strip text

otherwise the parser gives an indentation error also: we strip the text alo in the application
galaxyproject · Nov 26, 2024 · a587336 · a587336
1 parent 9cae33f
commit a587336
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/galaxy/tool_util/linters/output.py b/lib/galaxy/tool_util/linters/output.py
@@ -85,7 +85,7 @@ def lint(cls, tool_source: "ToolSource", lint_ctx: "LintContext"):
             return
         for filter in tool_xml.findall("./outputs/*/filter"):
             try:
-                ast.parse(filter.text, mode="eval")
+                ast.parse(filter.text.strip(), mode="eval")
             except Exception as e:
                 lint_ctx.warn(
                     f"Filter '{filter.text}' is no valid expression: {str(e)}",

diff --git a/test/unit/tool_util/test_tool_linters.py b/test/unit/tool_util/test_tool_linters.py
@@ -656,7 +656,7 @@
     <outputs>
         <data name="another_valid_name" format="fasta" label="a label">
             <filter>an invalid condition</filter>
-            <filter>an and condition</filter>
+            <filter> an and condition </filter>
         </data>
         <collection name="yet_another_valid_name" type="list" format="fasta" label="another label">
             <filter>another invalid condition</filter>