Author: kkolinko
Date: Thu Nov  3 18:43:09 2011
New Revision: 1197263

URL: http://svn.apache.org/viewvc?rev=1197263&view=rev
Log:
Clean up code and use logging framework instead of System.out.

Modified:
    
tomcat/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java

Modified: 
tomcat/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java?rev=1197263&r1=1197262&r2=1197263&view=diff
==============================================================================
--- 
tomcat/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
 (original)
+++ 
tomcat/trunk/test/org/apache/catalina/tribes/group/TestGroupChannelSenderConnections.java
 Thu Nov  3 18:43:09 2011
@@ -25,45 +25,49 @@ import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
+import org.apache.catalina.startup.LoggingBaseTest;
 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 org.apache.catalina.tribes.transport.ReplicationTransmitter;
 
-public class TestGroupChannelSenderConnections {
-    private static int count = 2;
+public class TestGroupChannelSenderConnections extends LoggingBaseTest {
+    private static final int count = 2;
     private ManagedChannel[] channels = new ManagedChannel[count];
     private TestMsgListener[] listeners = new TestMsgListener[count];
 
     @Before
+    @Override
     public void setUp() throws Exception {
+        super.setUp();
         for (int i = 0; i < channels.length; i++) {
             channels[i] = new GroupChannel();
             channels[i].getMembershipService().setPayload( ("Channel-" + (i + 
1)).getBytes("ASCII"));
             listeners[i] = new TestMsgListener( ("Listener-" + (i + 1)));
             channels[i].addChannelListener(listeners[i]);
             channels[i].start(Channel.SND_RX_SEQ|Channel.SND_TX_SEQ);
-
         }
     }
 
-    public void clear() {
-        // NOOP
-    }
-
     public void sendMessages(long delay, long sleep) throws Exception {
         Member local = channels[0].getLocalMember(true);
         Member dest = channels[1].getLocalMember(true);
         int n = 3;
-        System.out.println("Sending " + n + " messages from [" + 
local.getName() + "] to [" + dest.getName() + "]");
+        log.info("Sending " + n + " messages from [" + local.getName()
+                + "] to [" + dest.getName() + "] with delay of " + delay
+                + " ms between them.");
         for (int i = 0; i < n; i++) {
-            channels[0].send(new Member[] {dest}, new TestMsg(), 0);
-            if ( delay > 0 ) Thread.sleep(delay);
+            channels[0].send(new Member[] { dest }, new TestMsg(), 0);
+            if (delay > 0) {
+                Thread.sleep(delay);
+            }
+        }
+        log.info("Messages sent. Sleeping for " + (sleep / 1000)
+                + " seconds to inspect connections");
+        if (sleep > 0) {
+            Thread.sleep(sleep);
         }
-        System.out.println("Messages sent. Sleeping for "+(sleep/1000)+" 
seconds to inspect connections");
-        if ( sleep > 0 ) Thread.sleep(sleep);
-
     }
 
     @Test
@@ -73,7 +77,7 @@ public class TestGroupChannelSenderConne
 
     @Test
     public void testKeepAliveCount() throws Exception {
-        System.out.println("Setting keep alive count to 0");
+        log.info("Setting keep alive count to 0");
         for (int i = 0; i < channels.length; i++) {
             ReplicationTransmitter t = 
(ReplicationTransmitter)channels[0].getChannelSender();
             t.getTransport().setKeepAliveCount(0);
@@ -83,7 +87,7 @@ public class TestGroupChannelSenderConne
 
     @Test
     public void testKeepAliveTime() throws Exception {
-        System.out.println("Setting keep alive count to 1 second");
+        log.info("Setting keep alive count to 1 second");
         for (int i = 0; i < channels.length; i++) {
             ReplicationTransmitter t = 
(ReplicationTransmitter)channels[0].getChannelSender();
             t.getTransport().setKeepAliveTime(1000);
@@ -92,17 +96,22 @@ public class TestGroupChannelSenderConne
     }
 
     @After
+    @Override
     public void tearDown() throws Exception {
-        for (int i = 0; i < channels.length; i++) {
-            channels[i].stop(Channel.DEFAULT);
+        try {
+            for (int i = 0; i < channels.length; i++) {
+                channels[i].stop(Channel.DEFAULT);
+            }
+        } finally {
+            super.tearDown();
         }
-
     }
 
+    // Test message. The message size is random.
     public static class TestMsg implements Serializable {
         private static final long serialVersionUID = 1L;
-        static Random r = new Random();
-        HashMap<Integer, ArrayList<Object>> map =
+        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);
@@ -114,24 +123,22 @@ public class TestGroupChannelSenderConne
         }
     }
 
-    public static class TestMsgListener implements ChannelListener {
-        public String name = null;
+    public class TestMsgListener implements ChannelListener {
+        public final String name;
         public TestMsgListener(String name) {
             this.name = name;
         }
 
         @Override
         public void messageReceived(Serializable msg, Member sender) {
-            System.out.println("["+name+"] Received message:"+msg+" from " + 
sender.getName());
+            log.info("["+name+"] Received message:"+msg+" from " + 
sender.getName());
         }
 
-
         @Override
         public boolean accept(Serializable msg, Member sender) {
             return true;
         }
 
-
     }
 
 }



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

Reply via email to