Remove AopProxyUtils#completeJdkProxyInterfaces

Not required anymore since JDK dynamic proxies are inferred.

Closes spring-projectsgh-28980

Author: sdeleuze

spring-aop/src/main/java/org/springframework/aop/framework/AopProxyUtils.java

@@ -93,46 +93,6 @@ public static Class<?> ultimateTargetClass(Object candidate) {
 		return result;
 	}
 
-	/**
-	 * Complete the set of interfaces that are typically required in a JDK dynamic
-	 * proxy generated by Spring AOP.
-	 * <p>Specifically, {@link SpringProxy}, {@link Advised}, and {@link DecoratingProxy}
-	 * will be appended to the set of user-specified interfaces.
-	 * <p>This method can be useful when registering
-	 * {@linkplain org.springframework.aot.hint.ProxyHints proxy hints} for Spring's
-	 * AOT support, as demonstrated in the following example which uses this method
-	 * via a {@code static} import.
-	 * <pre class="code">
-	 * RuntimeHints hints = ...
-	 * hints.proxies().registerJdkProxy(completeJdkProxyInterfaces(MyInterface.class));
-	 * </pre>
-	 * @param userInterfaces the set of user-specified interfaces implemented by
-	 * the component to be proxied
-	 * @return the complete set of interfaces that the proxy should implement
-	 * @throws IllegalArgumentException if a supplied {@code Class} is {@code null},
-	 * is not an {@linkplain Class#isInterface() interface}, or is a
-	 * {@linkplain Class#isSealed() sealed} interface
-	 * @since 6.0
-	 * @see SpringProxy
-	 * @see Advised
-	 * @see DecoratingProxy
-	 * @see org.springframework.aot.hint.RuntimeHints#proxies()
-	 * @see org.springframework.aot.hint.ProxyHints#registerJdkProxy(Class...)
-	 */
-	public static Class<?>[] completeJdkProxyInterfaces(Class<?>... userInterfaces) {
-		List<Class<?>> completedInterfaces = new ArrayList<>(userInterfaces.length + 3);
-		for (Class<?> ifc : userInterfaces) {
-			Assert.notNull(ifc, "'userInterfaces' must not contain null values");
-			Assert.isTrue(ifc.isInterface() && !ifc.isSealed(),
-					() -> ifc.getName() + " must be a non-sealed interface");
-			completedInterfaces.add(ifc);
-		}
-		completedInterfaces.add(SpringProxy.class);
-		completedInterfaces.add(Advised.class);
-		completedInterfaces.add(DecoratingProxy.class);
-		return completedInterfaces.toArray(Class<?>[]::new);
-	}
-
 	/**
 	 * Determine the complete set of interfaces to proxy for the given AOP configuration.
 	 * <p>This will always add the {@link Advised} interface unless the AdvisedSupport's

spring-aop/src/test/java/org/springframework/aop/framework/AopProxyUtilsTests.java

@@ -23,7 +23,6 @@
 import org.springframework.aop.SpringProxy;
 import org.springframework.beans.testfixture.beans.ITestBean;
 import org.springframework.beans.testfixture.beans.TestBean;
-import org.springframework.core.DecoratingProxy;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
@@ -109,41 +108,6 @@ void proxiedUserInterfacesWithNoInterface() {
 		assertThatIllegalArgumentException().isThrownBy(() -> AopProxyUtils.proxiedUserInterfaces(proxy));
 	}
 
-	@Test
-	void completeJdkProxyInterfacesFromNullInterface() {
-		assertThatIllegalArgumentException()
-			.isThrownBy(() -> AopProxyUtils.completeJdkProxyInterfaces(ITestBean.class, null, Comparable.class))
-			.withMessage("'userInterfaces' must not contain null values");
-	}
-
-	@Test
-	void completeJdkProxyInterfacesFromClassThatIsNotAnInterface() {
-		assertThatIllegalArgumentException()
-			.isThrownBy(() -> AopProxyUtils.completeJdkProxyInterfaces(TestBean.class))
-			.withMessage(TestBean.class.getName() + " must be a non-sealed interface");
-	}
-
-	@Test
-	void completeJdkProxyInterfacesFromSealedInterface() {
-		assertThatIllegalArgumentException()
-			.isThrownBy(() -> AopProxyUtils.completeJdkProxyInterfaces(SealedInterface.class))
-			.withMessage(SealedInterface.class.getName() + " must be a non-sealed interface");
-	}
-
-	@Test
-	void completeJdkProxyInterfacesFromSingleClass() {
-		Class<?>[] jdkProxyInterfaces = AopProxyUtils.completeJdkProxyInterfaces(ITestBean.class);
-		assertThat(jdkProxyInterfaces).containsExactly(
-				ITestBean.class, SpringProxy.class, Advised.class, DecoratingProxy.class);
-	}
-
-	@Test
-	void completeJdkProxyInterfacesFromMultipleClasses() {
-		Class<?>[] jdkProxyInterfaces = AopProxyUtils.completeJdkProxyInterfaces(ITestBean.class, Comparable.class);
-		assertThat(jdkProxyInterfaces).containsExactly(
-				ITestBean.class, Comparable.class, SpringProxy.class, Advised.class, DecoratingProxy.class);
-	}
-
 
 	sealed interface SealedInterface {
 	}