Polish

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/mongo/MongoClientFactory.java

@@ -16,7 +16,6 @@
 
 package org.springframework.boot.autoconfigure.mongo;
 
-import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
@@ -89,39 +88,48 @@ public class MongoClientFactory {
 	}
 
 	private MongoClient createNetworkMongoClient(MongoClientOptions options) {
-		if (this.properties.getUri() != null) {
-			return new MongoClient(
-					new MongoClientURI(this.properties.getUri(), builder(options)));
+		MongoProperties properties = this.properties;
+		if (properties.getUri() != null) {
+			return createMongoClient(properties.getUri(), options);
 		}
 		if (hasCustomAddress() || hasCustomCredentials()) {
 			if (options == null) {
 				options = MongoClientOptions.builder().build();
 			}
-			List<MongoCredential> credentials = new ArrayList<>();
-			if (hasCustomCredentials()) {
-				String database = this.properties.getAuthenticationDatabase() == null
-						? this.properties.getMongoClientDatabase()
-						: this.properties.getAuthenticationDatabase();
-				credentials.add(
-						MongoCredential.createCredential(this.properties.getUsername(),
-								database, this.properties.getPassword()));
-			}
-			String host = this.properties.getHost() == null ? "localhost"
-					: this.properties.getHost();
-			int port = this.properties.getPort() != null ? this.properties.getPort()
-					: MongoProperties.DEFAULT_PORT;
-			return new MongoClient(
-					Collections.singletonList(new ServerAddress(host, port)), credentials,
-					options);
+			List<MongoCredential> credentials = getCredentials(properties);
+			String host = getValue(properties.getHost(), "localhost");
+			int port = getValue(properties.getPort(), MongoProperties.DEFAULT_PORT);
+			List<ServerAddress> seeds = Collections
+					.singletonList(new ServerAddress(host, port));
+			return new MongoClient(seeds, credentials, options);
 		}
-		return new MongoClient(
-				new MongoClientURI(MongoProperties.DEFAULT_URI, builder(options)));
+		return createMongoClient(MongoProperties.DEFAULT_URI, options);
+	}
+
+	private MongoClient createMongoClient(String uri, MongoClientOptions options) {
+		return new MongoClient(new MongoClientURI(uri, builder(options)));
+	}
+
+	private <T> T getValue(T value, T fallback) {
+		return (value == null ? fallback : value);
 	}
 
 	private boolean hasCustomAddress() {
 		return this.properties.getHost() != null || this.properties.getPort() != null;
 	}
 
+	private List<MongoCredential> getCredentials(MongoProperties properties) {
+		if (!hasCustomCredentials()) {
+			return Collections.emptyList();
+		}
+		String username = properties.getUsername();
+		String database = getValue(properties.getAuthenticationDatabase(),
+				properties.getMongoClientDatabase());
+		char[] password = properties.getPassword();
+		return Collections.singletonList(
+				MongoCredential.createCredential(username, database, password));
+	}
+
 	private boolean hasCustomCredentials() {
 		return this.properties.getUsername() != null
 				&& this.properties.getPassword() != null;

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/jsonb/JsonbAutoConfigurationTests.java

@@ -46,7 +46,6 @@ public class JsonbAutoConfigurationTests {
 
 	public class DataObject {
 
-		@SuppressWarnings("unused")
 		private String data = "hello";
 
 		public String getData() {

spring-boot-samples/spring-boot-sample-custom-layout/pom.xml

@@ -50,7 +50,6 @@
 							<addTestClassPath>true</addTestClassPath>
 							<skipInvocation>${skipTests}</skipInvocation>
 							<streamLogs>true</streamLogs>
-							<!-- <properties> <revision>${revision}</revision> </properties> -->
 						</configuration>
 					</execution>
 				</executions>