RFR: JDK-8262277: java.net.URLClassLoader.getResource throws undocumented IllegalArgumentException [v3]
Craig Andrews
github.com+194713+candrews at openjdk.java.net
Fri Mar 5 16:04:48 UTC 2021
On Fri, 5 Mar 2021 15:46:40 GMT, Craig Andrews <github.com+194713+candrews at openjdk.org> wrote:
>> test/jdk/java/net/URLClassLoader/TestBug8262277.java line 30:
>>
>>> 28: * @summary Test to see if URLClassLoader.getResource and URLClassLoader.getResources
>>> 29: * throw IllegalArgumentException
>>> 30: */
>>
>> I'd prefer if the test checked that findResource returned null and that findResources returned an empty enumeration. I think we should be able to find a better name for the test too.
>> Do you really want the author tag in the test? I know they exist in some tests but they are impossible to remove, even when tests/code are significantly changed by someone else.
>
>> I'd prefer if the test checked that findResource returned null and that findResources returned an empty enumeration.
>
> I'll update the test accordingly.
>
>> think we should be able to find a better name for the test too.
>> Do you really want the author tag in the test? I know they exist in some tests but they are impossible to remove, even when tests/code are significantly changed by someone else.
>
> I can rename the test class to be something descriptive and remove the `@author` tag. I was following other tests which is why I did it this way.
I've made the changes you requested.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2662
More information about the core-libs-dev
mailing list