Not fail analysis when syntax error (#80)

1 files changed, 28 insertions, 17 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 da781aa..ce98ede 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
@@ -93,7 +93,7 @@ public class CssRuleSensor implements Sensor {
 
       try (InputStreamReader inputStreamReader = new InputStreamReader(process.getInputStream(), StandardCharsets.UTF_8)) {
         IssuesPerFile[] issues = new Gson().fromJson(inputStreamReader, IssuesPerFile[].class);
-        saveIssues(context, cssRules, issues);
+        saveIssues(context, issues);
       }
 
     } catch (IOException e) {
@@ -155,7 +155,7 @@ public class CssRuleSensor implements Sensor {
     }
   }
 
-  private static void saveIssues(SensorContext context, CssRules cssRules, IssuesPerFile[] issues) {
+  private void saveIssues(SensorContext context, IssuesPerFile[] issues) {
     FileSystem fileSystem = context.fileSystem();
 
     for (IssuesPerFile issuesPerFile : issues) {
@@ -163,24 +163,35 @@ public class CssRuleSensor implements Sensor {
 
       if (inputFile != null) {
         for (Issue issue : issuesPerFile.warnings) {
-          NewIssue sonarIssue = context.newIssue();
-
-          NewIssueLocation location = sonarIssue.newLocation()
-            .on(inputFile)
-            .at(inputFile.selectLine(issue.line))
-            .message(normalizeMessage(issue.text));
-
-          RuleKey ruleKey = cssRules.getActiveSonarKey(issue.rule);
-          if (ruleKey == null) {
-            throw new IllegalStateException("Unknown stylelint rule or rule not enabled " + issue.rule);
-          }
-          sonarIssue
-            .at(location)
-            .forRule(ruleKey)
-            .save();
+          saveIssue(context, inputFile, issue);
         }
       }
     }
   }
 
+  private void saveIssue(SensorContext context, InputFile inputFile, Issue issue) {
+    NewIssue sonarIssue = context.newIssue();
+
+    RuleKey ruleKey = cssRules.getActiveSonarKey(issue.rule);
+
+    if (ruleKey == null) {
+      if ("CssSyntaxError".equals(issue.rule)) {
+        LOG.error("Failed to parse " + inputFile.uri());
+      } else {
+        LOG.error("Unknown stylelint rule or rule not enabled: '" + issue.rule + "'");
+      }
+
+    } else {
+      NewIssueLocation location = sonarIssue.newLocation()
+        .on(inputFile)
+        .at(inputFile.selectLine(issue.line))
+        .message(normalizeMessage(issue.text));
+
+      sonarIssue
+        .at(location)
+        .forRule(ruleKey)
+        .save();
+    }
+  }
+
 }