Refine TypeModelProcessor class filtering and error handling

Closes gh-1376

Author: sdeleuze

samples/data-mongodb/src/main/java/com/example/data/mongo/Order.java

@@ -25,11 +25,17 @@
 import org.springframework.data.annotation.LastModifiedBy;
 import org.springframework.data.annotation.LastModifiedDate;
 import org.springframework.data.annotation.PersistenceConstructor;
+import org.springframework.data.domain.Sort;
+import org.springframework.data.mongodb.core.index.Index;
+import org.springframework.data.mongodb.core.index.IndexDefinition;
+import org.springframework.data.mongodb.core.index.IndexResolver;
 import org.springframework.data.mongodb.core.index.Indexed;
+import org.springframework.data.mongodb.core.index.PartialIndexFilter;
 import org.springframework.data.mongodb.core.mapping.DBRef;
 import org.springframework.data.mongodb.core.mapping.Document;
 import org.springframework.data.mongodb.core.mapping.DocumentReference;
 import org.springframework.data.mongodb.core.mapping.Field;
+import org.springframework.data.mongodb.core.query.Criteria;
 
 @Document
 public class Order {
@@ -189,4 +195,15 @@ public String toString() {
 				", reduction=" + reduction +
 				'}';
 	}
+
+	// Reproducer for https://github.com/spring-projects-experimental/spring-native/issues/1376
+	public static List<IndexDefinition> getIndexes(IndexResolver indexResolver) {
+		List<IndexDefinition> ret = new ArrayList<>();
+		indexResolver.resolveIndexFor(Order.class).forEach(ret::add);
+		ret.add(new Index()
+				.unique()
+				.on("createdBy", Sort.Direction.ASC)
+				.partial(PartialIndexFilter.of(Criteria.where("createdBy").exists(true))));
+				return ret;
+		}
 }

spring-aot/src/main/java/org/springframework/data/TypeModelProcessor.java

@@ -35,15 +35,22 @@
 import java.util.function.Predicate;
 import java.util.stream.Collectors;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.springframework.core.ResolvableType;
 import org.springframework.util.ReflectionUtils;
 
 /**
  * @author Christoph Strobl
+ * @author Sebastien Deleuze
  */
 public class TypeModelProcessor {
 
-	static final Set<String> EXCLUDED_DOMAINS = new HashSet<>(Arrays.asList("sun.", "jdk.", "reactor.", "kotlinx.", "kotlin."));
+	private static Log logger = LogFactory.getLog(TypeModelProcessor.class);
+
+	static final Set<String> EXCLUDED_DOMAINS = new HashSet<>(Arrays.asList("sun.", "jdk.", "reactor.", "kotlinx.", "kotlin.",
+			"org.springframework.core.", "org.springframework.data.", "org.springframework.boot."));
 
 	private Predicate<Class<?>> typeFilter = (type) -> EXCLUDED_DOMAINS.stream().noneMatch(it -> {
 		if (type.getPackageName().startsWith("java.")) {
@@ -151,17 +158,21 @@ Set<Type> visitMethodsOfType(ResolvableType type, TypeModel result) {
 			return Collections.emptySet();
 		}
 		Set<Type> discoveredTypes = new LinkedHashSet<>();
-		ReflectionUtils.doWithLocalMethods(type.toClass(), method -> {
-			if (!methodFilter.test(method)) {
-				return;
-			}
-			result.addMethod(method);
-			for (Class<?> signatureType : TypeUtils.resolveTypesInSignature(type.toClass(), method)) {
-				if (typeFilter.test(signatureType)) {
-					discoveredTypes.add(signatureType);
+		try {
+			ReflectionUtils.doWithLocalMethods(type.toClass(), method -> {
+				if (!methodFilter.test(method)) {
+					return;
 				}
-			}
-		});
+				result.addMethod(method);
+				for (Class<?> signatureType : TypeUtils.resolveTypesInSignature(type.toClass(), method)) {
+					if (typeFilter.test(signatureType)) {
+						discoveredTypes.add(signatureType);
+					}
+				}
+			});
+		} catch (Exception ex) {
+			logger.warn(ex);
+		}
 		return new HashSet<>(discoveredTypes);
 	}