@ServiceConnection is not found when used in an interface implemented by a test class

[therealaleko]

Using the new @ServiceConnection does not work as I would expect when applied within an interface. It starts the container on a random port rather than my spring defined port.

@DynamicPropertySource does work as I expect.

public interface PostgresContainer {
  @Container
  //  @ServiceConnection
  PostgreSQLContainer<?> postgreSQLContainer = new PostgreSQLContainer<>("postgres:11.1");

  @DynamicPropertySource
  static void initialize(DynamicPropertyRegistry registry) {
    registry.add("spring.datasource.url", postgreSQLContainer::getJdbcUrl);
    registry.add("spring.datasource.username", postgreSQLContainer::getUsername);
    registry.add("spring.datasource.password", postgreSQLContainer::getPassword);
  }
}

[eddumelendez]

It starts the container on a random port rather than my spring defined port.

Can you elaborate more? I think I didn't get it.

Take into account Testcontainers starts containers in random ports and @ServiceConnection maps the properties from the container to spring boot

[philwebb]

Thanks @eddumelendez, I was just writing the same thing!

@therealaleko perhaps you could provide a sample application that shows the problem?

[therealaleko]

Please check the repo here. Thanks!
https://github.com/therealaleko/spring_boot_issue_37671

@philwebb @eddumelendez

[eddumelendez]

Thanks! So, the ServiceConnection doesn't work when using interfaces

I think this line here

spring-boot/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/service/connection/ServiceConnectionContextCustomizerFactory.java

Line 57 in d310fb6

MergedAnnotations annotations = MergedAnnotations.from(field);

should change to MergedAnnotations annotations = MergedAnnotations.from(field, SearchStrategy.TYPE_HIERARCHY);

[andreasevers]

Would it also be possible to add support for inherited interfaces? At the moment the test class has to implement the interface directly, but ideally extending a superclass that implements the interface should work too.
I could submit a PR if this is something that could be accepted.

[philwebb]

Good spot @andreasevers, I've refined the search algorithm in e01e4f1

[andreasevers]

Wow, thanks Phil! This is great, using it right away!

[andreasevers]

Could this latest commit also get backported into 3.1.x perhaps?

[wilkinsona]

Good catch, @andreasevers. Thank you. I've done that in fcb75b6.

[andreasevers]

Much appreciated 🙏

[therealaleko]

Thank you!