DATACMNS-444 - DefaultPersistentPropertyPath now skips null converted…

… names.

If a Converter handed into DefaultPersistentPropertyPath.toDotPath(…) returns null or an empty string for a mapped property name, that value is skipped during path construction.

src/main/java/org/springframework/data/mapping/context/DefaultPersistentPropertyPath.java

@@ -93,10 +93,15 @@ public String toPath(String delimiter, Converter<? super T, String> converter) {
 		List<String> result = new ArrayList<String>();
 
 		for (T property : properties) {
-			result.add(converterToUse.convert(property));
+
+			String convert = converterToUse.convert(property);
+
+			if (StringUtils.hasText(convert)) {
+				result.add(convert);
+			}
 		}
 
-		return StringUtils.collectionToDelimitedString(result, delimiterToUse);
+		return result.isEmpty() ? null : StringUtils.collectionToDelimitedString(result, delimiterToUse);
 	}
 
 	/* 

src/test/java/org/springframework/data/mapping/context/DefaultPersistenPropertyPathUnitTests.java

@@ -38,11 +38,9 @@
 @RunWith(MockitoJUnitRunner.class)
 public class DefaultPersistenPropertyPathUnitTests<T extends PersistentProperty<T>> {
 
-	@Mock
-	T first, second;
+	@Mock T first, second;
 
-	@Mock
-	Converter<T, String> converter;
+	@Mock Converter<T, String> converter;
 
 	PersistentPropertyPath<T> oneLeg;
 	PersistentPropertyPath<T> twoLegs;
@@ -121,4 +119,39 @@ public void pathReturnsCorrectSize() {
 		assertThat(oneLeg.getLength(), is(1));
 		assertThat(twoLegs.getLength(), is(2));
 	}
+
+	/**
+	 * @see DATACMNS-444
+	 */
+	@Test
+	public void skipsMappedPropertyNameIfConverterReturnsNull() {
+
+		String result = twoLegs.toDotPath(new Converter<T, String>() {
+
+			@Override
+			public String convert(T source) {
+				return null;
+			}
+		});
+
+		assertThat(result, is(nullValue()));
+	}
+
+	/**
+	 * @see DATACMNS-444
+	 */
+	@Test
+	public void skipsMappedPropertyNameIfConverterReturnsEmptyStrings() {
+
+		String result = twoLegs.toDotPath(new Converter<T, String>() {
+
+			@Override
+			public String convert(T source) {
+				return "";
+			}
+		});
+
+		assertThat(result, is(nullValue()));
+	}
+
 }