CommonsExecWebServerFactoryBean.systemProperties(Consumer<Map<String,String>>)

Closes gh-33

Author: rwinch

spring-boot-testjars/src/main/java/org/springframework/experimental/boot/server/exec/CommonsExecWebServerFactoryBean.java

@@ -94,8 +94,8 @@ public CommonsExecWebServerFactoryBean classpath(Consumer<ClasspathBuilder> conf
 		return this;
 	}
 
-	public CommonsExecWebServerFactoryBean addSystemProperties(Map<String, String> systemProperties) {
-		this.systemProperties.putAll(systemProperties);
+	public CommonsExecWebServerFactoryBean systemProperties(Consumer<Map<String, String>> systemProperties) {
+		systemProperties.accept(this.systemProperties);
 		return this;
 	}
 

spring-boot-testjars/src/test/java/org/springframework/experimental/boot/server/exec/CommonsExecWebServerFactoryBeanTests.java

@@ -23,7 +23,6 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
-import java.util.Map;
 
 import org.junit.jupiter.api.Test;
 
@@ -107,7 +106,7 @@ void serverPortWhenSpecifiedThenNotOverridden() throws Exception {
 		String expectedPort = "1234";
 		String portSystemProperty = "server.port";
 		CommonsExecWebServer server = CommonsExecWebServerFactoryBean.builder()
-				.addSystemProperties(Map.of(portSystemProperty, expectedPort)).getObject();
+				.systemProperties((props) -> props.put(portSystemProperty, expectedPort)).getObject();
 		assertThat(server.getCommandLine().getArguments()).contains("-D" + portSystemProperty + "=" + expectedPort);
 	}