spring-cloud / spring-cloud-consul

Spring Cloud Consul
http://cloud.spring.io/spring-cloud-consul/
Apache License 2.0
813 stars 541 forks source link

spring.config.import in DynamicPropertySource is not contributed using ConfigDataApplicationContextInitializer in tests #832

Open eddumelendez opened 1 year ago

eddumelendez commented 1 year ago

Describe the bug If ConfigDataApplicationContextInitializer is added to tests then properties declared in DynamicPropertySource are not contributed. When I was debugging and put a breakpoint in this class, it was never called.

I did the same for spring-cloud-vault in this example. However, in this case it works.

Locally, I removed those lines and test it similar to the vault example, see changes here, and it worked. Looks like those lines were available since version 3.0.2.

The same happens for spring-cloud-zookeeper and spring-cloud-config. I haven't open issues for those projects yet but if you agree I can do it too.

Sample https://github.com/eddumelendez/testcontainers-samples/tree/spring-cloud-consul/spring-cloud-consul