From ea345f6421a865e185e4c39ebb43a7a1def48280 Mon Sep 17 00:00:00 2001
From: Trustin Lee <t@motd.kr>
Date: Fri, 7 Feb 2014 14:18:39 -0800
Subject: [PATCH] (Kind of) skip autobahn testsuite on -DskipTests

- autobahntestsuite-maven-plugin does not provide a way to skip all
  tests, so we run only one simple test when -DskipTests is set
---
 testsuite/pom.xml | 91 ++++++++++++++++++++++++++++-------------------
 1 file changed, 54 insertions(+), 37 deletions(-)

diff --git a/testsuite/pom.xml b/testsuite/pom.xml
index db105d0473..0aae021871 100644
--- a/testsuite/pom.xml
+++ b/testsuite/pom.xml
@@ -71,12 +71,44 @@
     </dependency>
   </dependencies>
 
+  <!--
+    autobahntestsuite-maven-plugin lacks an option to skip the testsuite,
+    so we run the least possible number of tests when skipTests or skipAutobahnTestsuite is set.
+  -->
+  <properties>
+    <enabledCase>*</enabledCase>
+  </properties>
+
+  <profiles>
+    <profile>
+      <id>skipTests</id>
+      <activation>
+        <property>
+          <name>skipTests</name>
+        </property>
+      </activation>
+      <properties>
+        <enabledCase>1.1.1</enabledCase>
+      </properties>
+    </profile>
+    <profile>
+      <id>skipAutobahnTestsuite</id>
+      <activation>
+        <property>
+          <name>skipAutobahnTestsuite</name>
+        </property>
+      </activation>
+      <properties>
+        <enabledCase>1.1.1</enabledCase>
+      </properties>
+    </profile>
+  </profiles>
+
   <build>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
-        <version>2.4</version>
         <executions>
           <execution>
             <goals>
@@ -85,44 +117,29 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>me.normanmaurer.maven.autobahntestsuite</groupId>
+        <artifactId>autobahntestsuite-maven-plugin</artifactId>
+        <version>0.1.1</version>
+        <configuration>
+          <mainClass>io.netty.testsuite.websockets.autobahn.AutobahnServer</mainClass>
+          <cases>
+            <case>${enabledCase}</case>
+          </cases>
+          <excludeCases/>
+          <failOnNonStrict>false</failOnNonStrict>
+        </configuration>
+        <executions>
+          <execution>
+            <phase>test</phase>
+            <goals>
+              <goal>fuzzingclient</goal>
+            </goals>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
   </build>
 
-  <profiles>
-    <profile>
-      <!-- Only run the autobahntestsuite if it is not disabled -->
-      <id>autobahntestsuite</id>
-      <activation>
-        <property>
-          <name>!skipAutobahnTestsuite</name>
-        </property>
-      </activation>
-      <build>
-        <plugins>
-          <plugin>
-            <groupId>me.normanmaurer.maven.autobahntestsuite</groupId>
-            <artifactId>autobahntestsuite-maven-plugin</artifactId>
-            <version>0.1.1</version>
-            <configuration>
-              <mainClass>io.netty.testsuite.websockets.autobahn.AutobahnServer</mainClass>
-              <cases>
-                <case>*</case>
-              </cases>
-              <excludeCases></excludeCases>
-              <failOnNonStrict>false</failOnNonStrict>
-            </configuration>
-            <executions>
-              <execution>
-                <phase>test</phase>
-                <goals>
-                  <goal>fuzzingclient</goal>
-                </goals>
-              </execution>
-            </executions>
-          </plugin>
-        </plugins>
-      </build>
-    </profile>
-  </profiles>
 </project>