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

Reply via email to