Adjust auto-configuration to register at most one Vault health indica…

…tor.

Fixes gh-676
Original pull request: gh-677

spring-cloud-vault-config/src/main/java/org/springframework/cloud/vault/config/VaultHealthIndicatorAutoConfiguration.java

@@ -30,7 +30,7 @@
  * {@link org.springframework.boot.autoconfigure.EnableAutoConfiguration
  * Auto-configuration} for Vault providing beans for the application context.
  *
- * @author Mark Paluch
+ * @author Mark Paluch, Rastislav Zlacky
  * @since 2.0
  */
 @Configuration(proxyBeanMethods = false)
@@ -39,7 +39,7 @@
 @ConditionalOnProperty(name = "spring.cloud.vault.enabled", matchIfMissing = true)
 @AutoConfigureBefore(HealthContributorAutoConfiguration.class)
 @AutoConfigureAfter({ VaultAutoConfiguration.class, VaultReactiveAutoConfiguration.class })
-@Import({ VaultHealthIndicatorConfiguration.class, VaultReactiveHealthIndicatorConfiguration.class })
+@Import({ VaultReactiveHealthIndicatorConfiguration.class, VaultHealthIndicatorConfiguration.class })
 public class VaultHealthIndicatorAutoConfiguration {
 
 }

spring-cloud-vault-config/src/main/java/org/springframework/cloud/vault/config/VaultReactiveHealthIndicatorConfiguration.java

@@ -32,7 +32,7 @@
 /**
  * Configuration for {@link VaultReactiveHealthIndicator}.
  *
- * @author Mark Paluch
+ * @author Mark Paluch, Rastislav Zlacky
  * @since 2.0
  */
 @Configuration(proxyBeanMethods = false)
@@ -47,9 +47,9 @@ class VaultReactiveHealthIndicatorConfiguration
 		this.reactiveVaultTemplates = reactiveVaultTemplates;
 	}
 
-	@Bean
-	@ConditionalOnMissingBean(name = { "vaultReactiveHealthIndicator" })
-	ReactiveHealthContributor vaultReactiveHealthIndicator() {
+	@Bean(name = { "vaultHealthIndicator", "vaultReactiveHealthIndicator" })
+	@ConditionalOnMissingBean(name = { "vaultHealthIndicator" })
+	ReactiveHealthContributor vaultHealthIndicator() {
 		return createContributor(this.reactiveVaultTemplates);
 	}
 

spring-cloud-vault-config/src/test/java/org/springframework/cloud/vault/config/VaultHealthIndicatorAutoConfigurationTests.java

@@ -30,7 +30,7 @@
 /**
  * Unit tests for {@link VaultHealthIndicatorAutoConfiguration}.
  *
- * @author Mark Paluch
+ * @author Mark Paluch, Rastislav Zlacky
  */
 class VaultHealthIndicatorAutoConfigurationTests {
 
@@ -62,6 +62,17 @@ void shouldConfigureReactiveHealthIndicator() {
 
 	}
 
+	@Test
+	void shouldConfigureSingleHealthIndicator() {
+
+		this.contextRunner.withUserConfiguration(ImperativeConfiguration.class, ReactiveConfiguration.class)
+				.run(context -> {
+					assertThat(context).hasBean("vaultHealthIndicator")
+							.hasSingleBean(VaultReactiveHealthIndicator.class)
+							.doesNotHaveBean(VaultHealthIndicator.class);
+				});
+	}
+
 	static class ImperativeConfiguration {
 
 		@Bean
@@ -74,7 +85,7 @@ VaultOperations vaultOperations() {
 	static class ReactiveConfiguration {
 
 		@Bean
-		ReactiveVaultOperations vaultOperations() {
+		ReactiveVaultOperations reactiveVaultOperations() {
 			return mock(ReactiveVaultOperations.class);
 		}