Fixes testcontainers#543 by adding JdbcDatabaseContainer(Future) cons…

…tructor
iNikem · Feb 13, 2018 · 1f5f83b · 1f5f83b
1 parent 0c6ea40
commit 1f5f83b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@ All notable changes to this project will be documented in this file.
 
 ### Changed
 - Abstracted and changed database init script functionality to support use of SQL-like scripts with non-JDBC connections. ([\#551](https://github.com/testcontainers/testcontainers-java/pull/551))
+- Added `JdbcDatabaseContainer(Future)` constructor. ([\#543](https://github.com/testcontainers/testcontainers-java/issues/543))
 
 ## [1.6.0] - 2018-01-28
 

diff --git a/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java b/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java
@@ -1,5 +1,7 @@
 package org.testcontainers.containers;
 
+import java.util.concurrent.Future;
+import lombok.NonNull;
 import org.jetbrains.annotations.NotNull;
 import org.rnorth.ducttape.ratelimits.RateLimiter;
 import org.rnorth.ducttape.ratelimits.RateLimiterBuilder;
@@ -31,10 +33,14 @@ public abstract class JdbcDatabaseContainer<SELF extends JdbcDatabaseContainer<S
             .withConstantThroughput()
             .build();
 
-    public JdbcDatabaseContainer(String dockerImageName) {
+    public JdbcDatabaseContainer(@NonNull final String dockerImageName) {
         super(dockerImageName);
     }
 
+    public JdbcDatabaseContainer(@NonNull final Future<String> image) {
+        super(image);
+    }
+
     /**
      * @return the name of the actual JDBC driver to use
      */