test: close context (#518)

* test: close context * Update discovery-client/src/test/groovy/io/micronaut/discovery/consul/MockConfigurationDiscoverySpec.groovy Co-authored-by: Tim Yates <[email protected]> --------- Co-authored-by: Tim Yates <[email protected]>
micronaut-projects · Oct 6, 2023 · 8c495cd · 8c495cd
1 parent f14ec25
commit 8c495cd
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 1 deletion.
diff --git a/discovery-client/src/test/groovy/io/micronaut/discovery/EagerInitSpec.groovy b/discovery-client/src/test/groovy/io/micronaut/discovery/EagerInitSpec.groovy
@@ -7,10 +7,13 @@ class EagerInitSpec extends Specification {
 
     void 'test run with eager init'() {
         given:
-        def context = ApplicationContext.builder().eagerInitSingletons(true).eagerInitConfiguration(true)
+        ApplicationContext context = ApplicationContext.builder().eagerInitSingletons(true).eagerInitConfiguration(true)
                 .start()
 
         expect: "context starts up successfully"
         context != null
+
+        cleanup:
+        context.close()
     }
 }
diff --git a/...c/test/groovy/io/micronaut/discovery/consul/ConsulMockConfigurationClientFilesSpec.groovy b/...c/test/groovy/io/micronaut/discovery/consul/ConsulMockConfigurationClientFilesSpec.groovy
@@ -120,6 +120,9 @@ not:
         env.getRequiredProperty('datasource.driver', String) == 'java.SomeDriver'
         !env.getProperty('not.there', Boolean).isPresent()
         !env.getProperty('also.not.there', Boolean).isPresent()
+
+        cleanup:
+        applicationContext.close()
     }
 
 

diff --git a/...lient/src/test/groovy/io/micronaut/discovery/consul/MockConfigurationDiscoverySpec.groovy b/...lient/src/test/groovy/io/micronaut/discovery/consul/MockConfigurationDiscoverySpec.groovy
@@ -89,6 +89,9 @@ class MockConfigurationDiscoverySpec extends Specification {
 
         then:"The value is retrieved again"
         environment.getProperty('must.override1', String).get() == 'changed'
+
+        cleanup:
+        applicationContext.close()
     }
 
     void "test multiple environment precedence"() {
@@ -161,6 +164,8 @@ class MockConfigurationDiscoverySpec extends Specification {
         expect:
         !result.isPresent()
 
+        cleanup:
+        applicationContext.close()
     }
 
     private void writeValue(String env, String name, String value) {

diff --git a/discovery-client/src/test/groovy/io/micronaut/discovery/eureka/EurekaClientConfigSpec.groovy b/discovery-client/src/test/groovy/io/micronaut/discovery/eureka/EurekaClientConfigSpec.groovy
@@ -39,6 +39,9 @@ class EurekaClientConfigSpec extends Specification {
         expect:
         serviceInstances == result
 
+        cleanup:
+        applicationContext.close()
+
         where:
         value                           | result
         'localhost:8087'                | [newServiceInstance("http://$value")]

diff --git a/...ent/src/test/groovy/io/micronaut/discovery/eureka/health/EurekaHealthIndicatorSpec.groovy b/...ent/src/test/groovy/io/micronaut/discovery/eureka/health/EurekaHealthIndicatorSpec.groovy
@@ -51,5 +51,6 @@ class EurekaHealthIndicatorSpec extends Specification {
 
         cleanup:
         eurekaServer?.stop()
+        applicationContext.close()
     }
 }