Hi Frank,

Thanks for fixing this. How did you find this out? Are you running some kind of tools? I hope that's the case since then we could be sure there were no other cases in the test suite.

The patch looks good. It's fortunate we haven't broken anything ;-) The test passed just fine.

Best,
Joe

On 8/27/19 2:06 AM, Frank Yuan wrote:
Hi all

We found 2 jaxp tests, which didn't run indeed.

Bugs:

https://bugs.openjdk.java.net/browse/JDK-8230002

Annotation @Test was missed in TestNG test.

https://bugs.openjdk.java.net/browse/JDK-8230010

The old test was left during it was converted to TestNG test.

Webrev:

http://cr.openjdk.java.net/~fyuan/8230002_8230010/webrev.00/

Thanks

Frank


Reply via email to