[jboss-remoting-commits] JBoss Remoting SVN: r5452 - in remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport: bisocket/socketexception and 1 other directories.

jboss-remoting-commits at lists.jboss.org jboss-remoting-commits at lists.jboss.org
Thu Sep 10 12:01:04 EDT 2009


Author: ron.sigal at jboss.com
Date: 2009-09-10 12:01:03 -0400 (Thu, 10 Sep 2009)
New Revision: 5452

Added:
   remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/socketexception/
   remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/socketexception/BisocketSocketCreationExceptionTestCase.java
   remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/socket/socketexception/SocketCreationExceptionTestCase.java
Log:
JBREM-1152: New unit tests.

Added: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/socketexception/BisocketSocketCreationExceptionTestCase.java
===================================================================
--- remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/socketexception/BisocketSocketCreationExceptionTestCase.java	                        (rev 0)
+++ remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/bisocket/socketexception/BisocketSocketCreationExceptionTestCase.java	2009-09-10 16:01:03 UTC (rev 5452)
@@ -0,0 +1,231 @@
+/*
+* JBoss, Home of Professional Open Source
+* Copyright 2009, JBoss Inc., and individual contributors as indicated
+* by the @authors tag. See the copyright.txt in the distribution for a
+* full listing of individual contributors.
+*
+* This is free software; you can redistribute it and/or modify it
+* under the terms of the GNU Lesser General Public License as
+* published by the Free Software Foundation; either version 2.1 of
+* the License, or (at your option) any later version.
+*
+* This software is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+* Lesser General Public License for more details.
+*
+* You should have received a copy of the GNU Lesser General Public
+* License along with this software; if not, write to the Free
+* Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+* 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+*/
+package org.jboss.test.remoting.transport.bisocket.socketexception;
+
+import java.io.IOException;
+import java.lang.reflect.Field;
+import java.net.InetAddress;
+import java.net.ServerSocket;
+import java.net.Socket;
+import java.net.SocketException;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.Set;
+
+import javax.net.ServerSocketFactory;
+
+import org.apache.log4j.Logger;
+import org.jboss.remoting.Client;
+import org.jboss.remoting.InvokerLocator;
+import org.jboss.remoting.callback.Callback;
+import org.jboss.remoting.callback.HandleCallbackException;
+import org.jboss.remoting.callback.InvokerCallbackHandler;
+import org.jboss.remoting.transport.Connector;
+import org.jboss.remoting.transport.socket.LRUPool;
+import org.jboss.remoting.transport.socket.ServerThread;
+import org.jboss.remoting.transport.socket.SocketServerInvoker;
+import org.jboss.test.remoting.transport.socket.socketexception.SocketCreationExceptionTestCase;
+
+/**
+ * Unit tests for JBREM-1152.
+ * 
+ * @author <a href="ron.sigal at jboss.com">Ron Sigal</a>
+ * @version $Rev$
+ * <p>
+ * Copyright Sep 9, 2009
+ * </p>
+ */
+public class BisocketSocketCreationExceptionTestCase extends SocketCreationExceptionTestCase
+{
+   private static Logger log = Logger.getLogger(BisocketSocketCreationExceptionTestCase.class);
+   
+   
+   public void testCallbackException() throws Throwable
+   {
+      log.info("entering " + getName());
+      
+      // Start server.
+      setupServer(new TestServerSocketFactory(2, new SocketException(getName())));
+      
+      // Create client.
+      InvokerLocator clientLocator = new InvokerLocator(locatorURI);
+      HashMap clientConfig = new HashMap();
+      clientConfig.put(InvokerLocator.FORCE_REMOTE, "true");
+      addExtraClientConfig(clientConfig);
+      Client client = new Client(clientLocator, clientConfig);
+      client.connect();
+      log.info("client is connected");
+      
+      // Test connection.
+      assertEquals("abc", client.invoke("abc"));
+      log.info("connection is good");
+      
+      TestCallbackHandler callbackHandler = new TestCallbackHandler();
+      client.addListener(callbackHandler, new HashMap());
+      
+      // Get client side ServerThread pool.
+      Set callbackConnectors = client.getCallbackConnectors(callbackHandler);
+      Connector callbackConnector = (Connector) callbackConnectors.iterator().next();
+      SocketServerInvoker serverInvoker = (SocketServerInvoker) callbackConnector.getServerInvoker();
+      Field field = SocketServerInvoker.class.getDeclaredField("clientpool");
+      field.setAccessible(true);
+      LRUPool clientpool = (LRUPool) field.get(serverInvoker);
+      
+      // Verify MicroSocketClientInvoker retries invocation after failure to get a connection.
+      client.invoke(SEND_CALLBACK);
+      assertEquals(1, callbackHandler.received);
+      Set serverThreads = clientpool.getContents();   
+      Iterator it = serverThreads.iterator();
+      while (it.hasNext())
+      {
+         ServerThread st = (ServerThread) it.next();
+         st.shutdown();
+      }
+      client.invoke(SEND_CALLBACK);
+      assertEquals(2, callbackHandler.received);
+      serverThreads = clientpool.getContents();   
+      it = serverThreads.iterator();
+      while (it.hasNext())
+      {
+         ServerThread st = (ServerThread) it.next();
+         st.shutdown();
+      }
+      client.invoke(SEND_CALLBACK);
+      assertEquals(3, callbackHandler.received);
+      
+      client.removeListener(callbackHandler);
+      client.disconnect();
+      shutdownServer();
+      log.info(getName() + " PASSES");
+   }
+   
+   
+   protected String getTransport()
+   {
+      return "bisocket";
+   }
+   
+   
+   static class TestCallbackHandler implements InvokerCallbackHandler
+   {
+      public int received;
+      
+      public void handleCallback(Callback callback) throws HandleCallbackException
+      {
+         log.info("received callback");
+         received++;
+      }  
+   }
+   
+   static public class TestServerSocketFactory extends ServerSocketFactory
+   {
+      int initialSuccesses;
+      IOException exception;
+      
+      public TestServerSocketFactory()
+      {
+         this.initialSuccesses = -1;
+         this.exception = new IOException();
+      }      
+      public TestServerSocketFactory(int initialSuccesses, IOException exception)
+      {
+         this.initialSuccesses = initialSuccesses;
+         this.exception = exception;
+      }
+      public ServerSocket createServerSocket() throws IOException
+      {
+         ServerSocket ss = new TestServerSocket(initialSuccesses, exception);
+         log.info(this + " returning: " + ss);
+         return ss;
+      }
+      public ServerSocket createServerSocket(int port) throws IOException
+      {
+         ServerSocket ss = new TestServerSocket(port, initialSuccesses, exception);
+         log.info(this + " returning: " + ss);
+         return ss;
+      }
+
+      public ServerSocket createServerSocket(int port, int backlog) throws IOException
+      {
+         ServerSocket ss = new TestServerSocket(port, backlog, initialSuccesses, exception);
+         log.info(this + " returning: " + ss);
+         return ss;
+      }
+
+      public ServerSocket createServerSocket(int port, int backlog, InetAddress ifAddress) throws IOException
+      {
+         ServerSocket ss = new TestServerSocket(port, backlog, ifAddress, initialSuccesses, exception);
+         log.info(this + " returning: " + ss);
+         return ss;
+      }
+   }
+   
+   
+   static class TestServerSocket extends ServerSocket
+   {
+      int initialSuccesses;
+      IOException exception;
+      int counter;
+
+      public TestServerSocket(int initialSuccesses, IOException exception) throws IOException
+      {
+         super();
+         this.initialSuccesses = initialSuccesses;
+         this.exception = exception;
+      }
+      public TestServerSocket(int port, int initialSuccesses, IOException exception) throws IOException
+      {
+         super(port);
+         this.initialSuccesses = initialSuccesses;
+         this.exception = exception;
+      }
+      public TestServerSocket(int port, int backlog, int initialSuccesses, IOException exception) throws IOException
+      {
+         super(port, backlog);
+         this.initialSuccesses = initialSuccesses;
+         this.exception = exception;
+      }
+      public TestServerSocket(int port, int backlog, InetAddress bindAddr, int initialSuccesses, IOException exception) throws IOException
+      {
+         super(port, backlog, bindAddr);
+         this.initialSuccesses = initialSuccesses;
+         this.exception = exception;
+      }
+      public Socket accept() throws IOException
+      {
+         ++counter;
+         Socket s = super.accept();
+         log.info(this + " counter: " + counter);
+         if (counter > initialSuccesses && counter <= initialSuccesses + 2)
+         {
+            throw exception;
+         }
+         log.info(this + " returning: " + s);
+         return s;
+      }
+      public String toString()
+      {
+         return "TestServerSocket[" + getLocalPort() + "]";
+      }
+   }
+   
+}
\ No newline at end of file

