Author: markt Date: Tue Jul 26 12:28:00 2011 New Revision: 1151074 URL: http://svn.apache.org/viewvc?rev=1151074&view=rev Log: Tests are run with all three HTTP connectors. No need for separate tests.
Modified: tomcat/trunk/test/org/apache/coyote/http11/TestInternalInputBuffer.java Modified: tomcat/trunk/test/org/apache/coyote/http11/TestInternalInputBuffer.java URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/coyote/http11/TestInternalInputBuffer.java?rev=1151074&r1=1151073&r2=1151074&view=diff ============================================================================== --- tomcat/trunk/test/org/apache/coyote/http11/TestInternalInputBuffer.java (original) +++ tomcat/trunk/test/org/apache/coyote/http11/TestInternalInputBuffer.java Tue Jul 26 12:28:00 2011 @@ -31,7 +31,6 @@ import static org.junit.Assert.assertTru import org.junit.Test; import org.apache.catalina.Context; -import org.apache.catalina.connector.Connector; import org.apache.catalina.startup.SimpleHttpClient; import org.apache.catalina.startup.Tomcat; import org.apache.catalina.startup.TomcatBaseTest; @@ -40,54 +39,28 @@ public class TestInternalInputBuffer ext /** * Test case for https://issues.apache.org/bugzilla/show_bug.cgi?id=48839 - * with BIO */ @Test - public void testBug48839BIO() { + public void testBug48839() { Bug48839Client client = new Bug48839Client(); client.setPort(getPort()); - // BIO test - client.doRequest(false); + client.doRequest(); assertTrue(client.isResponse200()); assertTrue(client.isResponseBodyOK()); } /** - * Test case for https://issues.apache.org/bugzilla/show_bug.cgi?id=48839 - * with NIO - */ - @Test - public void testBug48839NIO() { - - Bug48839Client client = new Bug48839Client(); - client.setPort(getPort()); - - // NIO test - client.doRequest(true); - assertTrue(client.isResponse200()); - assertTrue(client.isResponseBodyOK()); - } - - /** * Bug 48839 test client. */ private class Bug48839Client extends SimpleHttpClient { - private Exception doRequest(boolean useNio) { + private Exception doRequest() { Tomcat tomcat = getTomcatInstance(); - if (useNio) { - Connector connector = - new Connector("org.apache.coyote.http11.Http11NioProtocol"); - connector.setPort(getPort()); - tomcat.getService().addConnector(connector); - tomcat.setConnector(connector); - } - Context root = tomcat.addContext("", TEMP_DIR); Tomcat.addServlet(root, "Bug48839", new Bug48839Servlet()); root.addServletMapping("/test", "Bug48839"); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org