On Mon, 12 Jun 2023 11:30:19 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

> Can I please get a review of this test-only change which moves the tests from 
> under `test/jdk/sun/misc/URLClassPath` to 
> `test/jdk/jdk/internal/loader/URLClassPath`? This addresses 
> https://bugs.openjdk.org/browse/JDK-8309763.
> 
> As noted in that issue, in Java 9, the `sun.misc.URLClassPath` was moved to 
> `jdk.internal.loader.URLClassPath`. These are some pre-existing tests which 
> are now being moved to this more relevant location.
> 
> I've checked that none of the problem list files have this 
> `sun/misc/URLClassPath` directory problem listed. Additionally, I have 
> verified that `sun/misc/URLClassPath` and `jdk/internal/loader` both belong 
> to the same test group (`jdk_lang` which belongs to `tier1`) and as such this 
> refactoring won't impact the test grouping.
> 
> tier1 testing with this change has passed.

Thank you Alan and Mandy for the reviews.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/14415#issuecomment-1588335413

Reply via email to