diff --git a/roles/tox/library/test-cases/ansible-lint_output.yaml b/roles/tox/library/test-cases/ansible-lint_output.yaml deleted file mode 100644 index 0d0e6ad58..000000000 --- a/roles/tox/library/test-cases/ansible-lint_output.yaml +++ /dev/null @@ -1,8 +0,0 @@ -workdir: . -output: | - roles/tox/tasks/main.yaml:41: [EZUULJOBS0001] Loop vars should have zj_ prefix - -comments: - roles/tox/tasks/main.yaml: - - line: 41 - message: "[EZUULJOBS0001] Loop vars should have zj_ prefix" diff --git a/roles/tox/library/test-cases/warning_output.yaml b/roles/tox/library/test-cases/warning_output.yaml new file mode 100644 index 000000000..4d0cebd94 --- /dev/null +++ b/roles/tox/library/test-cases/warning_output.yaml @@ -0,0 +1,10 @@ +--- +workdir: . +output: | + tests/test_discovery.py:25: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details. + config = yaml.load(open('hiera/common.yaml', 'r')) + tests/test_discovery.py:714: UserWarning: Policy "foo:create_bar":"role:fizz" was deprecated in N in favor of "foo:create_bar":"". Reason: because of reasons. Either ensure your deployment is ready for the new default or copy/paste the deprecated policy into your policy file and maintain it manually. + warnings.warn(deprecated_msg) + WARNING [oslo_policy.policy] Policies ['foo', 'bar'] are part of a cyclical reference. + WARNING [oslo_policy.policy] Policies ['foo', 'bar'] are part of a cyclical reference. +comments: {} diff --git a/roles/tox/library/tox_parse_output.py b/roles/tox/library/tox_parse_output.py index 861a230e5..3a6d77c44 100644 --- a/roles/tox/library/tox_parse_output.py +++ b/roles/tox/library/tox_parse_output.py @@ -41,7 +41,8 @@ import re from ansible.module_utils.basic import AnsibleModule ANSI_RE = re.compile(r'(?:\x1B[@-_]|[\x80-\x9F])[0-?]*[ -/]*[@-~]') -LINT_RE = re.compile(r"^([^:]*):(\d+):((\d+):)?\s?(.*)$") +PEP8_RE = re.compile(r"^(.*):(\d+):(\d+): (.*)$") +SPHINX_RE = re.compile(r"^([^:]*):([\d]+):(\w.+)$") def simple_matcher(line, regex, file_path_group, start_line_group, @@ -57,12 +58,17 @@ def simple_matcher(line, regex, file_path_group, start_line_group, return file_path, start_line, message -def lint_matcher(line): - return simple_matcher(line, LINT_RE, 1, 2, 5) +def pep8_matcher(line): + return simple_matcher(line, PEP8_RE, 1, 2, 4) + + +def sphinx_matcher(line): + return simple_matcher(line, SPHINX_RE, 1, 2, 3) matchers = [ - lint_matcher, + pep8_matcher, + sphinx_matcher, ]