INT-4544: Allow runtime MBeans (un)registered (#2601)

* INT-4544: Allow runtime MBeans (un)registered

JIRA: https://jira.spring.io/browse/INT-4544

* Fix `SourcePollingChannelAdapterFactoryBean` to register an
`outputChannelName` into the `SourcePollingChannelAdapter` for late
binding, especially in case of dynamic `IntegrationFlow` registrations.
This must be back-ported to `5.0.x`
* Implement `DestructionAwareBeanPostProcessor` in the
`IntegrationMBeanExporter` for the runtime beans tracking, e.g. via
dynamic `IntegrationFlow` registrations
* Refactor `IntegrationMBeanExporter` for static and dynamic beans
registration and destruction
* Remove unused properties; introduce some new for tracking beans and
their relationship

* * Move `IntegrationMBeanExporter.registerProducer()` after
`postProcessAfterInitialization()`
* Refactor JMX test configurations to reuse MBeanServer as much as
possible and destroy the server whenever it is necessary

* * More JMX tests polishing

* * And more JMX tests polishing

* * `try...catch` in the `IntegrationMBeanExporter.postProcessAfterInitialization()`
to avoid breaking changes for the current GA phase
* Do not unregister those MBeans explicitly which weren't created at
runtime

* * Log ObjectNames in the `MBeanExporterIntegrationTests`

* * Fix `MessageMetricsAdviceTests` to rely on the application context
* Do not use a cst in the `StandardIntegrationFlowContext` for
`BeanFactory`, but an explicit `BeanDefinitionRegistry`
* Extract targets from proxies in the
`IntegrationMBeanExporter.postProcessAfterInitialization()`
* Remove logging in the `MBeanExporterIntegrationTests`

* * Fix `NotificationListeningMessageProducerTests` to reuse existing `MBeanServer`

* * Change JMX `domain` in the `DslMBeanTests` to `dsl` do not clash with
similar in the `foo` in the `MessageSourceTests`
* Use `MBeanServer` bean in the `Int2307Tests`
* Use JUnit 5ctor injection injection in the `MessageMetricsAdviceTests`

* * Fix JMX tests do not reuse existing `MBeanServer` and make them rely
on the server provided by the managed `MBeanServerFactoryBean` which
destroys a server on application context close
* Fix `StoredProcJmxManagedBeanTests-context.xml` to use `MBeanServer`
from the `<context:mbean-server>`

Author: artembilan

spring-integration-core/src/main/java/org/springframework/integration/config/SourcePollingChannelAdapterFactoryBean.java

@@ -145,7 +145,7 @@ public void afterPropertiesSet() throws Exception {
 	}
 
 	@Override
-	public SourcePollingChannelAdapter getObject() throws Exception {
+	public SourcePollingChannelAdapter getObject() {
 		if (this.adapter == null) {
 			initializeAdapter();
 		}
@@ -169,15 +169,18 @@ private void initializeAdapter() {
 			}
 			Assert.notNull(this.source, "source is required");
 
+			SourcePollingChannelAdapter spca = new SourcePollingChannelAdapter();
+			spca.setSource(this.source);
+
 			if (StringUtils.hasText(this.outputChannelName)) {
 				Assert.isNull(this.outputChannel, "'outputChannelName' and 'outputChannel' are mutually exclusive.");
-				this.outputChannel = this.channelResolver.resolveDestination(this.outputChannelName);
+				spca.setOutputChannelName(this.outputChannelName);
+			}
+			else {
+				Assert.notNull(this.outputChannel, "outputChannel is required");
+				spca.setOutputChannel(this.outputChannel);
 			}
 
-			Assert.notNull(this.outputChannel, "outputChannel is required");
-			SourcePollingChannelAdapter spca = new SourcePollingChannelAdapter();
-			spca.setSource(this.source);
-			spca.setOutputChannel(this.outputChannel);
 			if (this.pollerMetadata == null) {
 				this.pollerMetadata = PollerMetadata.getDefaultPollerMetadata(this.beanFactory);
 				Assert.notNull(this.pollerMetadata, "No poller has been defined for channel-adapter '"

spring-integration-core/src/main/java/org/springframework/integration/dsl/context/StandardIntegrationFlowContext.java

@@ -134,7 +134,7 @@ private Object registerBean(Object bean, String beanName, String parentName) {
 				BeanDefinitionBuilder.genericBeanDefinition((Class<Object>) bean.getClass(), () -> bean)
 						.getRawBeanDefinition();
 
-		((BeanDefinitionRegistry) this.beanFactory).registerBeanDefinition(beanName, beanDefinition);
+		this.beanDefinitionRegistry.registerBeanDefinition(beanName, beanDefinition);
 
 		if (parentName != null) {
 			this.beanFactory.registerDependentBean(parentName, beanName);

spring-integration-jdbc/src/test/java/org/springframework/integration/jdbc/StoredProcJmxManagedBeanTests-context.xml

@@ -9,8 +9,9 @@
 		http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans.xsd
 		http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context.xsd">
 
-	<context:mbean-server />
-	<context:mbean-export default-domain="org.springframework.integration.jdbc.test" />
+	<context:mbean-server id="mbeanServer"/>
+
+	<context:mbean-export server="mbeanServer" default-domain="org.springframework.integration.jdbc.test" />
 
 	<import resource="classpath:derby-stored-procedures-setup-context.xml"/>