From 38a9f9c998530f989e94c4d7f27674950436d5ea Mon Sep 17 00:00:00 2001
From: Alban Auzeill
Date: Thu, 19 Dec 2019 20:03:16 +0100
Subject: Remove ugly exception stack traces during build

---
 sonar-css-plugin/pom.xml                                   | 14 ++++++++++++++
 .../test/java/org/sonar/css/plugin/CssRuleSensorTest.java  |  4 ++++
 .../org/sonar/css/plugin/StylelintReportSensorTest.java    |  4 ++++
 .../css/plugin/server/CssAnalyzerBridgeServerTest.java     | 13 +++++++++----
 4 files changed, 31 insertions(+), 4 deletions(-)

(limited to 'sonar-css-plugin')

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;
@@ -58,6 +59,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();
 
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;
 
@@ -52,6 +53,9 @@ public class StylelintReportSensorTest {
   @Rule
   public TemporaryFolder tmpDir = new TemporaryFolder();
 
+  @Rule
+  public final LogbackSuppressorRule logSuppressor = new LogbackSuppressorRule();
+
   @Rule
   public final LogTester logTester = new LogTester();
 
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;
-- 
cgit v1.2.3