[JDK 14] RFR 8230004: jdk/internal/jimage/JImageOpenTest.java runs no test

Jim Laskey james.laskey at oracle.com
Fri Aug 30 12:14:12 UTC 2019


+1

> On Aug 29, 2019, at 11:34 PM, Amy Lu <amy.lu at oracle.com> wrote:
> 
> jdk/internal/jimage/JImageOpenTest.java
> 
> This test is not a testng test, and is not running any test after TEST.properties was introduced at the same directory with:
> TestNG.dirs = .
> 
> There are two tests in this directory: JImageReadTest is a testng test, JImageOpenTest is not.
> It's safe to remove testng related settings from TEST.properties, and specify that in JImageReadTest.
> 
> Other issues found:
> 1. JImageOpenTest.java has a wrong build tag, which can be removed.
> 2. To override the default limit for the amount of output that will be saved on any stream being recorded by the test harness in TEST.properties, maxOutputSize should be used, instead of javatest.maxOutputSize [1]. This issue also exists in some other tests and will be fixed in [2].
> 
> Please review the patch to fix these issues and get JImageOpenTest.java back.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8230004
> webrev: http://cr.openjdk.java.net/~amlu/8230004
> 
> Thanks,
> Amy
> 
> [1] http://openjdk.java.net/jtreg/tag-spec.html
> [2] https://bugs.openjdk.java.net/browse/JDK-8230374
> 



More information about the core-libs-dev mailing list