DATACMNS-1006 - SimpleTypeHolder now considers java.lang types simple ones.

This prevents AbstractMappingContext from adding them as entities.

Related tickets: DATAREST-1018.

Author: odrotbohm

src/main/java/org/springframework/data/mapping/model/SimpleTypeHolder.java

@@ -123,6 +123,10 @@ public boolean isSimpleType(Class<?> type) {
 			return true;
 		}
 
+		if (type.getName().startsWith("java.lang")) {
+			return true;
+		}
+
 		for (Class<?> clazz : simpleTypes) {
 			if (clazz.isAssignableFrom(type)) {
 				simpleTypes.add(type);

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

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011 by the original author(s).
+ * Copyright 2011-2017 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,8 +18,10 @@
 import static org.hamcrest.CoreMatchers.*;
 import static org.junit.Assert.*;
 
+import java.lang.reflect.Type;
 import java.util.Collections;
 import java.util.HashSet;
+import java.util.UUID;
 
 import org.junit.Test;
 import org.springframework.data.mapping.model.SimpleTypeHolder;
@@ -60,7 +62,7 @@ public void doesNotAddDefaultConvertersIfConfigured() {
 
 		SimpleTypeHolder holder = new SimpleTypeHolder(new HashSet<Class<?>>(), false);
 
-		assertThat(holder.isSimpleType(String.class), is(false));
+		assertThat(holder.isSimpleType(UUID.class), is(false));
 	}
 
 	@Test
@@ -104,6 +106,14 @@ public void considersComplexEnumAsSimple() {
 		assertThat(holder.isSimpleType(ComplexEnum.FOO.getClass()), is(true));
 	}
 
+	@Test // DATACMNS-1006
+	public void considersJavaLangTypesSimple() {
+
+		SimpleTypeHolder holder = new SimpleTypeHolder();
+
+		assertThat(holder.isSimpleType(Type.class), is(true));
+	}
+
 	enum SimpleEnum {
 
 		FOO;