Resolve issue SonarSource#4586

ericmorand-sonarsource · Mar 19, 2024 · de0adb2 · de0adb2
1 parent fba5e23
commit de0adb2
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/...avascript-plugin/src/main/java/org/sonar/plugins/javascript/bridge/AnalysisProcessor.java b/...avascript-plugin/src/main/java/org/sonar/plugins/javascript/bridge/AnalysisProcessor.java
@@ -145,7 +145,7 @@ private void processParsingError(BridgeServer.ParsingError parsingError) {
     String message = parsingError.message;
 
     if (line != null) {
-      LOG.error("Failed to parse file [{}] at line {}: {}", file, line, message);
+      LOG.warn("Failed to parse file [{}] at line {}: {}", file, line, message);
     } else if (parsingError.code == BridgeServer.ParsingErrorCode.FAILING_TYPESCRIPT) {
       LOG.error("Failed to analyze file [{}] from TypeScript: {}", file, message);
     } else {

diff --git a/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/HtmlSensorTest.java b/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/HtmlSensorTest.java
@@ -216,7 +216,7 @@ void should_raise_a_parsing_error() throws IOException {
     assertThat(issue.primaryLocation().textRange().start().line()).isEqualTo(1);
     assertThat(issue.primaryLocation().message()).isEqualTo("Parse error message");
     assertThat(context.allAnalysisErrors()).hasSize(1);
-    assertThat(logTester.logs(LoggerLevel.ERROR))
+    assertThat(logTester.logs(LoggerLevel.WARN))
       .contains("Failed to parse file [dir/file.html] at line 1: Parse error message");
   }
 

diff --git a/...in/src/test/java/org/sonar/plugins/javascript/bridge/JavaScriptEslintBasedSensorTest.java b/...in/src/test/java/org/sonar/plugins/javascript/bridge/JavaScriptEslintBasedSensorTest.java
@@ -588,7 +588,7 @@ void should_raise_a_parsing_error() throws IOException {
     assertThat(issue.primaryLocation().textRange().start().line()).isEqualTo(3);
     assertThat(issue.primaryLocation().message()).isEqualTo("Parse error message");
     assertThat(context.allAnalysisErrors()).hasSize(1);
-    assertThat(logTester.logs(LoggerLevel.ERROR))
+    assertThat(logTester.logs(LoggerLevel.WARN))
       .contains("Failed to parse file [dir/file.js] at line 3: Parse error message");
   }
 
@@ -615,7 +615,7 @@ void should_not_create_parsing_issue_when_no_rule() throws IOException {
     Collection<Issue> issues = context.allIssues();
     assertThat(issues).isEmpty();
     assertThat(context.allAnalysisErrors()).hasSize(1);
-    assertThat(logTester.logs(LoggerLevel.ERROR))
+    assertThat(logTester.logs(LoggerLevel.WARN))
       .contains("Failed to parse file [dir/file.js] at line 3: Parse error message");
   }
 

diff --git a/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/JsTsSensorTest.java b/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/JsTsSensorTest.java
@@ -286,7 +286,7 @@ void should_raise_a_parsing_error() throws IOException {
     assertThat(issue.primaryLocation().textRange().start().line()).isEqualTo(3);
     assertThat(issue.primaryLocation().message()).isEqualTo("Parse error message");
     assertThat(context.allAnalysisErrors()).hasSize(1);
-    assertThat(logTester.logs(LoggerLevel.ERROR))
+    assertThat(logTester.logs(LoggerLevel.WARN))
       .contains("Failed to parse file [dir/file.ts] at line 3: Parse error message");
   }
 

diff --git a/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/YamlSensorTest.java b/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/bridge/YamlSensorTest.java
@@ -182,7 +182,7 @@ void should_raise_a_parsing_error() throws IOException {
     assertThat(issue.primaryLocation().textRange().start().line()).isEqualTo(1);
     assertThat(issue.primaryLocation().message()).isEqualTo("Parse error message");
     assertThat(context.allAnalysisErrors()).hasSize(1);
-    assertThat(logTester.logs(LoggerLevel.ERROR))
+    assertThat(logTester.logs(LoggerLevel.WARN))
       .contains("Failed to parse file [dir/file.yaml] at line 1: Parse error message");
   }