Make Testcontainers tests conditional via Maven profile

Disabled for Windows Server (CI) cause its windows containers are not supported by Testcontainers.

Author: m-dzianishchyts

.github/workflows/maven.yml

@@ -30,4 +30,4 @@ jobs:
           cache: 'maven'
 
       - name: Build and test
-        run: mvn clean verify -U
+        run: mvn clean verify -U ${{ matrix.os != 'windows-latest' && '-Pdocker' || '' }}

jjava-distro/src/test/java/org/dflib/jjava/distro/ContainerizedKernelCase.java

@@ -1,5 +1,6 @@
 package org.dflib.jjava.distro;
 
+import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.BeforeAll;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -24,7 +25,7 @@ public abstract class ContainerizedKernelCase {
 
     private static final Logger LOGGER = LoggerFactory.getLogger(ContainerizedKernelCase.class);
 
-    protected static final GenericContainer<?> container;
+    protected static GenericContainer<?> container;
     protected static final String WORKING_DIRECTORY = "/test";
     protected static final String CONTAINER_KERNELSPEC = "/usr/share/jupyter/kernels/java";
     protected static final String CONTAINER_RESOURCES = WORKING_DIRECTORY + "/resources";
@@ -34,20 +35,15 @@ public abstract class ContainerizedKernelCase {
     private static final String FS_KERNELSPEC = "../kernelspec/java";
     private static final String FS_RESOURCES = "src/test/resources";
 
-    static {
-        container = new GenericContainer<>(BASE_IMAGE)
-                .withWorkingDirectory(WORKING_DIRECTORY)
-                .withCopyToContainer(MountableFile.forHostPath(FS_KERNELSPEC), CONTAINER_KERNELSPEC)
-                .withCopyToContainer(MountableFile.forHostPath(FS_RESOURCES), CONTAINER_RESOURCES)
-                .withCommand("bash", "-c", getStartupCommand())
-                .withLogConsumer(new Slf4jLogConsumer(LOGGER))
-                .waitingFor(Wait.forSuccessfulCommand(getSuccessfulCommand()))
-                .withStartupTimeout(Duration.ofMinutes(1));
-        container.start();
-    }
+    private static final String TESTS_ENABLED_PROPERTY = "docker.tests.enabled";
 
     @BeforeAll
-    static void compileSources() throws IOException, InterruptedException {
+    static void setUp() throws IOException, InterruptedException {
+        System.out.println("ContainerizedKernelCase.setUp");
+        initializeContainer();
+        Assumptions.assumeTrue(container != null, "Docker tests are disabled. Enable with -Pdocker");
+        System.out.println("container = " + container);
+
         String source = "$(find " + CONTAINER_RESOURCES + "/src -name '*.java')";
         Container.ExecResult compileResult = executeInContainer("javac -d " + TEST_CLASSPATH + " " + source);
 
@@ -85,6 +81,22 @@ protected static Container.ExecResult executeInKernel(String snippet, Map<String
         return execResult;
     }
 
+    private static void initializeContainer() {
+        if (container != null || !"true".equals(System.getProperty(TESTS_ENABLED_PROPERTY))) {
+            return;
+        }
+
+        container = new GenericContainer<>(BASE_IMAGE)
+                .withWorkingDirectory(WORKING_DIRECTORY)
+                .withCopyToContainer(MountableFile.forHostPath(FS_KERNELSPEC), CONTAINER_KERNELSPEC)
+                .withCopyToContainer(MountableFile.forHostPath(FS_RESOURCES), CONTAINER_RESOURCES)
+                .withCommand("bash", "-c", getStartupCommand())
+                .withLogConsumer(new Slf4jLogConsumer(LOGGER))
+                .waitingFor(Wait.forSuccessfulCommand(getSuccessfulCommand()))
+                .withStartupTimeout(Duration.ofMinutes(1));
+        container.start();
+    }
+
     private static String getStartupCommand() {
         return String.join(" && ",
                 "apt-get update",

pom.xml

@@ -227,6 +227,11 @@
                         </goals>
                     </execution>
                 </executions>
+                <configuration>
+                    <systemPropertyVariables>
+                        <docker.tests.enabled>${docker.tests.enabled}</docker.tests.enabled>
+                    </systemPropertyVariables>
+                </configuration>
             </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
@@ -309,5 +314,11 @@
                 </plugins>
             </build>
         </profile>
+        <profile>
+            <id>docker</id>
+            <properties>
+                <docker.tests.enabled>true</docker.tests.enabled>
+            </properties>
+        </profile>
     </profiles>
 </project>