From 3533bc8646b2cc2c75d5513be2940735934ec08d Mon Sep 17 00:00:00 2001 From: Per Lundberg Date: Tue, 16 Jan 2024 13:48:41 +0200 Subject: [PATCH] Fix missing WaitStrategy support in JdbcDatabaseContainer As pointed out in https://github.com/testcontainers/testcontainers-java/issues/2994, the fact that `JdbcDatabaseContainer` ignores any `WaitStrategy` set on the container is very confusing to the user. This change tries to make the public API less confusing. --- .../org/testcontainers/containers/JdbcDatabaseContainer.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java b/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java index 9533a4c7d10..c066aca1c64 100644 --- a/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java +++ b/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java @@ -140,6 +140,8 @@ public SELF withInitScript(String initScriptPath) { @SneakyThrows(InterruptedException.class) @Override protected void waitUntilContainerStarted() { + super.waitUntilContainerStarted(); + logger() .info( "Waiting for database connection to become available at {} using query '{}'",