]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-8335 drop dead code ServerLogging#getCurrentLogFile
authorSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Tue, 29 Nov 2016 09:20:39 +0000 (10:20 +0100)
committerSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Thu, 1 Dec 2016 11:29:39 +0000 (12:29 +0100)
server/sonar-server/src/main/java/org/sonar/server/platform/ServerLogging.java
server/sonar-server/src/test/java/org/sonar/server/platform/ServerLoggingTest.java

index 2f996b7bca15b954753a3905075e7868e7026058..9dd8c447592a6306d842b0ca4890e83a9f55f794 100644 (file)
@@ -83,11 +83,4 @@ public class ServerLogging {
     return new File(settings.getString(ProcessProperties.PATH_LOGS));
   }
 
-  /**
-   * The file sonar.log, may not exist.
-   */
-  public File getCurrentLogFile() {
-    return new File(getLogsDir(), "sonar.log");
-  }
-
 }
index 04ed09576185d5a702530795a879af6e4f04e222..4092a0cce72a7c6cc9b0ee3864b58dbf374000f2 100644 (file)
@@ -22,7 +22,6 @@ package org.sonar.server.platform;
 import ch.qos.logback.classic.Level;
 import java.io.File;
 import java.io.IOException;
-import org.apache.commons.io.FileUtils;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
@@ -31,8 +30,8 @@ import org.sonar.api.config.MapSettings;
 import org.sonar.api.config.Settings;
 import org.sonar.api.utils.log.LogTester;
 import org.sonar.api.utils.log.LoggerLevel;
-import org.sonar.process.logging.LogbackHelper;
 import org.sonar.process.ProcessProperties;
+import org.sonar.process.logging.LogbackHelper;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.Mockito.mock;
@@ -67,16 +66,6 @@ public class ServerLoggingTest {
     assertThat(underTest.getRootLoggerLevel()).isEqualTo(LoggerLevel.TRACE);
   }
 
-  @Test
-  public void getCurrentLogFile() throws IOException {
-    File dir = temp.newFolder();
-    File logFile = new File(dir, "sonar.log");
-    FileUtils.touch(logFile);
-    settings.setProperty(ProcessProperties.PATH_LOGS, dir.getAbsolutePath());
-
-    assertThat(underTest.getCurrentLogFile()).isEqualTo(logFile);
-  }
-
   @Test
   public void configureHardcodedLevels() {
     LogbackHelper logbackHelper = mock(LogbackHelper.class);