[jboss-remoting-commits] JBoss Remoting SVN: r4067 - remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/oneway.

[jboss-remoting-commits at lists.jboss.org]

Author: ron.sigal at jboss.com
Date: 2008-04-24 21:30:45 -0400 (Thu, 24 Apr 2008)
New Revision: 4067

Modified:
   remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/oneway/OnewayConnectionManagerTestCase.java
Log:
JBREM-930: In testHeavyLoadWithoutTimeouts() deleted test for size of connection pool.

Modified: remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/oneway/OnewayConnectionManagerTestCase.java
===================================================================
--- remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/oneway/OnewayConnectionManagerTestCase.java	2008-04-25 01:00:39 UTC (rev 4066)
+++ remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/oneway/OnewayConnectionManagerTestCase.java	2008-04-25 01:30:45 UTC (rev 4067)
@@ -546,8 +546,12 @@
       
       Thread.sleep(2000);
       assertEquals(0, invoker.getNumberOfUsedConnections());
-      assertEquals(100, pool.size());
       
+      // Commenting out the following test.  If the server is busy, it is possible
+      // for some oneway connections to time out, in which case they would not
+      // be returned to the connection pool.
+      // assertEquals(100, pool.size());
+      
       client.disconnect();
       connector.stop();
       log.info(getName() + " PASSES");