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

[Ron Sigal]

  User: rsigal  
  Date: 07/11/02 21:03:41

  Modified:    src/tests/org/jboss/test/remoting/timeout  Tag: remoting_2_x
                        QuickDisconnectClientParent.java
  Log:
  JBREM-798:  In testQuickRemovePullListener() and testQuickRemovePullListenerZeroTimeout(), return without testing for bidirectional transports. 
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.6   +23 -1     JBossRemoting/src/tests/org/jboss/test/remoting/timeout/Attic/QuickDisconnectClientParent.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: QuickDisconnectClientParent.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/tests/org/jboss/test/remoting/timeout/Attic/QuickDisconnectClientParent.java,v
  retrieving revision 1.1.2.5
  retrieving revision 1.1.2.6
  diff -u -b -r1.1.2.5 -r1.1.2.6
  --- QuickDisconnectClientParent.java	31 Aug 2007 23:59:12 -0000	1.1.2.5
  +++ QuickDisconnectClientParent.java	3 Nov 2007 01:03:41 -0000	1.1.2.6
  @@ -40,6 +40,8 @@
   import org.jboss.remoting.callback.Callback;
   import org.jboss.remoting.callback.HandleCallbackException;
   import org.jboss.remoting.callback.InvokerCallbackHandler;
  +import org.jboss.remoting.transport.BidirectionalClientInvoker;
  +import org.jboss.remoting.transport.ClientInvoker;
   
   /** 
    * QuickDisconnectTestParent verifies that LeasePinger can set its
  @@ -50,7 +52,7 @@
    * removeListener().
    * 
    * @author <a href="ron.sigal at jboss.com">Ron Sigal</a>
  - * @version $Revision: 1.1.2.5 $
  + * @version $Revision: 1.1.2.6 $
    * <p>
    * Copyright Jan 24, 2007
    * </p>
  @@ -461,6 +463,16 @@
         {
            e.printStackTrace();
         }
  +      
  +      // Client won't set up pull callbacks for bidirectional transport.
  +      ClientInvoker invoker = client.getInvoker();
  +      if (invoker instanceof BidirectionalClientInvoker)
  +      {
  +         log.info(getTransport() + " is bidirectional");
  +         log.info(getName() + " PASSES");
  +         return;
  +      }
  +      
         log.info("making first invocation");
         Object response = client.invoke("test");
         assertEquals("test", response);
  @@ -648,6 +660,16 @@
         {
            e.printStackTrace();
         }
  +      
  +      // Client won't set up pull callbacks for bidirectional transport.
  +      ClientInvoker invoker = client.getInvoker();
  +      if (invoker instanceof BidirectionalClientInvoker)
  +      {
  +         log.info(getTransport() + " is bidirectional");
  +         log.info(getName() + " PASSES");
  +         return;
  +      }
  +      
         log.info("making first invocation");
         Object response = client.invoke("test");
         assertEquals("test", response);