diff options
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; | 