Added: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/socket/socketexception/SocketCreationExceptionTestCase.java
===================================================================
--- remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/socket/socketexception/SocketCreationExceptionTestCase.java	                        (rev 0)
+++ remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/socket/socketexception/SocketCreationExceptionTestCase.java	2009-09-10 16:01:03 UTC (rev 5452)
@@ -0,0 +1,300 @@
+/*
+* JBoss, Home of Professional Open Source
+* Copyright 2009, JBoss Inc., and individual contributors as indicated
+* by the @authors tag. See the copyright.txt in the distribution for a
+* full listing of individual contributors.
+*
+* This is free software; you can redistribute it and/or modify it
+* under the terms of the GNU Lesser General Public License as
+* published by the Free Software Foundation; either version 2.1 of
+* the License, or (at your option) any later version.
+*
+* This software is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+* Lesser General Public License for more details.
+*
+* You should have received a copy of the GNU Lesser General Public
+* License along with this software; if not, write to the Free
+* Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+* 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+*/
+package org.jboss.test.remoting.transport.socket.socketexception;
+
+import java.io.IOException;
+import java.net.InetAddress;
+import java.net.Socket;
+import java.net.SocketException;
+import java.net.UnknownHostException;
+import java.util.HashMap;
+import java.util.Map;
+
+import javax.management.MBeanServer;
+import javax.net.ServerSocketFactory;
+import javax.net.SocketFactory;
+
+import junit.framework.TestCase;
+
+import org.apache.log4j.ConsoleAppender;
+import org.apache.log4j.Level;
+import org.apache.log4j.Logger;
+import org.apache.log4j.PatternLayout;
+import org.jboss.logging.XLevel;
+import org.jboss.remoting.Client;
+import org.jboss.remoting.InvocationRequest;
+import org.jboss.remoting.InvokerLocator;
+import org.jboss.remoting.Remoting;
+import org.jboss.remoting.ServerInvocationHandler;
+import org.jboss.remoting.ServerInvoker;
+import org.jboss.remoting.callback.Callback;
+import org.jboss.remoting.callback.HandleCallbackException;
+import org.jboss.remoting.callback.InvokerCallbackHandler;
+import org.jboss.remoting.transport.Connector;
+import org.jboss.remoting.transport.PortUtil;
+
+/**
+ * Unit tests for JBREM-1152.
+ * 
+ * @author <a href="ron.sigal at jboss.com">Ron Sigal</a>
+ * @version $Rev$
+ * <p>
+ * Copyright Sep 9, 2009
+ * </p>
+ */
+public class SocketCreationExceptionTestCase extends TestCase
+{
+   protected static String SEND_CALLBACK = "sendCallback";
+   private static Logger log = Logger.getLogger(SocketCreationExceptionTestCase.class);
+   private static boolean firstTime = true;
+   
+   protected String host;
+   protected int port;
+   protected String locatorURI;
+   protected InvokerLocator serverLocator;
+   protected Connector connector;
+   protected TestInvocationHandler invocationHandler;
+
+   
+   public void setUp() throws Exception
+   {
+      if (firstTime)
+      {
+         firstTime = false;
+         Logger.getLogger("org.jboss.remoting").setLevel(XLevel.TRACE);
+         Logger.getLogger("org.jboss.test.remoting").setLevel(Level.INFO);
+         String pattern = "[%d{ABSOLUTE}] [%t] %5p (%F:%L) - %m%n";
+         PatternLayout layout = new PatternLayout(pattern);
+         ConsoleAppender consoleAppender = new ConsoleAppender(layout);
+         Logger.getRootLogger().addAppender(consoleAppender);  
+      }
+   }
+
+   
+   public void tearDown()
+   {
+   }
+   
+   
+   public void testInvocationWithSocketException() throws Throwable
+   {
+      log.info("entering " + getName());
+      
+      // Start server.
+      setupServer(null);
+      
+      // Create client.
+      InvokerLocator clientLocator = new InvokerLocator(locatorURI);
+      HashMap clientConfig = new HashMap();
+      clientConfig.put(InvokerLocator.FORCE_REMOTE, "true");
+      clientConfig.put(Remoting.CUSTOM_SOCKET_FACTORY, new TestSocketFactory(new SocketException(getName()), 2));
+      addExtraClientConfig(clientConfig);
+      Client client = new Client(clientLocator, clientConfig);
+      client.connect();
+      log.info("client is connected");
+      
+      // Test connection.
+      assertEquals("abc", client.invoke("abc"));
+      log.info("connection is good");
+      
+      client.disconnect();
+      shutdownServer();
+      log.info(getName() + " PASSES");
+   }
+   
+   
+   public void testInvocationWithIOException() throws Throwable
+   {
+      log.info("entering " + getName());
+      
+      // Start server.
+      setupServer(null);
+      
+      // Create client.
+      InvokerLocator clientLocator = new InvokerLocator(locatorURI);
+      HashMap clientConfig = new HashMap();
+      clientConfig.put(InvokerLocator.FORCE_REMOTE, "true");
+      clientConfig.put(Remoting.CUSTOM_SOCKET_FACTORY, new TestSocketFactory(new IOException("Connection reset"), 2));
+      clientConfig.put("generalizeSocketException", "true");
+      addExtraClientConfig(clientConfig);
+      Client client = new Client(clientLocator, clientConfig);
+      client.connect();
+      log.info("client is connected");
+      
+      // Test connection.
+      assertEquals("abc", client.invoke("abc"));
+      log.info("connection is good");
+      
+      client.disconnect();
+      shutdownServer();
+      log.info(getName() + " PASSES");
+   }
+   
+   
+   protected String getTransport()
+   {
+      return "socket";
+   }
+   
+   
+   protected void addExtraClientConfig(Map config) {}
+   protected void addExtraServerConfig(Map config) {}
+   
+
+   protected void setupServer(ServerSocketFactory ssf) throws Exception
+   {
+      host = InetAddress.getLocalHost().getHostAddress();
+      port = PortUtil.findFreePort(host);
+      locatorURI = getTransport() + "://" + host + ":" + port;
+      String metadata = System.getProperty("remoting.metadata");
+      if (metadata != null)
+      {
+         locatorURI += "&" + metadata;
+      }
+      serverLocator = new InvokerLocator(locatorURI);
+      log.info("Starting remoting server with locator uri of: " + locatorURI);
+      HashMap config = new HashMap();
+      config.put(InvokerLocator.FORCE_REMOTE, "true");
+      config.put("timeout", "5000");
+      if (ssf != null)
+      {
+         config.put(Remoting.CUSTOM_SERVER_SOCKET_FACTORY, ssf);
+      }
+      addExtraServerConfig(config);
+      connector = new Connector(serverLocator, config);
+      connector.create();
+      invocationHandler = new TestInvocationHandler();
+      connector.addInvocationHandler("test", invocationHandler);
+      connector.start();
+   }
+   
+   
+   protected void shutdownServer() throws Exception
+   {
+      if (connector != null)
+         connector.stop();
+   }
+   
+   
+   static class TestInvocationHandler implements ServerInvocationHandler
+   {
+      InvokerCallbackHandler callbackHandler;
+      
+      public void addListener(InvokerCallbackHandler callbackHandler)
+      {
+         this.callbackHandler = callbackHandler;
+      }
+      public Object invoke(final InvocationRequest invocation) throws Throwable
+      {
+         Object parameter = invocation.getParameter();
+         if (SEND_CALLBACK.equals(parameter))
+         {
+            try
+            {
+               callbackHandler.handleCallback(new Callback("callback"));
+            }
+            catch (HandleCallbackException e)
+            {
+               log.error("unable to send callback", e);
+            }
+         }
+         return parameter;
+      }
+      public void removeListener(InvokerCallbackHandler callbackHandler) {}
+      public void setMBeanServer(MBeanServer server) {}
+      public void setInvoker(ServerInvoker invoker) {}
+   }
+   
+   
+   public static class TestSocketFactory extends SocketFactory
+   {
+      int counter;
+      int limit; 
+      IOException exception;
+      
+      public TestSocketFactory()
+      {
+      }
+      public TestSocketFactory(IOException exception, int limit)
+      {
+         this.exception = exception;
+         this.limit = limit;
+      }
+      public Socket createSocket() throws IOException
+      {
+         counter++;
+         log.info("counter: " + counter);
+         if (counter <= limit)
+         {
+            log.info("throwing exception");
+            throw exception;
+         }
+         log.info("returning socket");
+         return new Socket();
+      }
+      public Socket createSocket(String arg0, int arg1) throws IOException, UnknownHostException
+      {
+         counter++;
+         log.info("counter: " + counter);
+         if (counter <= limit)
+         {
+            throw exception;
+         }
+         log.info("returning socket");
+         return new Socket(arg0, arg1);
+      }
+      public Socket createSocket(InetAddress arg0, int arg1) throws IOException
+      {
+         counter++;
+         log.info("counter: " + counter);
+         if (counter <= limit)
+         {
+            throw exception;
+         }
+         log.info("returning socket");
+         return new Socket(arg0, arg1);
+      }
+      public Socket createSocket(String arg0, int arg1, InetAddress arg2, int arg3) throws IOException, UnknownHostException
+      {
+         counter++;
+         log.info("counter: " + counter);
+         if (counter <= limit)
+         {
+            throw exception;
+         }
+         log.info("returning socket");
+         return new Socket(arg0, arg1, arg2, arg3);
+      }
+
+      public Socket createSocket(InetAddress arg0, int arg1, InetAddress arg2, int arg3) throws IOException
+      {
+         counter++;
+         log.info("counter: " + counter);
+         if (counter <= limit)
+         {
+            throw exception;
+         }
+         log.info("returning socket");
+         return new Socket(arg0, arg1, arg2, arg3);
+      }
+   }
+}
\ No newline at end of file



More information about the jboss-remoting-commits mailing list