fixed yamllint parser

README.md

@@ -53,7 +53,7 @@ It supports:
  * [_Simian_](http://www.harukizaemon.com/simian/)
  * [_StyleCop_](https://stylecop.codeplex.com/)
  * [_XMLLint_](http://xmlsoft.org/xmllint.html)
- * [_YAMLLint_](https://yamllint.readthedocs.io/en/stable/index.html)
+ * [_YAMLLint_](https://yamllint.readthedocs.io/en/stable/index.html) with `-f parsable`
  * [_ZPTLint_](https://pypi.python.org/pypi/zptlint)
 
 Example reports are available [in the test resources](https://github.com/tomasbjerre/violations-lib/tree/master/src/test/resources), examples of how to generate them are available [here](https://github.com/tomasbjerre/violations-test/blob/master/build.sh).

src/main/java/se/bjurr/violations/lib/parsers/YAMLlintParser.java

@@ -6,6 +6,7 @@
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
 import static se.bjurr.violations.lib.reports.Parser.YAMLLINT;
+import static se.bjurr.violations.lib.util.Utils.isNullOrEmpty;
 import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
@@ -24,14 +25,17 @@ public class YAMLlintParser implements ViolationsParser {
   public List<Violation> parseReportOutput(final String string) throws Exception {
     List<Violation> violations = new ArrayList<>();
     List<List<String>> partsPerLine =
-        getLines(string, "([^:]*):(\\d+):(\\d+):\\s?\\[(error|warning)\\]([^\\(]+)\\(([^\\)]+)\\)");
+        getLines(string, "([^:]*):(\\d+):(\\d+):\\s?\\[(error|warning)\\]([^\\(]+)(\\(([^\\)]+)\\))?");
     for (List<String> parts : partsPerLine) {
       String filename = parts.get(1).trim();
       Integer line = parseInt(parts.get(2));
       Integer column = parseInt(parts.get(3));
       String severity = parts.get(4).trim();
       String message = parts.get(5).trim();
-      String rule = parts.get(6).trim();
+      String rule = null;
+      if (!isNullOrEmpty(parts.get(7))) {
+        rule = parts.get(7).trim();
+      }
       violations.add( //
           violationBuilder() //
               .setParser(YAMLLINT) //

src/test/java/se/bjurr/violations/lib/YAMLlintTest.java

@@ -41,7 +41,16 @@ public void testThatViolationsCanBeParsed() {
     assertThat(violation1.getSeverity()) //
         .isEqualTo(ERROR);
 
+    final Violation violation2 = actual.get(3);
+    assertThat(violation2.getMessage()) //
+            .isEqualTo("syntax error: expected '<document start>', but found '<block mapping start>'");
+    assertThat(violation2.getFile()) //
+            .isEqualTo("./molecule-default/create.yml");
+    assertThat(violation2.getSeverity()) //
+            .isEqualTo(ERROR);
+    assertThat(violation2.getRule()).isEmpty();
+
     assertThat(actual) //
-        .hasSize(3);
+        .hasSize(4);
   }
 }

src/test/resources/yamllint/yamllint.txt

@@ -1,3 +1,4 @@
 file.yml:6:2: [warning] missing starting space in comment (comments)
 test/file.yml:57:1: [error] trailing spaces (trailing-spaces)
-file.yml:60:3: [error] wrong indentation: expected 4 but found 2 (indentation)
+file.yml:60:3: [error] wrong indentation: expected 4 but found 2 (indentation)
+./molecule-default/create.yml:117:89: [error] syntax error: expected '<document start>', but found '<block mapping start>'