On 30/08/2019 03:34, Amy Lu 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
This looks okay.
-Alan