Segregate tests into client-, server-, conformance-tests

Author: Michael Bridgen

build.xml

@@ -327,9 +327,9 @@
   	</fail>
   </target>
 
-  <target name="test-suite-run" depends="test, test-ssl, test-persister-restart, test-functional, test-main-silent"/>
+  <target name="test-suite-run" depends="test-client, test-ssl, test-server, test-functional, test-main-silent"/>
 
-  <target name="test" depends="test-build">
+  <target name="test-client" depends="test-build">
     <junit printSummary="withOutAndErr"
         haltOnFailure="${haltOnFailureJunit}"
         failureproperty="test.failure"
@@ -338,7 +338,7 @@
 
       <formatter type="plain"/>
       <formatter type="xml"/>
-      <test todir="${build.out}" name="com.rabbitmq.client.test.AllTest"/>
+      <test todir="${build.out}" name="com.rabbitmq.client.test.ClientTests"/>
     </junit>
   </target>
 
@@ -375,7 +375,7 @@
     </junit>
   </target>
 
-  <target name="test-persister-restart" depends="test-build">
+  <target name="test-server" depends="test-build">
     <junit printSummary="withOutAndErr"
         haltOnFailure="${haltOnFailureJunit}"
         failureproperty="test.failure"
@@ -384,7 +384,7 @@
       <formatter type="plain"/>
       <formatter type="xml"/>
       <test todir="${build.out}"
-        name="com.rabbitmq.client.test.functional.PersisterRestartTests"/>
+        name="com.rabbitmq.client.test.server.ServerTests"/>
     </junit>
   </target>
 

test/src/com/rabbitmq/client/test/functional/BrokerTestCase.java renamed to test/src/com/rabbitmq/client/test/BrokerTestCase.java

@@ -29,7 +29,7 @@
 //   Contributor(s): ______________________________________.
 //
 
-package com.rabbitmq.client.test.functional;
+package com.rabbitmq.client.test;
 
 import java.io.IOException;
 

test/src/com/rabbitmq/client/test/functional/Bug20004Test.java renamed to test/src/com/rabbitmq/client/test/Bug20004Test.java

@@ -28,8 +28,9 @@
 //
 //   Contributor(s): ______________________________________.
 //
-package com.rabbitmq.client.test.functional;
+package com.rabbitmq.client.test;
 
+import com.rabbitmq.client.test.functional.*;
 import java.io.IOException;
 
 /**

test/src/com/rabbitmq/client/test/AllTest.java renamed to test/src/com/rabbitmq/client/test/ClientTests.java

@@ -34,16 +34,17 @@
 import junit.framework.TestCase;
 import junit.framework.TestSuite;
 
-public class AllTest extends TestCase {
+public class ClientTests extends TestCase {
     public static TestSuite suite() {
-        TestSuite suite = new TestSuite("all");
+        TestSuite suite = new TestSuite("client");
         suite.addTest(TableTest.suite());
         suite.addTest(BlockingCellTest.suite());
         suite.addTest(TruncatedInputStreamTest.suite());
         suite.addTest(AMQConnectionTest.suite());
         suite.addTest(ValueOrExceptionTest.suite());
         suite.addTest(BrokenFramesTest.suite());
         suite.addTest(ClonePropertiesTest.suite());
+        suite.addTestSuite(Bug20004Test.class);
         return suite;
     }
 }

test/src/com/rabbitmq/client/test/functional/AlternateExchange.java

@@ -31,6 +31,7 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.BrokerTestCase;
 import com.rabbitmq.client.AMQP;
 import com.rabbitmq.client.ReturnListener;
 import com.rabbitmq.client.GetResponse;

test/src/com/rabbitmq/client/test/functional/DoubleDeletion.java

@@ -31,6 +31,7 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.BrokerTestCase;
 import com.rabbitmq.client.AMQP;
 import com.rabbitmq.client.ShutdownSignalException;
 import java.io.IOException;

test/src/com/rabbitmq/client/test/functional/DurableOnTransient.java

@@ -31,6 +31,7 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.BrokerTestCase;
 import java.io.IOException;
 
 import com.rabbitmq.client.GetResponse;

test/src/com/rabbitmq/client/test/functional/FunctionalTests.java

@@ -31,6 +31,9 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.server.BindingLifecycle;
+import com.rabbitmq.client.test.Bug20004Test;
+import com.rabbitmq.client.test.server.Permissions;
 import junit.framework.TestCase;
 import junit.framework.TestSuite;
 
@@ -40,7 +43,6 @@ public static TestSuite suite() {
         suite.addTestSuite(Tables.class);
         suite.addTestSuite(DoubleDeletion.class);
         suite.addTestSuite(Routing.class);
-        suite.addTestSuite(BindingLifecycle.class);
         suite.addTestSuite(Transactions.class);
         suite.addTestSuite(PersistentTransactions.class);
         suite.addTestSuite(RequeueOnConnectionClose.class);
@@ -49,7 +51,6 @@ public static TestSuite suite() {
         suite.addTestSuite(NoRequeueOnCancel.class);
         suite.addTestSuite(Bug20004Test.class);
         suite.addTestSuite(QosTests.class);
-        suite.addTestSuite(Permissions.class);
         suite.addTestSuite(AlternateExchange.class);
         return suite;
     }

test/src/com/rabbitmq/client/test/functional/NoRequeueOnCancel.java

@@ -31,6 +31,7 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.BrokerTestCase;
 import java.io.IOException;
 
 import com.rabbitmq.client.QueueingConsumer;

test/src/com/rabbitmq/client/test/functional/QosTests.java

@@ -31,6 +31,7 @@
 
 package com.rabbitmq.client.test.functional;
 
+import com.rabbitmq.client.test.BrokerTestCase;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Arrays;