From 9198774f13daec6eaa74edeb043ef6683456564f Mon Sep 17 00:00:00 2001
From: Juergen Hoeller <juergen.hoeller@broadcom.com>
Date: Sun, 25 Feb 2024 16:32:11 +0100
Subject: [PATCH] Detect wildcard without bounds returned from VariableResolver
 as well

See gh-32327
See gh-20727
---
 .../springframework/core/ResolvableType.java  |  2 +-
 .../core/ResolvableTypeTests.java             | 19 +++++++++++++++----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/spring-core/src/main/java/org/springframework/core/ResolvableType.java b/spring-core/src/main/java/org/springframework/core/ResolvableType.java
index f4b9a0ce58..a1c515a77b 100644
--- a/spring-core/src/main/java/org/springframework/core/ResolvableType.java
+++ b/spring-core/src/main/java/org/springframework/core/ResolvableType.java
@@ -655,7 +655,7 @@ public class ResolvableType implements Serializable {
 				return true;
 			}
 			ResolvableType resolved = this.variableResolver.resolveVariable(variable);
-			if (resolved == null || resolved.isUnresolvableTypeVariable()) {
+			if (resolved == null || resolved.isUnresolvableTypeVariable() || resolved.isWildcardWithoutBounds()) {
 				return true;
 			}
 		}
diff --git a/spring-core/src/test/java/org/springframework/core/ResolvableTypeTests.java b/spring-core/src/test/java/org/springframework/core/ResolvableTypeTests.java
index d27f207011..966793c564 100644
--- a/spring-core/src/test/java/org/springframework/core/ResolvableTypeTests.java
+++ b/spring-core/src/test/java/org/springframework/core/ResolvableTypeTests.java
@@ -1370,13 +1370,19 @@ class ResolvableTypeTests {
 	@Test
 	void gh32327() throws Exception {
 		ResolvableType repository1 = ResolvableType.forField(Fields.class.getField("repository"));
-		ResolvableType repository2 = ResolvableType.forMethodReturnType(Methods.class.getMethod("repository"));
-		assertThat(repository1.hasUnresolvableGenerics());
+		ResolvableType repository2 = ResolvableType.forMethodReturnType(Methods.class.getMethod("someRepository"));
+		ResolvableType repository3 = ResolvableType.forMethodReturnType(Methods.class.getMethod("subRepository"));
+		assertThat(repository1.hasUnresolvableGenerics()).isFalse();
 		assertThat(repository1.isAssignableFrom(repository2)).isFalse();
 		assertThat(repository1.isAssignableFromResolvedPart(repository2)).isTrue();
-		assertThat(repository2.hasUnresolvableGenerics());
+		assertThat(repository1.isAssignableFrom(repository3)).isFalse();
+		assertThat(repository1.isAssignableFromResolvedPart(repository3)).isTrue();
+		assertThat(repository2.hasUnresolvableGenerics()).isTrue();
 		assertThat(repository2.isAssignableFrom(repository1)).isTrue();
 		assertThat(repository2.isAssignableFromResolvedPart(repository1)).isTrue();
+		assertThat(repository3.hasUnresolvableGenerics()).isTrue();
+		assertThat(repository3.isAssignableFrom(repository1)).isFalse();
+		assertThat(repository3.isAssignableFromResolvedPart(repository1)).isFalse();
 	}
 
 
@@ -1424,6 +1430,9 @@ class ResolvableTypeTests {
 		<T> T someMethod(Class<T> arg0, Class<?> arg1, Class<Object> arg2);
 	}
 
+	interface SubRepository<S extends Serializable> extends SomeRepository {
+	}
+
 
 	static class Fields<T> {
 
@@ -1501,7 +1510,9 @@ class ResolvableTypeTests {
 
 		List<String> list2();
 
-		SomeRepository<?> repository();
+		SomeRepository<?> someRepository();
+
+		SubRepository<?> subRepository();
 	}