From 5bd89949fac66dc876d35debb7338c886ef2a7e2 Mon Sep 17 00:00:00 2001
From: Simon Pamies <spamsch@gmail.com>
Date: Fri, 9 Aug 2013 20:31:50 +0200
Subject: [PATCH] Making it possible to disable selenium tests also reordered
 modules to have better overview over test setup

---
 configuration/buildAndUpdateProduct.sh     |  4 +-
 java/com.sap.sailing.selenium.test/pom.xml |  2 +-
 java/pom.xml                               | 64 ++++++++++++----------
 3 files changed, 38 insertions(+), 32 deletions(-)

diff --git a/configuration/buildAndUpdateProduct.sh b/configuration/buildAndUpdateProduct.sh
index 1118c1f5961..e5cd961902d 100755
--- a/configuration/buildAndUpdateProduct.sh
+++ b/configuration/buildAndUpdateProduct.sh
@@ -277,7 +277,9 @@ if [[ "$@" == "build" ]] || [[ "$@" == "all" ]]; then
 
 	if [ $testing -eq 0 ]; then
 	    echo "INFO: Skipping tests"
-	    extra="$extra -Dmaven.test.skip=true"
+	    extra="$extra -Dmaven.test.skip=true -DskipTests=true"
+    else
+        extra="$extra -DskipTests=false"
 	fi
 
 	if [ $offline -eq 1 ]; then
diff --git a/java/com.sap.sailing.selenium.test/pom.xml b/java/com.sap.sailing.selenium.test/pom.xml
index 216da7360a0..24dca9e67dd 100644
--- a/java/com.sap.sailing.selenium.test/pom.xml
+++ b/java/com.sap.sailing.selenium.test/pom.xml
@@ -41,7 +41,7 @@
                             <goal>test</goal>
                         </goals>
                         <configuration>
