diff options
| author | Elena Vilchik | 2019-06-07 14:40:01 +0200 |
|---|---|---|
| committer | Tibor Blenessy | 2019-06-07 14:40:01 +0200 |
| commit | ac5b495929efed77f9bd954bd34a34b934d8da3b (patch) | |
| tree | d1dfd893cc44226af9958f39d31977cb7dd08e8b /sonar-css-plugin | |
| parent | 3920def3a5882c9ebb9a4947d2c78df2fbd50693 (diff) | |
| download | sonar-css-ac5b495929efed77f9bd954bd34a34b934d8da3b.tar.bz2 | |
Add parse error text and position to the log (#172)
Diffstat (limited to 'sonar-css-plugin')
| -rw-r--r-- | sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java | 3 | ||||
| -rw-r--r-- | sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java index a4f8c9a..2b268cd 100644 --- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java @@ -219,7 +219,8 @@ public class CssRuleSensor implements Sensor { if (ruleKey == null) { if ("CssSyntaxError".equals(issue.rule)) { - LOG.error("Failed to parse " + inputFile.uri()); + String errorMessage = issue.text.replace("(CssSyntaxError)", "").trim(); + LOG.error("Failed to parse {}, line {}, {}", inputFile.uri(), issue.line, errorMessage); } else { LOG.error("Unknown stylelint rule or rule not enabled: '" + issue.rule + "'"); } diff --git a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java index 6637617..df3d034 100644 --- a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java +++ b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java @@ -239,7 +239,7 @@ public class CssRuleSensorTest { CssRuleSensor sensor = createCssRuleSensor(rulesExecution); sensor.execute(context); - assertThat(logTester.logs(LoggerLevel.ERROR)).contains("Failed to parse " + inputFile.uri()); + assertThat(logTester.logs(LoggerLevel.ERROR)).contains("Failed to parse " + inputFile.uri() + ", line 2, Missed semicolon"); } @Test |