+1

> On Aug 29, 2019, at 11:34 PM, Amy Lu <amy...@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
> 

Reply via email to