Replace all exposed superclasses in final step after traversal

See gh-28676

Author: jhoeller

Diff for: spring-context/src/main/java/org/springframework/context/annotation/ConfigurationClassParser.java

@@ -468,6 +468,9 @@ private Set<MethodMetadata> retrieveBeanMethodMetadata(SourceClass sourceClass)
 	 * the superclass exposure on a different config class with the same superclass.
 	 */
 	private void removeKnownSuperclass(String removedClass, boolean replace) {
+		String replacedSuperclass = null;
+		ConfigurationClass replacingClass = null;
+
 		Iterator<Map.Entry<String, List<ConfigurationClass>>> it = this.knownSuperclasses.entrySet().iterator();
 		while (it.hasNext()) {
 			Map.Entry<String, List<ConfigurationClass>> entry = it.next();
@@ -476,22 +479,29 @@ private void removeKnownSuperclass(String removedClass, boolean replace) {
 					it.remove();
 				}
 				else if (replace) {
-					try {
-						ConfigurationClass otherClass = entry.getValue().get(0);
-						SourceClass sourceClass = asSourceClass(otherClass, DEFAULT_EXCLUSION_FILTER).getSuperClass();
-						while (!sourceClass.getMetadata().getClassName().equals(entry.getKey()) &&
-								sourceClass.getMetadata().getSuperClassName() != null) {
-							sourceClass = sourceClass.getSuperClass();
-						}
-						doProcessConfigurationClass(otherClass, sourceClass, DEFAULT_EXCLUSION_FILTER);
-					}
-					catch (IOException ex) {
-						throw new BeanDefinitionStoreException(
-								"I/O failure while removing configuration class [" + removedClass + "]", ex);
-					}
+					replacedSuperclass = entry.getKey();
+					replacingClass = entry.getValue().get(0);
 				}
 			}
 		}
+
+		if (replacingClass != null) {
+			try {
+				SourceClass sourceClass = asSourceClass(replacingClass, DEFAULT_EXCLUSION_FILTER).getSuperClass();
+				while (!sourceClass.getMetadata().getClassName().equals(replacedSuperclass) &&
+						sourceClass.getMetadata().getSuperClassName() != null) {
+					sourceClass = sourceClass.getSuperClass();
+				}
+				do {
+					sourceClass = doProcessConfigurationClass(replacingClass, sourceClass, DEFAULT_EXCLUSION_FILTER);
+				}
+				while (sourceClass != null);
+			}
+			catch (IOException ex) {
+				throw new BeanDefinitionStoreException(
+						"I/O failure while removing configuration class [" + removedClass + "]", ex);
+			}
+		}
 	}
 
 	/**

Diff for: spring-context/src/test/java/org/springframework/context/annotation/configuration/ConfigurationPhasesKnownSuperclassesTests.java

@@ -31,6 +31,7 @@
 
 /**
  * @author Andy Wilkinson
+ * @author Juergen Hoeller
  * @since 6.2
  */
 class ConfigurationPhasesKnownSuperclassesTests {
@@ -40,6 +41,7 @@ void superclassSkippedInParseConfigurationPhaseShouldNotPreventSubsequentProcess
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext(ParseConfigurationPhase.class)) {
 			assertThat(context.getBean("subclassBean")).isEqualTo("bravo");
 			assertThat(context.getBean("superclassBean")).isEqualTo("superclass");
+			assertThat(context.getBean("baseBean")).isEqualTo("base");
 		}
 	}
 
@@ -48,12 +50,22 @@ void superclassSkippedInRegisterBeanPhaseShouldNotPreventSubsequentProcessingOfS
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext(RegisterBeanPhase.class)) {
 			assertThat(context.getBean("subclassBean")).isEqualTo("bravo");
 			assertThat(context.getBean("superclassBean")).isEqualTo("superclass");
+			assertThat(context.getBean("baseBean")).isEqualTo("base");
 		}
 	}
 
 
 	@Configuration(proxyBeanMethods = false)
-	static class Example {
+	static class Base {
+
+		@Bean
+		String baseBean() {
+			return "base";
+		}
+	}
+
+	@Configuration(proxyBeanMethods = false)
+	static class Example extends Base {
 
 		@Bean
 		String superclassBean() {