Log the filename for which an issue is saved

SonarSource · Aug 25, 2023 · d2579e5 · d2579e5
1 parent 74a17ae
commit d2579e5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/...avascript-plugin/src/main/java/org/sonar/plugins/javascript/eslint/AnalysisProcessor.java b/...avascript-plugin/src/main/java/org/sonar/plugins/javascript/eslint/AnalysisProcessor.java
@@ -175,7 +175,12 @@ private void processParsingError(BridgeServer.ParsingError parsingError) {
 
   private void saveIssues(List<Issue> issues) {
     for (Issue issue : issues) {
-      LOG.debug("Saving issue for rule {} on line {}", issue.ruleId, issue.line);
+      LOG.debug(
+        "Saving issue for rule {} on file {} at line {}",
+        issue.ruleId,
+        file.absolutePath(),
+        issue.line
+      );
       saveIssue(issue);
     }
   }

diff --git a/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/eslint/JsTsSensorTest.java b/...r-javascript-plugin/src/test/java/org/sonar/plugins/javascript/eslint/JsTsSensorTest.java
@@ -183,6 +183,13 @@ void should_analyse() throws Exception {
     sensor.execute(context);
     verify(bridgeServerMock, times(1)).initLinter(any(), any(), any(), any());
     assertThat(context.allIssues()).hasSize(expectedResponse.issues.size());
+    assertThat(logTester.logs(LoggerLevel.ERROR))
+      .contains(
+        String.format(
+          "Saving issue for rule no-all-duplicated-branches on file {} at line 1",
+          inputFile.absolutePath()
+        )
+      );
 
     Iterator<Issue> issues = context.allIssues().iterator();
     Issue firstIssue = issues.next();