RFR: 8244199: [TestBug]: Update sun/management/jmxremote/bootstrap tests

Thejasvi Voniadka thejasvi.v.voniadka at oracle.com
Tue May 12 02:18:27 UTC 2020


Ping..


-----Original Message-----
From: Thejasvi Voniadka 
Sent: Friday, May 8, 2020 6:17 AM
To: core-libs-dev at openjdk.java.net; jmx-dev at openjdk.java.net; security-dev at openjdk.java.net
Subject: RFR: 8244199: [TestBug]: Update sun/management/jmxremote/bootstrap tests


Hi,

May someone please sponsor this patch?

Bug: https://bugs.openjdk.java.net/browse/JDK-8244199

Description:
	The test "test/jdk/sun/management/jmxremote/bootstrap/RmiSslBootstrapTest.sh" fails intermittently on a lower release. While investigating the cause for the failure, I felt the diagnostic messaging built into the test was not adequate. Some of the issues observed:
	1. If the test throws a RuntimeException, the full stack trace is not displayed.
	2. The test has several logging statements. However, the logging environment is not properly initialized to allow levels such as FINE and FINER.
	3. The logging sequence could be improved: some messages end up at stdout, and others at stderr, which makes it harder to follow the output.

The patch is to improve the test in these lines.

Webrev: http://cr.openjdk.java.net/~tvoniadka/reviews/8244199/webrev.full.00

I have also used this opportunity to clean-up the test code as a whole, in lines of removal of redundant logic, formatting, coding guidelines, etc.... If the above patch looks too confusing and is hard to follow, here is a simpler version that shows only the core logic changes (I intend to submit the full patch to be pushed though): http://cr.openjdk.java.net/~tvoniadka/reviews/8244199/webrev.simple.00

The patch has been tested on mach5, and all jmxremote tests passed.






More information about the core-libs-dev mailing list