Fix formatting in PropertyReferenceException message.

A message like, "No property 'creat' found for type 'User' Did you mean ''created''" is now properly formatted as:
"No property 'creat' found for type 'User'; Did you mean 'created'".

Closes spring-projectsgh-2603.

Author: jxblum

src/main/java/org/springframework/data/mapping/PropertyReferenceException.java

@@ -33,12 +33,14 @@
  *
  * @author Oliver Gierke
  * @author Christoph Strobl
+ * @author John Blum
  */
 public class PropertyReferenceException extends RuntimeException {
 
 	private static final long serialVersionUID = -5254424051438976570L;
-	private static final String ERROR_TEMPLATE = "No property '%s' found for type '%s'";
-	private static final String HINTS_TEMPLATE = " Did you mean '%s'";
+
+	static final String ERROR_TEMPLATE = "No property '%s' found for type '%s'";
+	static final String HINTS_TEMPLATE = "Did you mean %s";
 
 	private final String propertyName;
 	private final TypeInformation<?> type;
@@ -105,13 +107,13 @@ public String getMessage() {
 		Collection<String> potentialMatches = getPropertyMatches();
 		if (!potentialMatches.isEmpty()) {
 			String matches = StringUtils.collectionToDelimitedString(potentialMatches, ",", "'", "'");
+			builder.append("; ");
 			builder.append(String.format(HINTS_TEMPLATE, matches));
 		}
 
 		if (!alreadyResolvedPath.isEmpty()) {
-			builder.append(" Traversed path: ");
+			builder.append("; Traversed path: ");
 			builder.append(alreadyResolvedPath.get(0).toString());
-			builder.append(".");
 		}
 
 		return builder.toString();

src/test/java/org/springframework/data/mapping/PropertyReferenceExceptionUnitTests.java

@@ -15,7 +15,8 @@
  */
 package org.springframework.data.mapping;
 
-import static org.assertj.core.api.Assertions.*;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
 import java.util.Collection;
 import java.util.Collections;
@@ -31,6 +32,7 @@
  *
  * @author Oliver Gierke
  * @author Mark Paluch
+ * @author John Blum
  */
 public class PropertyReferenceExceptionUnitTests {
 
@@ -68,6 +70,33 @@ public void exposesPotentialMatch() {
 		assertThat(matches).containsExactly("name");
 	}
 
+	@Test // GH-2750
+	public void formatsMessageWithTypeInfoAndHintsCorrectly() {
+
+		PropertyReferenceException exception = new PropertyReferenceException("nme", TYPE_INFO, NO_PATHS);
+
+		String expectedMessage = String.format("%s; %s", PropertyReferenceException.ERROR_TEMPLATE,
+				PropertyReferenceException.HINTS_TEMPLATE);
+
+		assertThat(exception)
+				.hasMessage(expectedMessage,"nme", TYPE_INFO.getType().getSimpleName(), "'name'");
+	}
+
+	@Test // GH-2750
+	public void formatsMessageWithTypeInfoHintsAndPathCorrectly() {
+
+		TypeInformation<C> ctype = ClassTypeInformation.from(C.class);
+
+		PropertyReferenceException exception = new PropertyReferenceException("nme", TYPE_INFO,
+				Collections.singletonList(PropertyPath.from("b.a", ctype)));
+
+		String expectedMessage = String.format("%s; %s; %s", PropertyReferenceException.ERROR_TEMPLATE,
+				PropertyReferenceException.HINTS_TEMPLATE, "Traversed path: C.b.a");
+
+		assertThat(exception)
+				.hasMessage(expectedMessage,"nme", TYPE_INFO.getType().getSimpleName(), "'name'");
+	}
+
 	static class Sample {
 
 		String name;
@@ -80,4 +109,16 @@ public void setName(String name) {
 			this.name = name;
 		}
 	}
+
+	static class A {
+
+	}
+
+	static class B {
+		A a;
+	}
+
+	static class C {
+		B b;
+	}
 }