Thanks Oleg. I had not realized how big this was until I committed the patch. I would have posted it for review if I had realized.

I have some more plans for work in the test code. I will post some ideas before I get to work. Please let me know if you have any ideas about how we can improve our tests.

Thanks,

Mike

On Wednesday, March 5, 2003, at 03:39 AM, Kalnichevski, Oleg wrote:

Mike
This is massive!!! It's a much needed and welcome refactoring. Many thanks for this work
Oleg


-----Original Message-----
From: Michael Becke [mailto:[EMAIL PROTECTED]
Sent: Mittwoch, 5. März 2003 05:34
To: Commons Project
Subject: cvs commit:
jakarta-commons/httpclient/src/test/org/apache/commons/httpclient


Below is a forward of a CVS commit I posted to the test directory a few minutes ago. It was rejected by the jakarta-commons-cvs mailing list as it was over 100KB. I had not realized it was so large. The diffs have been removed to reduce size.

Mike

mbecke 2003/03/04 20:02:57

Modified: httpclient/src/test/org/apache/commons/httpclient
TestHttpConnectionManager.java
TestWebappBasicAuth.java
TestWebappNoncompliant.java
TestMethodsLocalHost.java
TestWebappPostMethod.java
TestGetMethodLocal.java TestLocalHostBase.java
TestWebappCookie.java TestLocalHost.java
TestWebappMultiPostMethod.java
TestWebappParameters.java
TestTraceMethodLocal.java
TestWebappBase.java TestWebappMethods.java
TestHttpConnection.java TestWebappRedirect.java
TestWebappHeaders.java
Added: httpclient/src/test/org/apache/commons/httpclient
ExecuteMethodThread.java
Removed: httpclient/src/test/org/apache/commons/httpclient
TestHttpClientLocalHost.java
Log:
Refactored localhost tests. All tests that access the localhost now
subclass TestLocalHostBase. This way there is only once place that
configures an HttpClient instance. This will be useful if we want to
test against hosts other that localhost and if we want to test through
a proxy or with https.


Added multi-threaded test for HttpConnectionManager.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to