8183503: Update hotspot tests to allow for unique test classes directory
ioi.lam at oracle.com
Wed Jul 5 07:50:50 UTC 2017
The changes look good to me. Will these test work with older versions of
jtreg, or will they require the updated jtreg? I hope the former is
true, or else it may disrupt testing.
On 7/3/17 10:51 AM, Alan Bateman wrote:
> jtreg has been updated via CODETOOLS-7902003  to use a unique
> test.classes directory. This is part of improving the reliability when
> running tests concurrently. The change exposes issues in a small
> number of tests in the hotspot, jdk, and langtools repos and these
> issues will need to be fixed before the requiredVersion can be bumped
> to use a promoted jtreg with this change.
> For the hotspot repo the tests in runtime/ClassUnload all fail with
> this change due to a class loader that the tests create and
> unreference to exercise class unloading. Easily fixed by changing the
> class loader to one that attempts to loads the class before delegation
> to the parent (this ensure the class is not loaded by the parent).
> Another test that needed update is
> runtime/logging/ClassLoadUnlaodTest.java as it's a driver test and
> needs to start the child VM with test.class.path rather than
> test.classes. Finally one of the compiler tests wasn't locate the
> class bytes correctly so it needed an update too.
> The webrev with the changes is here:
> I'd like to push these to jdk10/jdk10 so that they can meet up with
> changes coming for related issues in the jdk and langtools repos.
>  https://bugs.openjdk.java.net/browse/CODETOOLS-7902003
More information about the hotspot-runtime-dev