You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ol...@apache.org on 2019/09/09 07:29:47 UTC

[sling-org-apache-sling-clam] branch master updated: SLING-8688 Enable integration tests with Docker

This is an automated email from the ASF dual-hosted git repository.

olli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-clam.git


The following commit(s) were added to refs/heads/master by this push:
     new a8f8e13  SLING-8688 Enable integration tests with Docker
a8f8e13 is described below

commit a8f8e13e42b89bf36cb9f69779e402adf568fb94
Author: Oliver Lietz <ol...@apache.org>
AuthorDate: Mon Sep 9 09:29:35 2019 +0200

    SLING-8688 Enable integration tests with Docker
    
    * Drop profile it and execute integration tests during default build
    * Increase container startup timeout from 3 to 5 minutes
---
 pom.xml                                            | 54 +++++++++-------------
 .../clam/it/support/ClamContainerFactory.java      |  2 +-
 2 files changed, 23 insertions(+), 33 deletions(-)

diff --git a/pom.xml b/pom.xml
index 5d8449d..44ae878 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,41 +62,31 @@
         <groupId>org.apache.servicemix.tooling</groupId>
         <artifactId>depends-maven-plugin</artifactId>
       </plugin>
+      <plugin>
+        <!-- integration tests require a running clam daemon -->
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-failsafe-plugin</artifactId>
+        <executions>
+          <execution>
+            <goals>
+              <goal>integration-test</goal>
+              <goal>verify</goal>
+            </goals>
+          </execution>
+        </executions>
+        <configuration>
+          <redirectTestOutputToFile>true</redirectTestOutputToFile>
+          <systemProperties>
+            <property>
+              <name>bundle.filename</name>
+              <value>${basedir}/target/${project.build.finalName}.jar</value>
+            </property>
+          </systemProperties>
+        </configuration>
+      </plugin>
     </plugins>
   </build>
 
-  <profiles>
-    <profile>
-      <!-- integration tests require a running clam daemon -->
-      <id>it</id>
-      <build>
-        <plugins>
-          <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-failsafe-plugin</artifactId>
-            <executions>
-              <execution>
-                <goals>
-                  <goal>integration-test</goal>
-                  <goal>verify</goal>
-                </goals>
-              </execution>
-            </executions>
-            <configuration>
-              <redirectTestOutputToFile>true</redirectTestOutputToFile>
-              <systemProperties>
-                <property>
-                  <name>bundle.filename</name>
-                  <value>${basedir}/target/${project.build.finalName}.jar</value>
-                </property>
-              </systemProperties>
-            </configuration>
-          </plugin>
-        </plugins>
-      </build>
-    </profile>
-  </profiles>
-
   <dependencies>
     <!-- javax -->
     <dependency>
diff --git a/src/test/java/org/apache/sling/clam/it/support/ClamContainerFactory.java b/src/test/java/org/apache/sling/clam/it/support/ClamContainerFactory.java
index 26dab1a..42c1e9a 100644
--- a/src/test/java/org/apache/sling/clam/it/support/ClamContainerFactory.java
+++ b/src/test/java/org/apache/sling/clam/it/support/ClamContainerFactory.java
@@ -28,7 +28,7 @@ public class ClamContainerFactory {
 
     public static GenericContainer container = new GenericContainer<>(CLAM_CONTAINER_IMAGE_NAME)
         .withExposedPorts(3310)
-        .withStartupTimeout(Duration.ofMinutes(3));
+        .withStartupTimeout(Duration.ofMinutes(5));
 
     public static void startContainer() {
         if (!container.isRunning()) {