rwaldhoff    01/08/15 16:19:35

  Modified:    httpclient/src/java/org/apache/commons/httpclient/methods
                        Tag: rlwrefactoring GetMethod.java PutMethod.java
               httpclient/src/java/org/apache/commons/httpclient Tag:
                        rlwrefactoring HttpMethod.java HttpMethodBase.java
               httpclient/src/test/org/apache/commons/httpclient Tag:
                        rlwrefactoring TestGetMethodLocal.java
                        TestHttps.java TestMethodsExternalHost.java
                        TestMethodsLocalHost.java TestWebappHeaders.java
                        TestWebappMethods.java TestWebappParameters.java
                        TestWebappRedirect.java
  Log:
  Moving getResponseBody, getResponseBodyAsString and getResponseBodyAsStream up to 
HttpMethod/HttpMethodBase (from PutMethod)
  
  Removing getData, getDataAsString, etc. from GetMethod (replaced by getResponseBody)
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.2.2.6   +39 -35    
jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/GetMethod.java
  
  Index: GetMethod.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/GetMethod.java,v
  retrieving revision 1.2.2.5
  retrieving revision 1.2.2.6
  diff -u -r1.2.2.5 -r1.2.2.6
  --- GetMethod.java    2001/08/13 23:35:29     1.2.2.5
  +++ GetMethod.java    2001/08/15 23:19:34     1.2.2.6
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/GetMethod.java,v
 1.2.2.5 2001/08/13 23:35:29 rwaldhoff Exp $
  - * $Revision: 1.2.2.5 $
  - * $Date: 2001/08/13 23:35:29 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/GetMethod.java,v
 1.2.2.6 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.2.2.6 $
  + * $Date: 2001/08/15 23:19:34 $
    *
    * ====================================================================
    *
  @@ -314,11 +314,46 @@
           setFollowRedirects(true);
       }
   
  +   public byte[] getResponseBody() {
  +       // should flip this around so AsString uses bytes
  +       return getResponseBodyAsString().getBytes();
  +   }
  +
  +   public String getResponseBodyAsString() {
  +       // should flip this around so AsString uses bytes
  +       checkUsed();
  +       if (useDisk) {
  +           try {
  +               InputStream is = new FileInputStream(fileData);
  +               byte[] buffer = new byte[4096];
  +               ByteArrayOutputStream os = new ByteArrayOutputStream();
  +               int nb = 0;
  +               while (true) {
  +                   nb = is.read(buffer);
  +                   if (nb == -1)
  +                       break;
  +                   os.write(buffer, 0, nb);
  +               }
  +               is.close();
  +               return os.toString();
  +           } catch(IOException e) {
  +               log.error("Exception in GetMethod.getResponseBodyAsString() while 
retrieving data from file \"" + fileData + "\".",e);
  +               return null;
  +           }
  +       } else {
  +           if (memoryData != null) {
  +               return new String(memoryData);
  +           } else {
  +               return "";
  +           }
  +       }
  +   }
   
  +
       /**
        * Get read data.
        */
  -    public InputStream getData()
  +    public InputStream getResponseBodyAsStream()
           throws IOException {
   
           checkUsed();
  @@ -329,37 +364,6 @@
               if (memoryData == null)
                   throw new IOException("Data not found");
               return new ByteArrayInputStream(memoryData);
  -        }
  -
  -    }
  -
  -
  -    /**
  -     * Get read data as a String.
  -     */
  -    public String getDataAsString()
  -        throws IOException {
  -
  -        checkUsed();
  -
  -        if (useDisk) {
  -            InputStream is = new FileInputStream(fileData);
  -            byte[] buffer = new byte[4096];
  -            ByteArrayOutputStream os = new ByteArrayOutputStream();
  -            int nb = 0;
  -            while (true) {
  -                nb = is.read(buffer);
  -                if (nb == -1)
  -                    break;
  -                os.write(buffer, 0, nb);
  -            }
  -            is.close();
  -            return os.toString();
  -        } else {
  -            if (memoryData != null)
  -                return new String(memoryData);
  -            else
  -                return "";
           }
   
       }
  
  
  
  1.3.2.7   +3 -11     
jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/PutMethod.java
  
  Index: PutMethod.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/PutMethod.java,v
  retrieving revision 1.3.2.6
  retrieving revision 1.3.2.7
  diff -u -r1.3.2.6 -r1.3.2.7
  --- PutMethod.java    2001/08/14 23:06:17     1.3.2.6
  +++ PutMethod.java    2001/08/15 23:19:34     1.3.2.7
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/PutMethod.java,v
 1.3.2.6 2001/08/14 23:06:17 rwaldhoff Exp $
  - * $Revision: 1.3.2.6 $
  - * $Date: 2001/08/14 23:06:17 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/PutMethod.java,v
 1.3.2.7 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.3.2.7 $
  + * $Date: 2001/08/15 23:19:34 $
    *
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -256,14 +256,6 @@
           } else {
               return 0;
           }
  -    }
  -
  -    public byte[] getResponseBody() {
  -        return responseBody;
  -    }
  -
  -    public String getResponseBodyAsString() {
  -        return null == responseBody ? null : new String(responseBody);
       }
   
       public void recycle() {
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.5.2.7   +9 -4      
jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethod.java
  
  Index: HttpMethod.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethod.java,v
  retrieving revision 1.5.2.6
  retrieving revision 1.5.2.7
  diff -u -r1.5.2.6 -r1.5.2.7
  --- HttpMethod.java   2001/08/15 22:55:04     1.5.2.6
  +++ HttpMethod.java   2001/08/15 23:19:34     1.5.2.7
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethod.java,v
 1.5.2.6 2001/08/15 22:55:04 rwaldhoff Exp $
  - * $Revision: 1.5.2.6 $
  - * $Date: 2001/08/15 22:55:04 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethod.java,v
 1.5.2.7 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.5.2.7 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -13,7 +13,6 @@
   package org.apache.commons.httpclient;
   
   import java.io.InputStream;
  -import java.io.OutputStream;
   import java.io.IOException;
   import java.util.Iterator;
   import java.util.Collection;
  @@ -161,6 +160,12 @@
        * Return the specified response headers.
        */
       public Header getResponseHeader(String headerName);
  +
  +    public byte[] getResponseBody();
  +
  +    public String getResponseBodyAsString();
  +
  +    public InputStream getResponseBodyAsStream() throws IOException;
   
       /**
        * Return <tt>true</tt> if I have been {@link #execute executed}
  
  
  
  1.10.2.17 +17 -4     
jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethodBase.java
  
  Index: HttpMethodBase.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethodBase.java,v
  retrieving revision 1.10.2.16
  retrieving revision 1.10.2.17
  diff -u -r1.10.2.16 -r1.10.2.17
  --- HttpMethodBase.java       2001/08/15 22:55:04     1.10.2.16
  +++ HttpMethodBase.java       2001/08/15 23:19:34     1.10.2.17
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethodBase.java,v
 1.10.2.16 2001/08/15 22:55:04 rwaldhoff Exp $
  - * $Revision: 1.10.2.16 $
  - * $Date: 2001/08/15 22:55:04 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/HttpMethodBase.java,v
 1.10.2.17 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.10.2.17 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -15,6 +15,7 @@
   import java.io.InputStream;
   import java.io.OutputStream;
   import java.io.ByteArrayOutputStream;
  +import java.io.ByteArrayInputStream;
   import java.io.IOException;
   import java.util.Iterator;
   import java.util.HashMap;
  @@ -67,7 +68,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Remy Maucherat</a>
    * @author Rodney Waldhoff
  - * @version $Id: HttpMethodBase.java,v 1.10.2.16 2001/08/15 22:55:04 rwaldhoff Exp $
  + * @version $Id: HttpMethodBase.java,v 1.10.2.17 2001/08/15 23:19:34 rwaldhoff Exp $
    */
   public abstract class HttpMethodBase implements HttpMethod {
   
  @@ -293,6 +294,18 @@
        */
       public Header getResponseHeader(String headerName) {
           return (Header)(responseHeaders.get(headerName.toLowerCase()));
  +    }
  +
  +    public byte[] getResponseBody() {
  +        return responseBody;
  +    }
  +
  +    public String getResponseBodyAsString() {
  +        return null == responseBody ? null : new String(responseBody);
  +    }
  +
  +    public InputStream getResponseBodyAsStream() throws IOException {
  +        return null == responseBody ? null : new ByteArrayInputStream(responseBody);
       }
   
       /**
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.3   +8 -8      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestGetMethodLocal.java
  
  Index: TestGetMethodLocal.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestGetMethodLocal.java,v
  retrieving revision 1.1.2.2
  retrieving revision 1.1.2.3
  diff -u -r1.1.2.2 -r1.1.2.3
  --- TestGetMethodLocal.java   2001/08/13 15:57:43     1.1.2.2
  +++ TestGetMethodLocal.java   2001/08/15 23:19:34     1.1.2.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestGetMethodLocal.java,v
 1.1.2.2 2001/08/13 15:57:43 rwaldhoff Exp $
  - * $Revision: 1.1.2.2 $
  - * $Date: 2001/08/13 15:57:43 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestGetMethodLocal.java,v
 1.1.2.3 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.3 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -31,7 +31,7 @@
    * "httpclient.test.localHost" property.
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestGetMethodLocal.java,v 1.1.2.2 2001/08/13 15:57:43 rwaldhoff 
Exp $
  + * @version $Id: TestGetMethodLocal.java,v 1.1.2.3 2001/08/15 23:19:34 rwaldhoff 
Exp $
    */
   public class TestGetMethodLocal extends TestCase {
   
  @@ -84,7 +84,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               assert("No data returned.",(data.length() > 0));
           } catch (Throwable t) {
               t.printStackTrace();
  @@ -107,7 +107,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               assert("No data returned.",(data.length() > 0));
           } catch (Throwable t) {
               t.printStackTrace();
  @@ -130,7 +130,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               assert("No data returned.",(data.length() > 0));
           } catch (Throwable t) {
               t.printStackTrace();
  @@ -149,7 +149,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               assert("No data returned.",(data.length() > 0));
           } catch (Throwable t) {
               t.printStackTrace();
  
  
  
  1.2.2.1   +6 -6      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestHttps.java
  
  Index: TestHttps.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestHttps.java,v
  retrieving revision 1.2
  retrieving revision 1.2.2.1
  diff -u -r1.2 -r1.2.2.1
  --- TestHttps.java    2001/07/27 22:52:25     1.2
  +++ TestHttps.java    2001/08/15 23:19:34     1.2.2.1
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestHttps.java,v
 1.2 2001/07/27 22:52:25 rwaldhoff Exp $
  - * $Revision: 1.2 $
  - * $Date: 2001/07/27 22:52:25 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestHttps.java,v
 1.2.2.1 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.2.2.1 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -28,7 +28,7 @@
    *    (see build.xml)
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestHttps.java,v 1.2 2001/07/27 22:52:25 rwaldhoff Exp $
  + * @version $Id: TestHttps.java,v 1.2.2.1 2001/08/15 23:19:34 rwaldhoff Exp $
    */
   public class TestHttps extends TestCase {
   
  @@ -70,7 +70,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.", (data.length() > 0));
           } catch (Throwable t) {
  @@ -92,7 +92,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.", (data.length() > 0));
           } catch (Throwable t) {
  
  
  
  1.1.2.1   +6 -6      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsExternalHost.java
  
  Index: TestMethodsExternalHost.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsExternalHost.java,v
  retrieving revision 1.1
  retrieving revision 1.1.2.1
  diff -u -r1.1 -r1.1.2.1
  --- TestMethodsExternalHost.java      2001/07/27 22:52:25     1.1
  +++ TestMethodsExternalHost.java      2001/08/15 23:19:34     1.1.2.1
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsExternalHost.java,v
 1.1 2001/07/27 22:52:25 rwaldhoff Exp $
  - * $Revision: 1.1 $
  - * $Date: 2001/07/27 22:52:25 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsExternalHost.java,v
 1.1.2.1 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.1 $
  + * $Date: 2001/08/15 23:19:34 $
    *
    * ====================================================================
    *
  @@ -74,7 +74,7 @@
    *
    * @author Remy Maucherat
    * @author Rodney Waldhoff
  - * @version $Id: TestMethodsExternalHost.java,v 1.1 2001/07/27 22:52:25 rwaldhoff 
Exp $
  + * @version $Id: TestMethodsExternalHost.java,v 1.1.2.1 2001/08/15 23:19:34 
rwaldhoff Exp $
    */
   public class TestMethodsExternalHost extends TestCase {
   
  @@ -145,7 +145,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.",
                      (data.length() > 0));
  @@ -166,7 +166,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.",
                      (data.length() > 0));
  
  
  
  1.1.2.2   +6 -6      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsLocalHost.java
  
  Index: TestMethodsLocalHost.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsLocalHost.java,v
  retrieving revision 1.1.2.1
  retrieving revision 1.1.2.2
  diff -u -r1.1.2.1 -r1.1.2.2
  --- TestMethodsLocalHost.java 2001/08/10 23:05:51     1.1.2.1
  +++ TestMethodsLocalHost.java 2001/08/15 23:19:34     1.1.2.2
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsLocalHost.java,v
 1.1.2.1 2001/08/10 23:05:51 rwaldhoff Exp $
  - * $Revision: 1.1.2.1 $
  - * $Date: 2001/08/10 23:05:51 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/TestMethodsLocalHost.java,v
 1.1.2.2 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.2 $
  + * $Date: 2001/08/15 23:19:34 $
    *
    * ====================================================================
    *
  @@ -79,7 +79,7 @@
    *
    * @author Remy Maucherat
    * @author Rodney Waldhoff
  - * @version $Id: TestMethodsLocalHost.java,v 1.1.2.1 2001/08/10 23:05:51 rwaldhoff 
Exp $
  + * @version $Id: TestMethodsLocalHost.java,v 1.1.2.2 2001/08/15 23:19:34 rwaldhoff 
Exp $
    */
   public class TestMethodsLocalHost extends TestCase {
   
  @@ -157,7 +157,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.",
                      (data.length() > 0));
  @@ -178,7 +178,7 @@
           }
   
           try {
  -            String data = method.getDataAsString();
  +            String data = method.getResponseBodyAsString();
               // This enumeration musn't be empty
               assert("No data returned.",
                      (data.length() > 0));
  
  
  
  1.1.2.2   +8 -8      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappHeaders.java
  
  Index: TestWebappHeaders.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappHeaders.java,v
  retrieving revision 1.1.2.1
  retrieving revision 1.1.2.2
  diff -u -r1.1.2.1 -r1.1.2.2
  --- TestWebappHeaders.java    2001/08/13 16:02:57     1.1.2.1
  +++ TestWebappHeaders.java    2001/08/15 23:19:34     1.1.2.2
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappHeaders.java,v
 1.1.2.1 2001/08/13 16:02:57 rwaldhoff Exp $
  - * $Revision: 1.1.2.1 $
  - * $Date: 2001/08/13 16:02:57 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappHeaders.java,v
 1.1.2.2 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.2 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -32,7 +32,7 @@
    * "httpclient.test.webappContext" property.
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestWebappHeaders.java,v 1.1.2.1 2001/08/13 16:02:57 rwaldhoff Exp 
$
  + * @version $Id: TestWebappHeaders.java,v 1.1.2.2 2001/08/15 23:19:34 rwaldhoff Exp 
$
    */
   public class TestWebappHeaders extends TestCase {
   
  @@ -85,8 +85,8 @@
               fail("Unable to execute method : " + t.toString());
           }
           // Tomcat 4 at least converts the header name to lower case
  -        
assert(method.getDataAsString().indexOf("name=\"addrequestheader(header)\";value=\"True\"<br>")
 >= 0);
  -        
assert(method.getDataAsString().indexOf("name=\"addrequestheader(string,string)\";value=\"Also
 True\"<br>") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"addrequestheader(header)\";value=\"True\"<br>")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"addrequestheader(string,string)\";value=\"Also
 True\"<br>") >= 0);
       }
   
       /**
  @@ -106,7 +106,7 @@
               fail("Unable to execute method : " + t.toString());
           }
           // Tomcat 4 at least converts the header name to lower case
  -        assert(!(method.getDataAsString().indexOf("xxx-a-header") >= 0));
  +        assert(!(method.getResponseBodyAsString().indexOf("xxx-a-header") >= 0));
       }
   
       /**
  @@ -126,7 +126,7 @@
               fail("Unable to execute method : " + t.toString());
           }
           // Tomcat 4 at least converts the header name to lower case
  -        
assert(method.getDataAsString().indexOf("name=\"xxx-a-header\";value=\"two\"<br>") >= 
0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"xxx-a-header\";value=\"two\"<br>")
 >= 0);
       }
   
       /**
  
  
  
  1.1.2.3   +9 -9      
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappMethods.java
  
  Index: TestWebappMethods.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappMethods.java,v
  retrieving revision 1.1.2.2
  retrieving revision 1.1.2.3
  diff -u -r1.1.2.2 -r1.1.2.3
  --- TestWebappMethods.java    2001/08/14 23:06:46     1.1.2.2
  +++ TestWebappMethods.java    2001/08/15 23:19:34     1.1.2.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappMethods.java,v
 1.1.2.2 2001/08/14 23:06:46 rwaldhoff Exp $
  - * $Revision: 1.1.2.2 $
  - * $Date: 2001/08/14 23:06:46 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappMethods.java,v
 1.1.2.3 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.3 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -32,7 +32,7 @@
    * "httpclient.test.webappContext" property.
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestWebappMethods.java,v 1.1.2.2 2001/08/14 23:06:46 rwaldhoff Exp 
$
  + * @version $Id: TestWebappMethods.java,v 1.1.2.3 2001/08/15 23:19:34 rwaldhoff Exp 
$
    */
   public class TestWebappMethods extends TestCase {
   
  @@ -83,7 +83,7 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
   
           method.recycle();
  @@ -95,7 +95,7 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
       }
   
  @@ -113,7 +113,7 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
           assertEquals(200,method.getStatusCode());
   
           method.recycle();
  @@ -125,7 +125,7 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
           assertEquals(200,method.getStatusCode());
       }
   
  @@ -272,7 +272,7 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        
