DATACMNS-485 - Fixed false negative type alias detection in MappingContextTypeInformationMapper.

MappingContextTypeInformationMapper now considers the raw type only for type alias conflict detection.

Author: odrotbohm

src/main/java/org/springframework/data/convert/MappingContextTypeInformationMapper.java

@@ -21,6 +21,7 @@
 
 import org.springframework.data.mapping.PersistentEntity;
 import org.springframework.data.mapping.context.MappingContext;
+import org.springframework.data.util.ClassTypeInformation;
 import org.springframework.data.util.TypeInformation;
 import org.springframework.util.Assert;
 
@@ -90,12 +91,31 @@ private void safelyAddToCache(TypeInformation<?> key, Object alias) {
 			return;
 		}
 
-		if (typeMap.containsValue(alias)) {
+		TypeInformation<?> toStore = ClassTypeInformation.from(key.getType());
+		Object existingAlias = typeMap.get(toStore);
+
+		// Reject second alias for same type
+
+		if (existingAlias != null && !alias.equals(existingAlias)) {
 			throw new IllegalArgumentException(String.format(
-					"Detected mapping ambiguity! String %s cannot be mapped to more than one type!", alias));
+					"Trying to register alias '%s', but found already registered alias '%s' for type %s!", alias, existingAlias,
+					toStore));
+		}
+
+		// Reject second type for same alias
+
+		if (typeMap.containsValue(alias)) {
+
+			for (Entry<TypeInformation<?>, Object> entry : typeMap.entrySet()) {
+				if (entry.getValue().equals(alias) && !entry.getKey().equals(toStore)) {
+					throw new IllegalArgumentException(String.format(
+							"Detected existing type mapping of %s to alias '%s' but attempted to bind the same alias to %s!",
+							toStore, alias, entry.getKey()));
+				}
+			}
 		}
 
-		typeMap.put(key, alias);
+		typeMap.put(toStore, alias);
 	}
 
 	/*

src/test/java/org/springframework/data/convert/MappingContextTypeInformationMapperUnitTests.java

@@ -19,7 +19,9 @@
 import static org.junit.Assert.*;
 import static org.springframework.data.util.ClassTypeInformation.*;
 
+import java.util.Arrays;
 import java.util.Collections;
+import java.util.HashSet;
 
 import org.junit.Before;
 import org.junit.Test;
@@ -98,8 +100,41 @@ public void detectsTypeForUnknownEntity() {
 		assertThat(mapper.resolveTypeFrom("foo"), is((TypeInformation) from(Entity.class)));
 	}
 
+	/**
+	 * @see DATACMNS-485
+	 */
+	@Test
+	@SuppressWarnings("unchecked")
+	public void createsTypeMapperForGenericTypesWithDifferentBindings() {
+
+		SampleMappingContext context = new SampleMappingContext();
+		context.setInitialEntitySet(new HashSet<Class<? extends Object>>(Arrays.asList(Entity.class, GenericType.class,
+				ConcreteWrapper.class, GenericWrapper.class)));
+		context.initialize();
+
+		new MappingContextTypeInformationMapper(context);
+	}
+
 	@TypeAlias("foo")
 	static class Entity {
 
 	}
+
+	@TypeAlias("genericType")
+	static class GenericType<T> {
+
+	}
+
+	@TypeAlias("concreteWrapper")
+	static class ConcreteWrapper {
+
+		GenericType<String> stringGeneric;
+		GenericType<Integer> integerGeneric;
+	}
+
+	@TypeAlias("genericWrapper")
+	static class GenericWrapper<T> {
+
+		GenericType<T> genericGeneric;
+	}
 }