[jboss-remoting-commits] JBoss Remoting SVN: r5490 - remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket.

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

Author: ron.sigal at jboss.com
Date: 2009-09-15 22:58:17 -0400 (Tue, 15 Sep 2009)
New Revision: 5490

Modified:
   remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/BisocketTestCase.java
Log:
JBREM-1147: Disabled testControlConnectionFailureBeforeFirstCallback() since improvements for JBREM-1147 eliminate the expected failure.

Modified: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/BisocketTestCase.java
===================================================================
--- remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/BisocketTestCase.java	2009-09-16 02:57:28 UTC (rev 5489)
+++ remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/BisocketTestCase.java	2009-09-16 02:58:17 UTC (rev 5490)
@@ -1614,7 +1614,14 @@
    }
    
    
-   public void testControlConnectionFailureBeforeFirstCallback() throws Throwable
+   /**
+    * Disabling this test because recent changes to BisocketClientInvoker 
+    * (JBREM-1147: "BisocketClientInvoker.createSocket() in callback mode should check for replaced control socket")
+    * allow better recovery, so that the failure expected in this test doesn't occur.
+    * 
+    * @throws Throwable
+    */
+   public void DISABLEDtestControlConnectionFailureBeforeFirstCallback() throws Throwable
    {
       log.info("entering " + getName());
       String host = InetAddress.getLocalHost().getHostName();