[JENKINS-50291] Test showing the issue

Expected to fail on the second assertion
jenkinsci · Apr 5, 2018 · c0dc007 · c0dc007
1 parent daad7c0
commit c0dc007
Show file tree

Hide file tree

Showing 2 changed files with 64 additions and 1 deletion.
diff --git a/src/main/java/jenkins/metrics/api/Metrics.java b/src/main/java/jenkins/metrics/api/Metrics.java
@@ -548,7 +548,7 @@ public void run() {
                             if (jenkins == null) {
                                 return;
                             }
-                            File logFile = new File(new File(jenkins.getRootDir(), "logs"), "health-checker.log");
+                            File logFile = getLogFile(jenkins);
                             if (!logFile.isFile()) {
                                 File oldFile = new File(jenkins.getRootDir(), HealthChecker.class.getName() + ".log");
                                 if (!logFile.getParentFile().isDirectory() && !logFile.getParentFile().mkdirs()) {
@@ -592,6 +592,12 @@ public void run() {
             }
         }
 
+        /* package */
+        static File getLogFile(Jenkins jenkins) {
+            File logsRoot = new File(jenkins.getRootDir(), "logs");
+            return new File(logsRoot, "health-checker.log");
+        }
+
         /**
          * The actual periodic work to run asynchronously.
          *

diff --git a/src/test/java/jenkins/metrics/api/MetricsTest.java b/src/test/java/jenkins/metrics/api/MetricsTest.java
@@ -0,0 +1,57 @@
+package jenkins.metrics.api;
+
+import hudson.util.VersionNumber;
+import jenkins.model.Jenkins;
+import org.junit.AfterClass;
+import org.junit.Assume;
+import org.junit.BeforeClass;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.TemporaryFolder;
+import org.jvnet.hudson.test.Issue;
+import org.jvnet.hudson.test.JenkinsRule;
+import org.jvnet.hudson.test.RestartableJenkinsRule;
+
+import java.io.IOException;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+public class MetricsTest {
+
+    private static final String SYSPROP_FOR_LOGS_PATH = "hudson.triggers.SafeTimerTask.logsTargetDir";
+    @Rule
+    public final TemporaryFolder folder = new TemporaryFolder();
+    @Rule
+    public RestartableJenkinsRule story = new RestartableJenkinsRule();
+
+    @BeforeClass
+    @AfterClass
+    public static void cleanProperty() throws IOException {
+        System.clearProperty(SYSPROP_FOR_LOGS_PATH);
+    }
+
+
+    @Issue("JENKINS-50291")
+    @Test
+    public void logsDirectory() {
+
+        // Lambda-ify when Java 8+
+        story.then(new RestartableJenkinsRule.Step() {
+            @Override
+            public void run(JenkinsRule j) throws Throwable {
+                Assume.assumeTrue(Jenkins.getVersion().isNewerThan(new VersionNumber("2.113"))); // JENKINS-50291 introduced in 2.114
+
+                assertTrue(Metrics.HealthChecker.getLogFile(j.jenkins).getAbsolutePath().startsWith(j.getInstance().getRootDir().getAbsolutePath()));
+                System.setProperty("hudson.triggers.SafeTimerTask.logsTargetDir", folder.newFolder().getAbsolutePath());
+            }
+        });
+
+        story.then(new RestartableJenkinsRule.Step() {
+            @Override
+            public void run(JenkinsRule j) throws Throwable {
+                assertFalse(Metrics.HealthChecker.getLogFile(j.jenkins).getAbsolutePath().startsWith(j.getInstance().getRootDir().getAbsolutePath()));
+            }
+        });
+    }
+}