commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 8753] New: - test-nohost unit tests assume https is installed, causing assertion
Date Thu, 02 May 2002 20:02:07 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=8753>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=8753

test-nohost unit tests assume https is installed, causing assertion

           Summary: test-nohost unit tests assume https is installed,
                    causing assertion
           Product: Commons
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: HttpClient
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: cloder@acm.org


The TestHttpConnectionManager tests, part of the "test-nohost" ant target for 
HttpClient, assume that https is installed. This causes the unit tests to fail.

test-nohost:
     [java] .........................................
     [java] .........................................
     [java] ...........F......
     [java] Time: 5.453
     [java] There was 1 failure:
     [java] 1) testGetConnection
(org.apache.commons.httpclient.TestHttpConnectionManager)
junit.framework.AssertionFailedError: C
aught unexpected MalformedURLException (java.net.MalformedURLException: unknown 
protocol: https)
     [java]     at 
org.apache.commons.httpclient.TestHttpConnectionManager.testGetConnection
(TestHttpConnectionManager.java:156)

     [java] FAILURES!!!
     [java] Tests run: 99,  Failures: 1,  Errors: 0

BUILD FAILED


The offending line in TestHttpConnectionManager is line 141, which reads:

            conn = mgr.getConnection("https://www.nosuchserver.com/path/path?
query=string");

This test should not be included in TestHttpConnectionManager.  If anything, it 
should be included in TestHttps. None of the other tests assume the https 
protocol handler has been installed. If the tests make this assumption, then 
they should install the protocol handler themselves.

I am using the most recent CVS revision:

 * $Header: /home/cvspublic/jakarta-
commons/httpclient/src/test/org/apache/commons/httpclient/TestHttpConnectionMana
ger.java,v 1.1 2002/04/12 21:17:06 marcsaeg Exp $
 * $Revision: 1.1 $
 * $Date: 2002/04/12 21:17:06 $

--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message