-                            <skip>false</skip>
+                            <skip>${skipTests}</skip>
                             <excludes>  
                                <exclude>**/core/**</exclude>  
                             </excludes>
diff --git a/java/pom.xml b/java/pom.xml
index 29746223294..7efca35109e 100755
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -16,61 +16,68 @@
 	<modules>
 		<module>com.sap.sailing.domain.common</module>
 		<module>com.sap.sailing.domain.shared.android</module>
-		<module>com.sap.sailing.domain.shared.android.test</module>
 		<module>com.sap.sailing.domain</module>
 		<module>com.sap.sailing.operationaltransformation</module>
-		<module>com.sap.sailing.operationaltransformation.test</module>
 		<module>com.sap.sailing.geocoding</module>
-		<module>com.sap.sailing.geocoding.test</module>
 		<module>com.sap.sailing.domain.persistence</module>
-		<module>com.sap.sailing.domain.test</module>
 		<module>com.sap.sailing.domain.tractracadapter</module>
-		<module>com.sap.sailing.domain.tractracadapter.test</module>
 		<module>com.sap.sailing.domain.tractracadapter.persistence</module>
 		<module>com.sap.sailing.domain.swisstimingadapter</module>
 		<module>com.sap.sailing.domain.swisstimingadapter.persistence</module>
-		<module>com.sap.sailing.domain.swisstimingadapter.test</module>
 		<module>com.sap.sailing.domain.swisstimingreplayadapter</module>
-		<module>com.sap.sailing.domain.swisstimingreplayadapter.test</module>
 		<module>com.sap.sailing.declination</module>
-		<module>com.sap.sailing.declination.test</module>
 		<module>com.sap.sailing.udpconnector</module>
 		<module>com.sap.sailing.expeditionconnector</module>
-		<module>com.sap.sailing.expeditionconnector.test</module>
 		<module>com.sap.sailing.server</module>
 		<module>com.sap.sailing.server.gateway</module>
-		<module>com.sap.sailing.server.gateway.test</module>
 		<module>com.sap.sailing.server.gateway.ess40</module>
 		<module>com.sap.sailing.server.gateway.serialization</module>
-		<module>com.sap.sailing.server.gateway.serialization.test</module>
 		<module>com.sap.sailing.server.replication</module>
-		<module>com.sap.sailing.server.replication.test</module>
 		<module>com.sap.sailing.www</module>
 		<module>com.sap.sailing.www.events</module>
 		<module>com.sap.sailing.xcelsiusadapter</module>
-	        <module>com.sap.sailing.gwt.ui</module>
-       	        <module>com.sap.sailing.gwt.ui.test</module>
 		<module>com.google.gwt.osgi</module>
-		<module>com.sap.sailing.server.test</module>
 		<module>com.sap.sailing.mongodb</module>
-		<module>com.sap.sailing.mongodb.test</module>
 		<module>com.tractrac.clientmodule</module>
 		<module>org.json.simple</module>
 		<module>com.mongodb.driver</module>
 		<module>com.googlecode.java-diff-utils</module>
-		<module>com.googlecode.java-diff-utils.test</module>
 		<module>com.sap.sailing.feature</module>
-		<module>com.sap.sailing.feature.runtime</module>
-		<module>com.sap.sailing.feature.p2build</module>
 		<module>org.moxieapps.gwt.highcharts</module>
 		<module>com.sap.sailing.resultimport</module>
 		<module>com.sap.sailing.kiworesultimport</module>
-		<module>com.sap.sailing.kiworesultimport.test</module>
 		<module>com.sap.sailing.ess40.resultimport</module>
-		<module>com.sap.sailing.ess40.resultimport.test</module>
 		<module>com.sap.sailing.xrr.resultimport</module>
-		<module>com.sap.sailing.xrr.resultimport.test</module>
 		<module>com.sap.sailing.freg.resultimport</module>
+		<module>org.openqa.selenium.osgi</module>
+		<module>com.sap.sailing.monitoring</module>
+		<module>com.sap.sailing.nmeaconnector</module>
+		<module>net.sf.marineapi</module>
+		<module>net.sf.marineapi.test</module>
+		<module>com.sap.sailing.simulator</module>
+		<module>com.sap.sailing.server.trackfiles</module>
+		<module>com.sap.sailing.feature.runtime</module>
+	    <module>com.sap.sailing.gwt.ui</module>
+
+		<module>com.sap.sailing.domain.tractracadapter.test</module>
+		<module>com.sap.sailing.domain.test</module>
+		<module>com.sap.sailing.domain.shared.android.test</module>
+		<module>com.sap.sailing.operationaltransformation.test</module>
+		<module>com.sap.sailing.geocoding.test</module>
+		<module>com.sap.sailing.domain.swisstimingadapter.test</module>
+		<module>com.sap.sailing.domain.swisstimingreplayadapter.test</module>
+		<module>com.sap.sailing.declination.test</module>
+		<module>com.sap.sailing.expeditionconnector.test</module>
+		<module>com.sap.sailing.server.test</module>
+		<module>com.sap.sailing.server.gateway.test</module>
+		<module>com.sap.sailing.server.gateway.serialization.test</module>
+		<module>com.sap.sailing.server.replication.test</module>
+       	<module>com.sap.sailing.gwt.ui.test</module>
+		<module>com.sap.sailing.mongodb.test</module>
+		<module>com.googlecode.java-diff-utils.test</module>
+		<module>com.sap.sailing.kiworesultimport.test</module>
+		<module>com.sap.sailing.ess40.resultimport.test</module>
+		<module>com.sap.sailing.xrr.resultimport.test</module>
 		<module>com.sap.sailing.freg.resultimport.test</module>
 		<module>com.sap.sailing.barbados.resultimport</module>
 		<module>com.sap.sailing.barbados.resultimport.test</module>
@@ -80,17 +87,13 @@
 		<module>com.sap.sailing.sailwave.resultimport.test</module>
 		<module>com.sap.sailing.manage2sail.resultimport</module>
 		<module>com.sap.sailing.manage2sail.resultimport.test</module>
-		<module>com.sap.sailing.selenium.test</module>
-		<module>org.openqa.selenium.osgi</module>
-		<module>com.sap.sailing.monitoring</module>
-		<module>com.sap.sailing.nmeaconnector</module>
 		<module>com.sap.sailing.nmeaconnector.test</module>
-		<module>net.sf.marineapi</module>
-		<module>net.sf.marineapi.test</module>
-		<module>com.sap.sailing.simulator</module>
 		<module>com.sap.sailing.simulator.test</module>
-		<module>com.sap.sailing.server.trackfiles</module>
 		<module>com.sap.sailing.server.trackfiles.test</module>
+
+		<module>com.sap.sailing.selenium.test</module>
+
+		<module>com.sap.sailing.feature.p2build</module>
 	</modules>
 	
 	<properties>
@@ -126,6 +129,7 @@
         -->
         <parameters.proxy>
         </parameters.proxy>
+        <skipTests>false</skipTests>
 	</properties>
 
     <profiles>