You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by bu...@apache.org on 2002/05/02 22:02:07 UTC
DO NOT REPLY [Bug 8753] New: -
test-nohost unit tests assume https is installed, causing assertion
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: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>