Rename remaining variables "sut" to "underTest" in tests

3 files changed, 32 insertions, 34 deletions

diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/BaseLoggerTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/BaseLoggerTest.java
index 5f34386d175..a2e44da3913 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/BaseLoggerTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/BaseLoggerTest.java
@@ -23,10 +23,8 @@ import org.junit.Test;
 
 public class BaseLoggerTest {
 
-  ConsoleLogger sut = new ConsoleLogger();
-
   @Test
   public void log() {
-    sut.log(LoggerLevel.WARN, "message");
+    new ConsoleLogger().log(LoggerLevel.WARN, "message");
   }
 }
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/LogTesterTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/LogTesterTest.java
index 9880b1ef13a..89e8e27e8dc 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/LogTesterTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/LogTesterTest.java
@@ -25,44 +25,44 @@ import static org.assertj.core.api.Assertions.assertThat;
 
 public class LogTesterTest {
 
-  LogTester sut = new LogTester();
+  LogTester underTest = new LogTester();
 
   @Test
   public void debugLevel() throws Throwable {
-    LoggerLevel initial = sut.getLevel();
+    LoggerLevel initial = underTest.getLevel();
 
     // when LogTester is used, then debug logs are enabled by default
-    sut.before();
-    assertThat(sut.getLevel()).isEqualTo(LoggerLevel.TRACE);
+    underTest.before();
+    assertThat(underTest.getLevel()).isEqualTo(LoggerLevel.TRACE);
     assertThat(Loggers.getFactory().getLevel()).isEqualTo(LoggerLevel.TRACE);
 
     // change
-    sut.setLevel(LoggerLevel.INFO);
-    assertThat(sut.getLevel()).isEqualTo(LoggerLevel.INFO);
+    underTest.setLevel(LoggerLevel.INFO);
+    assertThat(underTest.getLevel()).isEqualTo(LoggerLevel.INFO);
     assertThat(Loggers.getFactory().getLevel()).isEqualTo(LoggerLevel.INFO);
 
     // reset to initial level after execution of test
-    sut.after();
-    assertThat(sut.getLevel()).isEqualTo(initial);
+    underTest.after();
+    assertThat(underTest.getLevel()).isEqualTo(initial);
     assertThat(Loggers.getFactory().getLevel()).isEqualTo(initial);
   }
 
   @Test
   public void intercept_logs() throws Throwable {
-    sut.before();
+    underTest.before();
     Loggers.get("logger1").info("an information");
     Loggers.get("logger2").warn("warning: {}", 42);
 
-    assertThat(sut.logs()).containsExactly("an information", "warning: 42");
-    assertThat(sut.logs(LoggerLevel.ERROR)).isEmpty();
-    assertThat(sut.logs(LoggerLevel.INFO)).containsOnly("an information");
-    assertThat(sut.logs(LoggerLevel.WARN)).containsOnly("warning: 42");
+    assertThat(underTest.logs()).containsExactly("an information", "warning: 42");
+    assertThat(underTest.logs(LoggerLevel.ERROR)).isEmpty();
+    assertThat(underTest.logs(LoggerLevel.INFO)).containsOnly("an information");
+    assertThat(underTest.logs(LoggerLevel.WARN)).containsOnly("warning: 42");
 
-    sut.clear();
-    assertThat(sut.logs()).isEmpty();
-    assertThat(sut.logs(LoggerLevel.INFO)).isEmpty();
+    underTest.clear();
+    assertThat(underTest.logs()).isEmpty();
+    assertThat(underTest.logs(LoggerLevel.INFO)).isEmpty();
 
-    sut.after();
+    underTest.after();
     assertThat(LogInterceptors.get()).isSameAs(NullInterceptor.NULL_INSTANCE);
   }
 }
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/NullProfilerTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/NullProfilerTest.java
index bc9af01a006..563260bfa91 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/NullProfilerTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/utils/log/NullProfilerTest.java
@@ -25,24 +25,24 @@ import static org.assertj.core.api.Assertions.assertThat;
 
 public class NullProfilerTest {
 
-  NullProfiler sut = NullProfiler.NULL_INSTANCE;
+  NullProfiler underTest = NullProfiler.NULL_INSTANCE;
 
   @Test
   public void do_not_fail() {
-    assertThat(sut.start()).isSameAs(sut);
-    assertThat(sut.startTrace("")).isSameAs(sut);
-    assertThat(sut.startDebug("")).isSameAs(sut);
-    assertThat(sut.startInfo("")).isSameAs(sut);
+    assertThat(underTest.start()).isSameAs(underTest);
+    assertThat(underTest.startTrace("")).isSameAs(underTest);
+    assertThat(underTest.startDebug("")).isSameAs(underTest);
+    assertThat(underTest.startInfo("")).isSameAs(underTest);
 
-    assertThat(sut.stopTrace()).isSameAs(sut);
-    assertThat(sut.stopTrace("")).isSameAs(sut);
-    assertThat(sut.stopDebug("")).isSameAs(sut);
-    assertThat(sut.stopDebug()).isSameAs(sut);
-    assertThat(sut.stopInfo()).isSameAs(sut);
-    assertThat(sut.stopInfo("")).isSameAs(sut);
+    assertThat(underTest.stopTrace()).isSameAs(underTest);
+    assertThat(underTest.stopTrace("")).isSameAs(underTest);
+    assertThat(underTest.stopDebug("")).isSameAs(underTest);
+    assertThat(underTest.stopDebug()).isSameAs(underTest);
+    assertThat(underTest.stopInfo()).isSameAs(underTest);
+    assertThat(underTest.stopInfo("")).isSameAs(underTest);
 
-    assertThat(sut.isDebugEnabled()).isFalse();
-    assertThat(sut.isTraceEnabled()).isFalse();
-    assertThat(sut.addContext("foo", "bar")).isSameAs(sut);
+    assertThat(underTest.isDebugEnabled()).isFalse();
+    assertThat(underTest.isTraceEnabled()).isFalse();
+    assertThat(underTest.addContext("foo", "bar")).isSameAs(underTest);
   }
 }