Address PR review: name-based factory condition and stronger asymmetric regression assertions

- Make root EventHubClientBuilderFactory @ConditionalOnMissingBean name-based, consistent with the dedicated factory beans, so a user-provided differently-named factory does not suppress the named one.

- Strengthen the four asymmetric regression tests to assert the opposite-side DedicatedXxxConnectionConfiguration is actually activated, and assert multiple EventHubClientBuilder beans coexist in the shared-injection cases, so the tests truly exercise the failure mode the fix targets.

Author: rujche

sdk/spring/spring-cloud-azure-autoconfigure/src/main/java/com/azure/spring/cloud/autoconfigure/implementation/eventhubs/AzureEventHubsClientBuilderConfiguration.java

@@ -48,7 +48,7 @@ EventHubClientBuilder eventHubClientBuilder(@Qualifier(EVENT_HUB_CLIENT_BUILDER_
     }
 
     @Bean(EVENT_HUB_CLIENT_BUILDER_FACTORY_BEAN_NAME)
-    @ConditionalOnMissingBean
+    @ConditionalOnMissingBean(name = EVENT_HUB_CLIENT_BUILDER_FACTORY_BEAN_NAME)
     EventHubClientBuilderFactory eventHubClientBuilderFactory(
         ObjectProvider<ServiceConnectionStringProvider<AzureServiceType.EventHubs>> connectionStringProviders,
         ObjectProvider<AzureServiceClientBuilderCustomizer<EventHubClientBuilder>> customizers) {

sdk/spring/spring-cloud-azure-autoconfigure/src/test/java/com/azure/spring/cloud/autoconfigure/implementation/eventhubs/AzureEventHubsConsumerClientConfigurationTests.java

@@ -89,6 +89,10 @@ void producerOnlyDedicatedOverrideShouldNotActivateSharedConsumer() {
                     assertThat(context).doesNotHaveBean(AzureEventHubsConsumerClientConfiguration.class);
                     assertThat(context).doesNotHaveBean(EventHubConsumerClient.class);
                     assertThat(context).doesNotHaveBean(EventHubConsumerAsyncClient.class);
+                    // The producer dedicated path must actually be activated for this test to exercise the
+                    // "asymmetric, dedicated builder bean exists" scenario the bug fix targets.
+                    assertThat(context).hasSingleBean(AzureEventHubsProducerClientConfiguration.DedicatedProducerConnectionConfiguration.class);
+                    assertThat(context).hasBean(AzureContextUtils.EVENT_HUB_PRODUCER_CLIENT_BUILDER_BEAN_NAME);
                 }
             );
     }
@@ -110,6 +114,11 @@ void sharedConsumerInjectsRootBuilderWhenProducerHasDedicatedOverride() {
                 context -> {
                     assertThat(context).hasSingleBean(AzureEventHubsConsumerClientConfiguration.SharedConsumerConnectionConfiguration.class);
                     assertThat(context).doesNotHaveBean(AzureEventHubsConsumerClientConfiguration.DedicatedConsumerConnectionConfiguration.class);
+                    // Producer dedicated must be active so multiple EventHubClientBuilder beans coexist,
+                    // proving the shared consumer is selecting the root builder by qualifier rather than
+                    // succeeding by accident because only one builder bean exists.
+                    assertThat(context).hasSingleBean(AzureEventHubsProducerClientConfiguration.DedicatedProducerConnectionConfiguration.class);
+                    assertThat(context.getBeansOfType(EventHubClientBuilder.class)).hasSizeGreaterThan(1);
                     assertThat(context).hasBean(AzureContextUtils.EVENT_HUB_CLIENT_BUILDER_BEAN_NAME);
                     assertThat(context).hasSingleBean(EventHubConsumerClient.class);
                 }

sdk/spring/spring-cloud-azure-autoconfigure/src/test/java/com/azure/spring/cloud/autoconfigure/implementation/eventhubs/AzureEventHubsProducerClientConfigurationTests.java

@@ -87,6 +87,10 @@ void consumerOnlyDedicatedOverrideShouldNotActivateSharedProducer() {
                     assertThat(context).doesNotHaveBean(AzureEventHubsProducerClientConfiguration.class);
                     assertThat(context).doesNotHaveBean(EventHubProducerClient.class);
                     assertThat(context).doesNotHaveBean(EventHubProducerAsyncClient.class);
+                    // The consumer dedicated path must actually be activated for this test to exercise the
+                    // "asymmetric, dedicated builder bean exists" scenario the bug fix targets.
+                    assertThat(context).hasSingleBean(AzureEventHubsConsumerClientConfiguration.DedicatedConsumerConnectionConfiguration.class);
+                    assertThat(context).hasBean(AzureContextUtils.EVENT_HUB_CONSUMER_CLIENT_BUILDER_BEAN_NAME);
                 }
             );
     }
@@ -108,6 +112,11 @@ void sharedProducerInjectsRootBuilderWhenConsumerHasDedicatedOverride() {
                 context -> {
                     assertThat(context).hasSingleBean(AzureEventHubsProducerClientConfiguration.SharedProducerConnectionConfiguration.class);
                     assertThat(context).doesNotHaveBean(AzureEventHubsProducerClientConfiguration.DedicatedProducerConnectionConfiguration.class);
+                    // Consumer dedicated must be active so multiple EventHubClientBuilder beans coexist,
+                    // proving the shared producer is selecting the root builder by qualifier rather than
+                    // succeeding by accident because only one builder bean exists.
+                    assertThat(context).hasSingleBean(AzureEventHubsConsumerClientConfiguration.DedicatedConsumerConnectionConfiguration.class);
+                    assertThat(context.getBeansOfType(EventHubClientBuilder.class)).hasSizeGreaterThan(1);
                     assertThat(context).hasBean(AzureContextUtils.EVENT_HUB_CLIENT_BUILDER_BEAN_NAME);
                     assertThat(context).hasSingleBean(EventHubProducerClient.class);
                 }