Review request: jdk jtreg testing with jdk-module-image
Mandy Chung
mandy.chung at oracle.com
Fri Jun 4 08:26:41 PDT 2010
Alan Bateman wrote:
> Mandy Chung wrote:
>> Webrev:
>> http://cr.openjdk.java.net/~mchung/jigsaw/jprt-jigsaw-tests/forest
>> http://cr.openjdk.java.net/~mchung/jigsaw/jprt-jigsaw-tests/jdk/
>>
>> With this fix, we can run the jdk test targets with the
>> jdk-module-image from making jdk/test/Makefile or in a jprt job.
>> This also adds a new jdk_jigsaw test target for testing module image.
>>
>> I add a new "modules" keyword and the test/Makefile will set the
>> keyword to exclude modules tests if testing a legacy image. The
>> java/lang/module and java/lang/reflect/Module jtreg tests are tagged
>> with the new "modules" keyword since these tests are included in
>> jdk_lang target and also jdk_jigsaw is included in the jprt test
>> targets.
>>
>> test/ModulesProblemList.txt is intended for temporarily use. Some of
>> the failures are known issues that should be fixed soon.
> I did a pass over this. It mostly looks good to me.
>
> In test/Makefile would it be more readable/consistent to use
> MODULE_BUILD instead of TEST_MODULES to indicate that it's a modules
> build? Minor nit is that a spurious blank line is added at line 347.
>
ok. Will update that.
> In jprt.properties I see you've removed the fastdebug build from the
> list of flavors. Is that intended?
>
This is not intended. I'll revert that line's change.
> Finally, did you mean to include the new tests and fixes to the other
> tests in this webrev? I look through them and they look good.
>
Thanks for reviewing the tests. This webrev includes the test fixes
and new tests for review as well.
Mandy
More information about the jigsaw-dev
mailing list