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