Put all docker compose files into environment before starting

[zbstof]

Currently, only first docker-compose entry is put into environment:

new DockerComposeContainer(
                new File(Initializer.class.getResource("/docker/docker-compose.yml").toURI()),
                new File(Initializer.class.getResource("/docker/docker-compose-app.yml").toURI()))
              ...
                .withLocalCompose(true)
                .withPull(false);

will fail, when started, to the effect that properties from second docker-compose weren't applied

[rnorth]

Ah - good catch, and sorry if this caused you some problems. This looks good to me.

I think I'd like to have a test class to demonstrate that the problem is resolved. Is that something you could perhaps add?

[zbstof]

I can try, I'm not good with grade, though. Which command should I run to compile and run tests?

…

On Fri, Jun 15, 2018, 18:34 Richard North ***@***.***> wrote: ***@***.**** requested changes on this pull request. Ah - good catch, and sorry if this caused you some problems. This looks good to me. I think I'd like to have a test class to demonstrate that the problem is resolved. Is that something you could perhaps add? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#755 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABT6A2_3YaaQXplsYOEdD21FDiACCsMrks5t89P5gaJpZM4UpTTq> .

[rnorth]

Thanks - if you run ./gradlew check (Linux/MacOS) or gradlew.bat check (if on Windows) that should work!

The tests take quite a while to run, so I'd recommend just running your own test class in your IDE if you can while developing it.

Thanks again.

[zbstof]

I think this test will require docker-compose to be present on build service?
Is this the case for Circle-CI?

Also, Intellij fails compilation, and from command-line I cannot run single test for some reason:

./gradlew test --tests org.testcontainers.junit.DockerComposeLocalOverrideTest
> Task :couchbase:test FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':couchbase:test'.
> No tests found for given includes: [org.testcontainers.junit.DockerComposeLocalOverrideTest](--tests filter)

DockerComposeLocalOverrideTest - this is my test

[zbstof]

Ok, with new test, but without the fix:

Gradle Test Executor 6 > org.testcontainers.junit.DockerComposeLocalOverrideTest > testEnvVar FAILED
    org.rnorth.ducttape.TimeoutException: Timeout waiting for result with exception
        at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:51)
        at org.rnorth.ducttape.unreliables.Unreliables.retryUntilTrue(Unreliables.java:95)
        at org.testcontainers.junit.DockerComposeLocalOverrideTest.testEnvVar(DockerComposeLocalOverrideTest.java:54)

        Caused by:
        java.lang.RuntimeException: Not ready yet

With fix test passes

[zbstof]

BTW, could you please release this as a patch release, if this is not too much trouble?
We need this feature in my company. For some reason dockerized compose didn't work for us.

[rnorth]

Thanks @zbstof, I'll have a look now!

[rnorth]

I'm just going to push a slight change if that's OK; I've combined three similar test classes into one class that has parameterized tests.

This was something that needed to be done before your change, it's just a bit more necessary now.

[rnorth]

Thanks for this @zbstof - I've merged into master

[rnorth]

Released in 1.8.1! 🎉

Thanks for your contribution @zbstof!