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