Skip to content

Commit

Permalink
New linter for stdio tags.
Browse files Browse the repository at this point in the history
  • Loading branch information
jmchilton committed Mar 16, 2015
1 parent d35a9df commit 8207026
Showing 1 changed file with 56 additions and 0 deletions.
56 changes: 56 additions & 0 deletions planemo_ext/galaxy/tools/linters/stdio.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@


def lint_stdio(tool_xml, lint_ctx):
stdios = tool_xml.findall("./stdio")
if not stdios:
lint_ctx.info("No stdio definition found, tool will determines error from stderr.")
return

if len(stdios) > 1:
lint_ctx.error("More than one stdio tag found, behavior undefined.")
return

stdio = stdios[0]
for child in list(stdio):
if child.tag == "regex":
_lint_regex(child, lint_ctx)
elif child.tag == "exit_code":
_lint_exit_code(child, lint_ctx)
else:
message = "Unknown stdio child tag discovered [%s]. "
message += "Valid options are exit_code and regex."
lint_ctx.warn(message % child.tag)


def _lint_exit_code(child, lint_ctx):
for key, value in child.attrib.iteritems():
if key == "range":
# TODO: validate
pass
elif key == "level":
_lint_level(value, lint_ctx)
elif key == "description":
pass
else:
lint_ctx.warn("Unknown attribute [%s] encountered on exit_code tag." % key)


def _lint_regex(child, lint_ctx):
for key, value in child.attrib.iteritems():
if key == "source":
if value not in ["stderr", "stdout", "both"]:
lint_ctx.error("Unknown error code level encountered [%s]" % value)
elif key == "level":
_lint_level(value, lint_ctx)
elif key == "match":
# TODO: validate
pass
elif key == "description":
pass
else:
lint_ctx.warn("Unknown attribute [%s] encountered on regex tag." % key)


def _lint_level(level_value, lint_ctx):
if level_value not in ["warning", "fatal", "log"]:
lint_ctx.error("Unknown error code level encountered [%s]" % level_value)

0 comments on commit 8207026

Please sign in to comment.