JDK 9 RFR of JDK-8085814: Move stream test library to the jdk test library area

David Holmes david.holmes at oracle.com
Mon May 8 20:56:35 UTC 2017


Hi Amy,

Not a review ...

On 8/05/2017 7:00 PM, Amy Lu 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

If the bootlib files were simply moved then no update to their copyright 
year is needed.

Cheers,
David

> 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