JDK 9 RFR of JDK-8085814: Move stream test library to the jdk test library area
Paul Sandoz
paul.sandoz at oracle.com
Mon May 8 17:33:30 UTC 2017
Hi Amy,
The changes look good.
Would you mind re-basing this fix for JDK 10 instead? At this point we should be more conservative with such changes (even for tests).
Thanks,
Paul.
> On 8 May 2017, at 02:00, Amy Lu <amy.lu at oracle.com> wrote:
>
> Please review this test-only change to move the test library
> jdk/test/java/util/stream/bootlib
> to
> jdk/test/lib/testlibrary/bootlib
> to make it formally easier for tests to depend on.
>
> There is no change to the functionality of the tests.
>
> This patch also removed NullArgsTestCase.java which is an abstract class designed to be extended reused with a data provider. It provides no test itself but be wrongly put in the directory of TestNG.dirs thus be run as a test. In fact it is not even used and can be removed.
>
> bug:
> https://bugs.openjdk.java.net/browse/JDK-8085814
> https://bugs.openjdk.java.net/browse/JDK-8173414
>
> webrev:
> http://cr.openjdk.java.net/~amlu/8085814/webrev.00/
> (I did not update the year in the Copyright header for bootlib files, to make the review easier. I can do bulk update for those files on the header before push, if that is necessary.)
>
> Thanks,
> Amy
>
>
More information about the core-libs-dev
mailing list