fixup! Fix Sonar analysis bug issue (#56)

Use SuppressWarnings instead of NOSONAR
adnovum · Feb 20, 2020 · e51e80c · e51e80c
1 parent d705e9c
commit e51e80c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/org/sonar/plugins/buildbreaker/QualityGateBreaker.java b/src/main/java/org/sonar/plugins/buildbreaker/QualityGateBreaker.java
@@ -175,6 +175,8 @@ private File getReportTaskFile() {
   }
 
   @VisibleForTesting
+  // We have to treat a premature interrupt as a failure since we couldn't retrieve the analysis id.
+  @SuppressWarnings("squid:S2142")
   String getAnalysisId(WsClient wsClient, String ceTaskId) {
     WsRequest ceTaskRequest =
         new GetRequest("api/ce/task").setParam("id", ceTaskId).setMediaType(MediaTypes.PROTOBUF);
@@ -203,8 +205,7 @@ String getAnalysisId(WsClient wsClient, String ceTaskId) {
             throw new IllegalStateException(
                 "Report processing did not complete successfully: " + taskStatus);
         }
-      } catch (IOException | InterruptedException e) { // NOSONAR: We have to treat a premature
-        // interrupt as a failure since we couldn't retrieve the analysis id (java:S2142)
+      } catch (IOException | InterruptedException e) {
         throw new IllegalStateException(e.getMessage(), e);
       }
     }