8007322: make ftp regression tests optional
Alan Bateman
Alan.Bateman at oracle.com
Fri Feb 1 03:27:32 PST 2013
On 31/01/2013 22:16, Chris Hegarty wrote:
> Given the recent downgrade of the ftp handler by 8000941: "Remove ftp
> from the required list of protocol handlers", the regression tests
> that exercise the ftp protocol handler should be made optional. This
> will make testing the smallest profile and the base module easier (
> rather than trying to exclude individual tests ).
>
> http://cr.openjdk.java.net/~chegar/8007322/webrev.00/webrev/
>
> Alan,
> I hacked the ftp classes out of a full build to test this. You may
> want to try the patch with a profiles build. All the jdk_net tests
> should pass.
Thanks for doing this.
If you want to build the profile images yourself then just clone
jdk8/profiles forest, the make target is "profiles". David Holmes is
planning to push this to jdk8 later in the month.
I see you've updated tests in test/sun/net/ftp or
test/sun/net/www/protocol/ftp and I don't think that is strictly needed
as they are straight-forward to exclude as not-applicable directories.
I see that hasFTP catches IOException in many places, I assume this
should be MalformedURLException. I guess I would rename it to hasFtp but
that's a minor point.
Should test/sun/www/MarkResetTest* be moved to the ftp directory as it
is an ftp specific test?
The big refactoring seems to the URL/Constructor.java, looks like you've
moved all the test cases into the java file. The update looks reasonable
to me, I guess I would have chosen something like TestCase rather than
Entry but what you have is fine if you want to keep it.
-Alan
More information about the net-dev
mailing list