Tidy up log tests

Bamboo doesn't seem to be causing issues now, so assume
[Fixes #52871199] [bs-176] -Djava.util.logging.config.file
is passed to log4j as a file name?

Author: Dave Syer

spring-boot/src/test/java/org/springframework/boot/TestUtils.java

@@ -36,7 +36,7 @@ public static void addEnviroment(ConfigurableApplicationContext context,
 			int index = pair.indexOf(":");
 			String key = pair.substring(0, index > 0 ? index : pair.length());
 			String value = index > 0 ? pair.substring(index + 1) : "";
-			map.put(key, value);
+			map.put(key.trim(), value.trim());
 		}
 		context.getEnvironment().getPropertySources()
 				.addFirst(new MapPropertySource("test", map));

spring-boot/src/test/java/org/springframework/boot/context/initializer/LoggingApplicationContextInitializerTests.java

@@ -29,10 +29,10 @@
 import org.junit.rules.TemporaryFolder;
 import org.springframework.boot.OutputCapture;
 import org.springframework.boot.SpringApplication;
+import org.springframework.boot.TestUtils;
 import org.springframework.boot.logging.LogLevel;
 import org.springframework.boot.logging.java.JavaLoggingSystem;
 import org.springframework.context.support.GenericApplicationContext;
-import org.springframework.core.env.PropertySource;
 
 import static org.hamcrest.Matchers.containsString;
 import static org.hamcrest.Matchers.not;
@@ -90,16 +90,8 @@ public void testDefaultConfigLocation() {
 
 	@Test
 	public void testOverrideConfigLocation() {
-		this.context.getEnvironment().getPropertySources()
-				.addFirst(new PropertySource<String>("manual") {
-					@Override
-					public Object getProperty(String name) {
-						if ("logging.config".equals(name)) {
-							return "classpath:logback-nondefault.xml";
-						}
-						return null;
-					}
-				});
+		TestUtils.addEnviroment(this.context,
+				"logging.config: classpath:logback-nondefault.xml");
 		this.initializer.initialize(this.context);
 		this.logger.info("Hello world");
 		String output = this.outputCapture.toString().trim();
@@ -110,35 +102,16 @@ public Object getProperty(String name) {
 
 	@Test
 	public void testOverrideConfigDoesNotExist() throws Exception {
-		this.context.getEnvironment().getPropertySources()
-				.addFirst(new PropertySource<String>("manual") {
-					@Override
-					public Object getProperty(String name) {
-						if ("logging.config".equals(name)) {
-							return "doesnotexist.xml";
-						}
-						return null;
-					}
-				});
+		TestUtils.addEnviroment(this.context, "logging.config: doesnotexist.xml");
 		this.initializer.initialize(this.context);
 		// Should not throw
 	}
 
 	@Test
 	public void testAddLogFileProperty() {
-		this.context.getEnvironment().getPropertySources()
-				.addFirst(new PropertySource<String>("manual") {
-					@Override
-					public Object getProperty(String name) {
-						if ("logging.config".equals(name)) {
-							return "classpath:logback-nondefault.xml";
-						}
-						if ("logging.file".equals(name)) {
-							return "foo.log";
-						}
-						return null;
-					}
-				});
+		TestUtils.addEnviroment(this.context,
+				"logging.config: classpath:logback-nondefault.xml",
+				"logging.file: foo.log");
 		this.initializer.initialize(this.context);
 		Log logger = LogFactory.getLog(LoggingApplicationContextInitializerTests.class);
 		logger.info("Hello world");
@@ -148,19 +121,8 @@ public Object getProperty(String name) {
 
 	@Test
 	public void testAddLogPathProperty() {
-		this.context.getEnvironment().getPropertySources()
-				.addFirst(new PropertySource<String>("manual") {
-					@Override
-					public Object getProperty(String name) {
-						if ("logging.config".equals(name)) {
-							return "classpath:logback-nondefault.xml";
-						}
-						if ("logging.path".equals(name)) {
-							return "foo/";
-						}
-						return null;
-					}
-				});
+		TestUtils.addEnviroment(this.context,
+				"logging.config: classpath:logback-nondefault.xml", "logging.path: foo/");
 		this.initializer.initialize(this.context);
 		Log logger = LogFactory.getLog(LoggingApplicationContextInitializerTests.class);
 		logger.info("Hello world");