assert(method.getDataAsString().indexOf("<tt>quote=It+was+the+best+of+times%2C+it+was+the+worst+of+times.</tt>")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<tt>quote=It+was+the+best+of+times%2C+it+was+the+worst+of+times.</tt>")
 >= 0);
           assertEquals(200,method.getStatusCode());
       }
   
  
  
  
  1.1.2.3   +27 -27    
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappParameters.java
  
  Index: TestWebappParameters.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappParameters.java,v
  retrieving revision 1.1.2.2
  retrieving revision 1.1.2.3
  diff -u -r1.1.2.2 -r1.1.2.3
  --- TestWebappParameters.java 2001/08/15 22:55:05     1.1.2.2
  +++ TestWebappParameters.java 2001/08/15 23:19:34     1.1.2.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappParameters.java,v
 1.1.2.2 2001/08/15 22:55:05 rwaldhoff Exp $
  - * $Revision: 1.1.2.2 $
  - * $Date: 2001/08/15 22:55:05 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappParameters.java,v
 1.1.2.3 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.3 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -32,7 +32,7 @@
    * "httpclient.test.webappContext" property.
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestWebappParameters.java,v 1.1.2.2 2001/08/15 22:55:05 rwaldhoff 
Exp $
  + * @version $Id: TestWebappParameters.java,v 1.1.2.3 2001/08/15 23:19:34 rwaldhoff 
