Author: ron.sigal(a)jboss.com Date: 2009-05-12 02:37:04 -0400 (Tue, 12 May 2009) New Revision: 5202 Modified: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/web/WebInvokerTestClient.java Log: JBREM-1079: Added test for transport to support servlet and sslservlet transports. Modified: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/web/WebInvokerTestClient.java =================================================================== --- remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/web/WebInvokerTestClient.java 2009-05-12 06:35:42 UTC (rev 5201) +++ remoting2/branches/2.2/src/tests/org/jboss/test/remoting/transport/web/WebInvokerTestClient.java 2009-05-12 06:37:04 UTC (rev 5202) @@ -99,8 +99,8 @@ String locatorURI = getLocatorURI(); int index = locatorURI.indexOf(':'); String transport = locatorURI.substring(0, index); - String params = ("servlet".equals(transport) ? ",createUniqueObjectName=true" : ""); - String port = ("servlet".equals(transport) ? "8080" : "8888"); + String params = (isServlet(transport) ? ",createUniqueObjectName=true" : ""); + String port = "servlet".equals(transport) ? "8080" : ("sslservlet".equals(transport) ? "8443" : "8888"); assertEquals("jboss.remoting:service=invoker,transport= " + transport + ",host=localhost,port=" + port + params, response); checkUserAgent(remotingClient, metadata); @@ -205,6 +205,7 @@ Object response = null; // test with null return expected + System.out.println("calling Client.invoke()"); response = remotingClient.invoke((Object) null, metadata); System.out.println("Response should be " + WebInvocationHandler.HTML_PAGE_RESPONSE + " and was: " + response); assertEquals(WebInvocationHandler.HTML_PAGE_RESPONSE, response); @@ -215,6 +216,7 @@ } catch (Throwable throwable) { + System.out.println("invoke() failed"); throw new Exception(throwable); } finally @@ -228,6 +230,11 @@ } + protected boolean isServlet(String transport) + { + return transport.indexOf("servlet") >= 0; + } + protected void addHeaders(Properties headerProps) { //NO OP - for overriding by sub-classes.