Author: kkolinko
Date: Sat Nov  7 16:47:25 2015
New Revision: 1713153

URL: http://svn.apache.org/viewvc?rev=1713153&view=rev
Log:
Convert test classes to JUnit 4 and apply some trivial fixes & formatting from 
Tomcat 7
Non-trivial fixes will be in separate commits

Modified:
    
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelMemberArrival.java
    
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelOptionFlag.java
    
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
    
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelStartStop.java

Modified: 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelMemberArrival.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelMemberArrival.java?rev=1713153&r1=1713152&r2=1713153&view=diff
==============================================================================
--- 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelMemberArrival.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelMemberArrival.java
 Sat Nov  7 16:47:25 2015
@@ -18,26 +18,29 @@ package org.apache.catalina.tribes.group
 
 import java.util.ArrayList;
 
+import static org.junit.Assert.assertEquals;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
 import org.apache.catalina.tribes.Channel;
 import org.apache.catalina.tribes.ManagedChannel;
 import org.apache.catalina.tribes.Member;
 import org.apache.catalina.tribes.MembershipListener;
-import junit.framework.TestCase;
 
-public class TestGroupChannelMemberArrival
-    extends TestCase {
+public class TestGroupChannelMemberArrival {
     private static int count = 10;
     private ManagedChannel[] channels = new ManagedChannel[count];
     private TestMbrListener[] listeners = new TestMbrListener[count];
 
-    protected void setUp() throws Exception {
-        super.setUp();
+    @Before
+    public void setUp() throws Exception {
         for (int i = 0; i < channels.length; i++) {
             channels[i] = new GroupChannel();
             channels[i].getMembershipService().setPayload( ("Channel-" + (i + 
1)).getBytes("ASCII"));
             listeners[i] = new TestMbrListener( ("Listener-" + (i + 1)));
             channels[i].addMembershipListener(listeners[i]);
-
         }
     }
 
@@ -47,6 +50,7 @@ public class TestGroupChannelMemberArriv
         }
     }
 
+    @Test
     public void testMemberArrival() throws Exception {
         //purpose of this test is to make sure that we have received all the 
members
         //that we can expect before the start method returns
@@ -54,6 +58,7 @@ public class TestGroupChannelMemberArriv
         for (int i=0; i<channels.length; i++ ) {
             final Channel channel = channels[i];
             Thread t = new Thread() {
+                @Override
                 public void run() {
                     try {
                         channel.start(Channel.DEFAULT);
@@ -71,24 +76,27 @@ public class TestGroupChannelMemberArriv
         for (int i=listeners.length-1; i>=0; i-- ) assertEquals("Checking 
member arrival length",channels.length-1,listeners[i].members.size());
     }
 
-    protected void tearDown() throws Exception {
+    @After
+    public void tearDown() throws Exception {
 
         for (int i = 0; i < channels.length; i++) {
             try {
                 channels[i].stop(Channel.DEFAULT);
-            } catch (Exception ignore) {}
+            } catch (Exception ignore) {
+                // Ignore
+            }
         }
-        super.tearDown();
     }
 
-    public class TestMbrListener
+    public static class TestMbrListener
         implements MembershipListener {
         public String name = null;
         public TestMbrListener(String name) {
             this.name = name;
         }
 
-        public ArrayList members = new ArrayList();
+        public ArrayList<Member> members = new ArrayList<Member>();
+
         public void memberAdded(Member member) {
             if (!members.contains(member)) {
                 members.add(member);

Modified: 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelOptionFlag.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelOptionFlag.java?rev=1713153&r1=1713152&r2=1713153&view=diff
==============================================================================
--- 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelOptionFlag.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelOptionFlag.java
 Sat Nov  7 16:47:25 2015
@@ -16,34 +16,41 @@
  */
 package org.apache.catalina.tribes.group;
 
-import junit.framework.*;
-import org.apache.catalina.tribes.ChannelInterceptor;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
 import org.apache.catalina.tribes.ChannelException;
+import org.apache.catalina.tribes.ChannelInterceptor;
 
 /**
- * <p>Title: </p> 
- * 
- * <p>Description: </p> 
- * 
+ * <p>Title: </p>
+ *
+ * <p>Description: </p>
+ *
  * <p>Company: </p>
- * 
+ *
  * @author not attributable
  * @version 1.0
  */
-public class TestGroupChannelOptionFlag extends TestCase {
-    GroupChannel channel = null;
-    protected void setUp() throws Exception {
-        super.setUp();
+public class TestGroupChannelOptionFlag {
+    private GroupChannel channel = null;
+
+    @Before
+    public void setUp() throws Exception {
         channel = new GroupChannel();
     }
 
-    protected void tearDown() throws Exception {
-        super.tearDown();
+    @After
+    public void tearDown() throws Exception {
         if ( channel != null ) try {channel.stop(channel.DEFAULT);}catch ( 
Exception ignore) {}
         channel = null;
     }
-    
-    
+
+    @Test
     public void testOptionConflict() throws Exception {
         boolean error = false;
         channel.setOptionCheck(true);
@@ -58,9 +65,10 @@ public class TestGroupChannelOptionFlag
         }catch ( ChannelException x ) {
             if ( x.getMessage().indexOf("option flag conflict") >= 0 ) error = 
true;
         }
-        assertEquals(true,error);
+        assertTrue(error);
     }
 
+    @Test
     public void testOptionNoConflict() throws Exception {
         boolean error = false;
         channel.setOptionCheck(true);
@@ -78,11 +86,11 @@ public class TestGroupChannelOptionFlag
         }catch ( ChannelException x ) {
             if ( x.getMessage().indexOf("option flag conflict") >= 0 ) error = 
true;
         }
-        assertEquals(false,error);
+        assertFalse(error);
     }
-    
+
     public static class TestInterceptor extends ChannelInterceptorBase {
-        
+        // Just use base class
     }
 
 

Modified: 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java?rev=1713153&r1=1713152&r2=1713153&view=diff
==============================================================================
--- 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
 Sat Nov  7 16:47:25 2015
@@ -16,25 +16,28 @@
  */
 package org.apache.catalina.tribes.group;
 
+import java.io.Serializable;
 import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.Random;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
 
 import org.apache.catalina.tribes.Channel;
+import org.apache.catalina.tribes.ChannelListener;
 import org.apache.catalina.tribes.ManagedChannel;
 import org.apache.catalina.tribes.Member;
-import junit.framework.TestCase;
-import org.apache.catalina.tribes.ChannelListener;
-import java.io.Serializable;
-import java.util.Random;
-import java.util.HashMap;
 import org.apache.catalina.tribes.transport.ReplicationTransmitter;
 
-public class TestGroupChannelSenderConnections extends TestCase {
+public class TestGroupChannelSenderConnections {
     private static int count = 2;
     private ManagedChannel[] channels = new ManagedChannel[count];
     private TestMsgListener[] listeners = new TestMsgListener[count];
 
-    protected void setUp() throws Exception {
-        super.setUp();
+    @Before
+    public void setUp() throws Exception {
         for (int i = 0; i < channels.length; i++) {
             channels[i] = new GroupChannel();
             channels[i].getMembershipService().setPayload( ("Channel-" + (i + 
1)).getBytes("ASCII"));
@@ -62,10 +65,12 @@ public class TestGroupChannelSenderConne
 
     }
 
+    @Test
     public void testConnectionLinger() throws Exception {
         sendMessages(0,15000);
     }
     
+    @Test
     public void testKeepAliveCount() throws Exception {
         System.out.println("Setting keep alive count to 0");
         for (int i = 0; i < channels.length; i++) {
@@ -75,6 +80,7 @@ public class TestGroupChannelSenderConne
         sendMessages(1000,15000);
     }
 
+    @Test
     public void testKeepAliveTime() throws Exception {
         System.out.println("Setting keep alive count to 1 second");
         for (int i = 0; i < channels.length; i++) {
@@ -84,22 +90,26 @@ public class TestGroupChannelSenderConne
         sendMessages(2000,15000);
     }
 
-    protected void tearDown() throws Exception {
+    @After
+    public void tearDown() throws Exception {
         for (int i = 0; i < channels.length; i++) {
             channels[i].stop(Channel.DEFAULT);
         }
 
     }
-    
+
+    // Test message. The message size is random.
     public static class TestMsg implements Serializable {
-        static Random r = new Random();
-        HashMap map = new HashMap();
+        private static final long serialVersionUID = 1L;
+        private static Random r = new Random();
+        private HashMap<Integer, ArrayList<Object>> map =
+                new HashMap<Integer, ArrayList<Object>>();
         public TestMsg() {
             int size = Math.abs(r.nextInt() % 200);
             for (int i=0; i<size; i++ ) {
                 int length = Math.abs(r.nextInt() %65000);
-                ArrayList list = new ArrayList(length);
-                map.put(new Integer(i),list);
+                ArrayList<Object> list = new ArrayList<Object>(length);
+                map.put(Integer.valueOf(i),list);
             }
         }
     }

Modified: 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelStartStop.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelStartStop.java?rev=1713153&r1=1713152&r2=1713153&view=diff
==============================================================================
--- 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelStartStop.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelStartStop.java
 Sat Nov  7 16:47:25 2015
@@ -5,35 +5,43 @@
  * The ASF licenses this file to You under the Apache License, Version 2.0
  * (the "License"); you may not use this file except in compliance with
  * the License.  You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
+ * limitations under the License.
  */
 package org.apache.catalina.tribes.group;
 
-import junit.framework.TestCase;
+import static org.junit.Assert.assertEquals;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
 import org.apache.catalina.tribes.transport.ReceiverBase;
 
 /**
  * @author Filip Hanik
  * @version 1.0
  */
-public class TestGroupChannelStartStop extends TestCase {
-    GroupChannel channel = null;
-    protected void setUp() throws Exception {
-        super.setUp();
+public class TestGroupChannelStartStop {
+    private GroupChannel channel = null;
+
+    @Before
+    public void setUp() throws Exception {
         channel = new GroupChannel();
     }
 
-    protected void tearDown() throws Exception {
-        super.tearDown();
-        try {channel.stop(channel.DEFAULT);}catch (Exception ignore){}
+    @After
+    public void tearDown() throws Exception {
+        try {channel.stop(channel.DEFAULT);}catch (Exception ignore){ /* 
Ignore */ }
     }
-    
+
+    @Test
     public void testDoubleFullStart() throws Exception {
         int count = 0;
         try {
@@ -48,14 +56,15 @@ public class TestGroupChannelStartStop e
         channel.stop(channel.DEFAULT);
     }
 
+    @Test
     public void testScrap() throws Exception {
         System.out.println(channel.getChannelReceiver().getClass());
         ((ReceiverBase)channel.getChannelReceiver()).setMaxThreads(1);
-    } 
-
+    }
 
+    @Test
     public void testDoublePartialStart() throws Exception {
-        //try to double start the RX 
+        //try to double start the RX
         int count = 0;
         try {
             channel.start(channel.SND_RX_SEQ);
@@ -106,7 +115,8 @@ public class TestGroupChannelStartStop e
         assertEquals(count,1);
         channel.stop(channel.DEFAULT);
     }
-    
+
+    @Test
     public void testFalseOption() throws Exception {
         int flag = 0xFFF0;//should get ignored by the underlying components
         int count = 0;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to