Exp $
    */
   public class TestWebappParameters extends TestCase {
   
  @@ -84,9 +84,9 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"hadQuestionMark=true\"</p>") 
>= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"hadQuestionMark=true\"</p>")
 >= 0);
       }
   
       /**
  @@ -105,9 +105,9 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"hadQuestionMark=false\"</p>")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"hadQuestionMark=false\"</p>")
 >= 0);
       }
   
       /**
  @@ -126,9 +126,9 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"param-one=param-value\"</p>")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"param-one=param-value\"</p>")
 >= 0);
       }
   
       /**
  @@ -151,11 +151,11 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("name=\"special-chars\";value=\":/?~.\"") >= 
0);
  -        
assert(method.getDataAsString().indexOf("name=\"param-one\";value=\"param-value\"") >= 
0);
  -        
assert(method.getDataAsString().indexOf("name=\"param-two\";value=\"param-value2\"") 
>= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"special-chars\";value=\":/?~.\"")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"param-one\";value=\"param-value\"")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"param-two\";value=\"param-value2\"")
 >= 0);
       }
   
       /**
  @@ -174,9 +174,9 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"param-without-value\"</p>") 
>= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"param-without-value\"</p>")
 >= 0);
       }
   
       /**
  @@ -198,10 +198,10 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("name=\"foo\";value=\"one\"") >= 0);
  -        assert(method.getDataAsString().indexOf("name=\"foo\";value=\"two\"") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"foo\";value=\"one\"") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"foo\";value=\"two\"") >= 0);
       }
   
       public void testGetMethodOverwriteQueryString() throws Exception {
  @@ -220,11 +220,11 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("name=\"query\";value=\"string\"") 
== -1);
  -        assert(method.getDataAsString().indexOf("name=\"param\";value=\"eter\"") >= 
0);
  -        assert(method.getDataAsString().indexOf("name=\"para\";value=\"meter\"") >= 
0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"query\";value=\"string\"") == 
-1);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"param\";value=\"eter\"") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"para\";value=\"meter\"") >= 0);
       }
   
       /**
  @@ -246,11 +246,11 @@
               t.printStackTrace();
               fail("Unable to execute method : " + t.toString());
           }
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
           assertEquals(200,method.getStatusCode());
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"query=string\"</p>") >= 0);
  -        
assert(method.getDataAsString(),method.getDataAsString().indexOf("name=\"param\";value=\"eter\"")
 >= 0);
  -        assert(method.getDataAsString().indexOf("name=\"para\";value=\"meter\"") >= 
0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"query=string\"</p>") 
>= 0);
  +        
assert(method.getResponseBodyAsString(),method.getResponseBodyAsString().indexOf("name=\"param\";value=\"eter\"")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"para\";value=\"meter\"") >= 0);
       }
   }
   
  
  
  
  1.1.2.3   +13 -13    
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappRedirect.java
  
  Index: TestWebappRedirect.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappRedirect.java,v
  retrieving revision 1.1.2.2
  retrieving revision 1.1.2.3
  diff -u -r1.1.2.2 -r1.1.2.3
  --- TestWebappRedirect.java   2001/08/14 23:06:46     1.1.2.2
  +++ TestWebappRedirect.java   2001/08/15 23:19:34     1.1.2.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappRedirect.java,v
 1.1.2.2 2001/08/14 23:06:46 rwaldhoff Exp $
  - * $Revision: 1.1.2.2 $
  - * $Date: 2001/08/14 23:06:46 $
  + * $Header: 
/home/cvs/jakarta-commons/httpclient/src/test/org/apache/commons/httpclient/Attic/TestWebappRedirect.java,v
 1.1.2.3 2001/08/15 23:19:34 rwaldhoff Exp $
  + * $Revision: 1.1.2.3 $
  + * $Date: 2001/08/15 23:19:34 $
    * ====================================================================
    * Copyright (C) The Apache Software Foundation. All rights reserved.
    *
  @@ -33,7 +33,7 @@
    * "httpclient.test.webappContext" property.
    *
    * @author Rodney Waldhoff
  - * @version $Id: TestWebappRedirect.java,v 1.1.2.2 2001/08/14 23:06:46 rwaldhoff 
Exp $
  + * @version $Id: TestWebappRedirect.java,v 1.1.2.3 2001/08/15 23:19:34 rwaldhoff 
Exp $
    */
   public class TestWebappRedirect extends TestCase {
   
  @@ -83,7 +83,7 @@
               fail("Unable to execute method : " + t.toString());
           }
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
       }
   
       public void testRedirectWithQueryString() throws Exception {
  @@ -99,8 +99,8 @@
               fail("Unable to execute method : " + t.toString());
           }
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>") >= 
0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>")
 >= 0);
       }
   
       public void testRecursiveRedirect() throws Exception {
  @@ -121,8 +121,8 @@
               fail("Unable to execute method : " + t.toString());
           }
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>") >= 
0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
GET</title>") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>")
 >= 0);
       }
   
       public void testPostRedirect() throws Exception {
  @@ -140,10 +140,10 @@
               fail("Unable to execute method : " + t.toString());
           }
           assertEquals(200,method.getStatusCode());
  -        assert(method.getDataAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
  -        
assert(method.getDataAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>") >= 
0);
  -        assert(method.getDataAsString().indexOf("name=\"para\";value=\"meter\"") >= 
0);
  -        assert(method.getDataAsString().indexOf("name=\"param\";value=\"eter\"") >= 
0);
  +        assert(method.getResponseBodyAsString().indexOf("<title>Param Servlet: 
POST</title>") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("<p>QueryString=\"foo=bar&bar=foo\"</p>")
 >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"para\";value=\"meter\"") >= 0);
  +        
assert(method.getResponseBodyAsString().indexOf("name=\"param\";value=\"eter\"") >= 0);
       }
   
       public void testPutRedirect() throws Exception {
  
  
  

Reply via email to