Fix docker-compose executable check: renaming regarding review commen…

…ts (testcontainers#460)
bsnisar · Sep 21, 2017 · df95e7b · df95e7b
1 parent 61d9b0e
commit df95e7b
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/core/src/main/java/org/testcontainers/containers/DockerComposeContainer.java b/core/src/main/java/org/testcontainers/containers/DockerComposeContainer.java
@@ -494,7 +494,7 @@ public DockerCompose withEnv(Map<String, String> env) {
     @Override
     public void invoke() {
         // bail out early
-        if (!isDockerComposeExists()) {
+        if (!dockerComposeExecutableExists()) {
             throw new ContainerLaunchException("Local Docker Compose not found. Is " + COMPOSE_EXECUTABLE + " on the PATH?");
         }
 
@@ -531,9 +531,7 @@ public void invoke() {
         }
     }
 
-    // #460 : docker-compose are not resolvable on Win10 machines because there is docker-compose.exe binary,
-    // but not docker-compose
-    private boolean isDockerComposeExists() {
+    private boolean dockerComposeExecutableExists() {
         boolean dockerComposeExists = CommandLine.executableExists(COMPOSE_EXECUTABLE);
         if (!dockerComposeExists && SystemUtils.IS_OS_WINDOWS) {
             return CommandLine.executableExists(COMPOSE_EXECUTABLE + ".exe");