Merge "Revert "tox: update lint regex to not require column""

roles/tox/library/test-cases/ansible-lint_output.yaml

@@ -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"

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: {}

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):
+def pep8_matcher(line):
-    return simple_matcher(line, LINT_RE, 1, 2, 5)
+    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,
 ]