DATACMNS-1079 - Polishing.

Refactored all the checks done to check if ClassGeneratingProperty works into separate methods, so the body of isSupported becomes nice and readable.

Author: schauder

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

@@ -101,15 +101,12 @@ public boolean isSupported(PersistentEntity<?, ?> entity) {
 
 		Assert.notNull(entity, "PersistentEntity must not be null!");
 
-		try {
-			Evil.getClassLoaderMethod(entity);
-		} catch (Exception o_O) {
-			return false;
-		}
+		return isClassLoaderDefineClassAccessible(entity) //
+				&& isTypeInjectable(entity) //
+				&& arePropertyHashCodesUnique(entity);
+	}
 
-		if (entity.getType().getClassLoader() == null || entity.getType().getPackage().getName().startsWith("java")) {
-			return false;
-		}
+	private boolean arePropertyHashCodesUnique(PersistentEntity<?, ?> entity) {
 
 		final Set<Integer> hashCodes = new HashSet<>();
 		final AtomicInteger propertyCount = new AtomicInteger();
@@ -132,6 +129,20 @@ public boolean isSupported(PersistentEntity<?, ?> entity) {
 		return hashCodes.size() == propertyCount.get();
 	}
 
+	private static boolean isTypeInjectable(PersistentEntity<?, ?> entity) {
+		return entity.getType().getClassLoader() != null && !entity.getType().getPackage().getName().startsWith("java");
+	}
+
+	private static boolean isClassLoaderDefineClassAccessible(PersistentEntity<?, ?> entity) {
+
+		try {
+			Evil.getClassLoaderMethod(entity);
+		} catch (Exception o_O) {
+			return false;
+		}
+		return true;
+	}
+
 	/**
 	 * @param entity must not be {@literal null}.
 	 * @return