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

[Thejasvi Voniadka]

Hello,

A quick follow-up on this request..


-----Original Message-----
From: Thejasvi Voniadka 
Sent: Monday, May 4, 2020 12:56 PM
To: core-libs-dev at openjdk.java.net
Subject: [PATCH] 8244199: [TestBug]: Update sun/management/jmxremote/bootstrap tests

Hi,

May I please find a sponsor for 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 when needed.
	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 only the full patch to be committed though): http://cr.openjdk.java.net/~tvoniadka/reviews/8244199/webrev.simple.00

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