Merge pull request #45 from avast/CompileTestsBeforeComposeUp

Execute classes tasks before the composeUp task

Author: alenkacz

src/main/groovy/com/avast/gradle/dockercompose/ComposeExtension.groovy

@@ -40,6 +40,10 @@ class ComposeExtension {
     void isRequiredBy(Task task) {
         task.dependsOn upTask
         task.finalizedBy downTask
+        def ut = upTask // to access private field from closure
+        task.getTaskDependencies().getDependencies(task)
+            .findAll { Task.class.isAssignableFrom(it.class) && ((Task)it).name.toLowerCase().contains('classes') }
+            .each { ut.shouldRunAfter it }
     }
 
     Map<String, ServiceInfo> getServicesInfos() {

src/test/groovy/com/avast/gradle/dockercompose/DockerComposePluginTest.groovy

@@ -31,6 +31,17 @@ class DockerComposePluginTest extends Specification {
         task.getFinalizedBy().getDependencies(task).any { it == project.tasks.composeDown }
     }
 
+    def "isRequiredBy ensures right order of tasks"() {
+        def project = ProjectBuilder.builder().build()
+        project.plugins.apply 'docker-compose'
+        project.plugins.apply 'java'
+        when:
+        project.dockerCompose.isRequiredBy(project.tasks.test)
+        then:
+        project.tasks.composeUp.shouldRunAfter.values.any { it == project.tasks.testClasses }
+        noExceptionThrown()
+    }
+
     def "allows usage from integration test"() {
         def projectDir = new TmpDirTemporaryFileProvider().createTemporaryDirectory("gradle", "projectDir")
         new File(projectDir, 'docker-compose.yml') << '''