Remove ugly exception stack traces during buildissue-225

4 files changed, 31 insertions, 4 deletions

diff --git a/sonar-css-plugin/pom.xml b/sonar-css-plugin/pom.xml
index a4e82e8..f856737 100644
--- a/sonar-css-plugin/pom.xml
+++ b/sonar-css-plugin/pom.xml
@@ -80,6 +80,20 @@
           <groupId>com.squareup.okhttp3</groupId>
           <artifactId>okhttp</artifactId>
         </dependency>
+
+        <!-- test dependencies to suppress console logging when tests succeed using LogbackSuppressorRule -->
+        <dependency>
+            <groupId>ch.qos.logback</groupId>
+            <artifactId>logback-classic</artifactId>
+            <version>1.2.3</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+          <groupId>info.novatec.testit</groupId>
+          <artifactId>testutils-logsuppressor-logback</artifactId>
+          <version>0.3.4</version>
+           <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>
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 0dd35d8..a23352f 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
@@ -45,6 +45,7 @@ import org.sonar.api.notifications.AnalysisWarnings;
 import org.sonar.api.utils.log.LogTester;
 import org.sonar.api.utils.log.LoggerLevel;
 import org.sonar.css.plugin.server.CssAnalyzerBridgeServer;
+import org.testit.testutils.logsuppressor.junit4.LogbackSuppressorRule;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
@@ -59,6 +60,9 @@ import static org.sonar.css.plugin.server.CssAnalyzerBridgeServerTest.createCssA
 public class CssRuleSensorTest {
 
   @Rule
+  public final LogbackSuppressorRule logSuppressor = new LogbackSuppressorRule();
+
+  @Rule
   public final LogTester logTester = new LogTester();
 
   @Rule
diff --git a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/StylelintReportSensorTest.java b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/StylelintReportSensorTest.java
index 20779b4..5edde2c 100644
--- a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/StylelintReportSensorTest.java
+++ b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/StylelintReportSensorTest.java
@@ -44,6 +44,7 @@ import org.sonar.api.rules.RuleType;
 import org.sonar.api.utils.Version;
 import org.sonar.api.utils.log.LogTester;
 import org.sonar.api.utils.log.LoggerLevel;
+import org.testit.testutils.logsuppressor.junit4.LogbackSuppressorRule;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -53,6 +54,9 @@ public class StylelintReportSensorTest {
   public TemporaryFolder tmpDir = new TemporaryFolder();
 
   @Rule
+  public final LogbackSuppressorRule logSuppressor = new LogbackSuppressorRule();
+
+  @Rule
   public final LogTester logTester = new LogTester();
 
   private static final File BASE_DIR = new File("src/test/resources/stylelint-report/").getAbsoluteFile();
diff --git a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/server/CssAnalyzerBridgeServerTest.java b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/server/CssAnalyzerBridgeServerTest.java
index 302b6bb..87c82d0 100644
--- a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/server/CssAnalyzerBridgeServerTest.java
+++ b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/server/CssAnalyzerBridgeServerTest.java
@@ -22,6 +22,7 @@ package org.sonar.css.plugin.server;
 import java.nio.file.Path;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
 import org.sonar.api.batch.fs.internal.DefaultInputFile;
@@ -37,6 +38,7 @@ import org.sonar.css.plugin.server.exception.ServerAlreadyFailedException;
 import org.sonarsource.nodejs.NodeCommand;
 import org.sonarsource.nodejs.NodeCommandBuilder;
 import org.sonarsource.nodejs.NodeCommandException;
+import org.testit.testutils.logsuppressor.junit4.LogbackSuppressorRule;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
@@ -51,13 +53,16 @@ public class CssAnalyzerBridgeServerTest {
   private static final String CONFIG_FILE = "config.json";
   private static final int TEST_TIMEOUT_SECONDS = 1;
 
-  @org.junit.Rule
-  public LogTester logTester = new LogTester();
+  @Rule
+  public final LogbackSuppressorRule logSuppressor = new LogbackSuppressorRule();
 
-  @org.junit.Rule
+  @Rule
+  public final LogTester logTester = new LogTester();
+
+  @Rule
   public final ExpectedException thrown = ExpectedException.none();
 
-  @org.junit.Rule
+  @Rule
   public JUnitTempFolder tempFolder = new JUnitTempFolder();
 
   private SensorContextTester context;