[jboss-cvs] JBossRemoting/src/tests/org/jboss/test/remoting/transport/socket/timeout ...

Ron Sigal ron_sigal at yahoo.com
Tue Nov 13 03:48:01 EST 2007


  User: rsigal  
  Date: 07/11/13 03:48:01

  Modified:    src/tests/org/jboss/test/remoting/transport/socket/timeout 
                        Tag: remoting_2_x
                        SocketPerInvocationTimeoutTestCase.java
  Log:
  JBREM-800:  Removed second invocation in testNewSocketTimeout(), which is no longer necessary since ServerThreads close their sockets when they shut down.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.3   +2 -5      JBossRemoting/src/tests/org/jboss/test/remoting/transport/socket/timeout/SocketPerInvocationTimeoutTestCase.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: SocketPerInvocationTimeoutTestCase.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/tests/org/jboss/test/remoting/transport/socket/timeout/SocketPerInvocationTimeoutTestCase.java,v
  retrieving revision 1.1.2.2
  retrieving revision 1.1.2.3
  diff -u -b -r1.1.2.2 -r1.1.2.3
  --- SocketPerInvocationTimeoutTestCase.java	6 Feb 2007 23:22:59 -0000	1.1.2.2
  +++ SocketPerInvocationTimeoutTestCase.java	13 Nov 2007 08:48:01 -0000	1.1.2.3
  @@ -30,7 +30,7 @@
    * See javadoc for PerInvocationTimeoutTestRoot.
    *   
    * @author <a href="ron.sigal at jboss.com">Ron Sigal</a>
  - * @version $Revision: 1.1.2.2 $
  + * @version $Revision: 1.1.2.3 $
    * <p>
    * Copyright Feb 6, 2007
    * </p>
  @@ -144,10 +144,6 @@
                  // Wait for the server to be disabled.
                  Thread.sleep(4000);
                  
  -               // This invocation will cause the ServerThread to shut down.
  -               client.invoke("test 2");
  -               log.debug("calling client.disconnect()");
  -               
                  // This invocation will require the creation of a new socket, which
                  // should promptly time out.
                  HashMap metadata = new HashMap();
  @@ -168,6 +164,7 @@
         // and then verify that the Client has timed out according to the temporary timeout
         // value 1000 instead of the configureed value 20000.
         Thread.sleep(8000);
  +      log.info("testing timeout");
         assertTrue(timedOut.value);
         
         client.disconnect();
  
  
  



More information about the jboss-cvs-commits mailing list