Fix tests

Setting the log file name to a random value in smoke tests
doesn't work because the logger context has already been initialized.

Author: mbhave

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/context/logging/LoggingApplicationListenerIntegrationTests.java

@@ -18,12 +18,15 @@
 
 import org.junit.Rule;
 import org.junit.Test;
+import org.junit.rules.TemporaryFolder;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import org.springframework.beans.factory.ObjectProvider;
 import org.springframework.boot.WebApplicationType;
 import org.springframework.boot.builder.SpringApplicationBuilder;
 import org.springframework.boot.context.event.ApplicationStartingEvent;
+import org.springframework.boot.logging.LogFile;
 import org.springframework.boot.logging.LoggingSystem;
 import org.springframework.boot.testsupport.rule.OutputCapture;
 import org.springframework.context.ApplicationListener;
@@ -42,6 +45,9 @@ public class LoggingApplicationListenerIntegrationTests {
 	@Rule
 	public OutputCapture outputCapture = new OutputCapture();
 
+	@Rule
+	public final TemporaryFolder temp = new TemporaryFolder();
+
 	@Test
 	public void loggingSystemRegisteredInTheContext() {
 		try (ConfigurableApplicationContext context = new SpringApplicationBuilder(SampleService.class)
@@ -51,6 +57,17 @@ public void loggingSystemRegisteredInTheContext() {
 		}
 	}
 
+	@Test
+	public void logFileRegisteredInTheContextWhenApplicable() throws Exception {
+		String logFile = this.temp.newFile().getAbsolutePath();
+		try (ConfigurableApplicationContext context = new SpringApplicationBuilder(SampleService.class)
+				.web(WebApplicationType.NONE).properties("logging.file=" + logFile).run()) {
+			SampleService service = context.getBean(SampleService.class);
+			assertThat(service.logFile).isNotNull();
+			assertThat(service.logFile.toString()).isEqualTo(logFile);
+		}
+	}
+
 	@Test
 	public void loggingPerformedDuringChildApplicationStartIsNotLost() {
 		new SpringApplicationBuilder(Config.class).web(WebApplicationType.NONE).child(Config.class)
@@ -72,8 +89,11 @@ static class SampleService {
 
 		private final LoggingSystem loggingSystem;
 
-		SampleService(LoggingSystem loggingSystem) {
+		private final LogFile logFile;
+
+		SampleService(LoggingSystem loggingSystem, ObjectProvider<LogFile> logFile) {
 			this.loggingSystem = loggingSystem;
+			this.logFile = logFile.getIfAvailable();
 		}
 
 	}

spring-boot-samples/spring-boot-sample-actuator/src/test/java/sample/actuator/EndpointsPropertiesSampleActuatorApplicationTests.java

@@ -66,13 +66,6 @@ public void testCustomContextPath() {
 		assertThat(entity.getBody()).contains("\"hello\":\"world\"");
 	}
 
-	@Test
-	public void logfileWithRandomName() {
-		ResponseEntity<String> entity = this.restTemplate.withBasicAuth("user", getPassword())
-				.getForEntity("/admin/logfile", String.class);
-		assertThat(entity.getStatusCode()).isEqualTo(HttpStatus.OK);
-	}
-
 	private String getPassword() {
 		return "password";
 	}

spring-boot-samples/spring-boot-sample-actuator/src/test/resources/application-endpoints.properties

@@ -1,5 +1,3 @@
-server.error.path: /oops
-management.endpoint.health.show-details: always
-management.endpoints.web.base-path: /admin
-logging.file=./target/${spring.application.instance_id}.log
-spring.application.instance_id=${random.value}
+server.error.path:/oops
+management.endpoint.health.show-details:always
+management.endpoints.web.base-path:/admin