RFR: 8024014 & 8024015 : (xs) TEST.groups updates

Mandy Chung mandy.chung at oracle.com
Fri Aug 30 01:21:20 UTC 2013


On 8/29/2013 5:13 PM, Mike Duigou wrote:
> Hello all;
>
> This is a review for two changesets. The first change (JDK-8024014) splits up the jdk_util test group a bit by introducing three sub-groups, jdk_collections, jdk_stream and jdk_concurrent. The main advantage is that it's easier/quicker to test individual components. The intent is that the test groups are aligned with bug database sub-components.
>
> The second change moves some important lambda related tests from languishing in obscurity in the jdk_other group to the jdk_lang group to reflect their importance and relation to other tests. These tests are contained in the jdk/lambda directory.
>
> The combined webrev is here:
>
> http://cr.openjdk.java.net/~mduigou/JDK-8024015/0/webrev/

This change looks good to me.  It's good to break jdk_util into 
sub-groups.  Is there a way to avoid listing the rest of java/util/* not 
covered by these new sub-groups (i.e. something like this:
    :jdk_collections \
    :jdk_concurrent \
    :jdk_stream \
    java/util \
    sun/util

so that new subdirectory added under java/util will be included? I have 
seen cases that new tests in a new directory added but not listed in the 
old test/Makefile were never run but unnoticed (you cleaned up that 
logic in JDK-8015068 - thanks).

Mandy

> The effect of these changes won't be visible to most people until JDK-8015068 is integrated.
>
> Thanks,
>
> Mike




More information about the core-libs-dev mailing list