[9] RFR(S): 8142464: PlatformLoggerTest.java throws java.lang.RuntimeException: Logger test.logger.bar does not exist

[Vladimir Kozlov]

Good.

thanks,
Vladimir

On 4/28/16 5:01 AM, Nils Eliasson wrote:
> Hi,
>
> Please review the fix of jdk/test/sun/util/logging/PlatformLoggerTest.java that has been failing intermittently in our nightlies.
>
> Summary:
> The test uses loggers that are accessible by weak refs from a LogManager. Since the test doesn't keep a strong ref to the loggers they may get collected during the test.
>
> Solution:
> Save loggers to a static field  in the test class.
>
> Other:
> Also removing "@compile -XDignore.symbol.file" that is unnecessary after Jigsaw. The compile tag will force a compile even if the class already exists, wasting times during reruns.
>
> Testing:
> Running tests on all platforms.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8142464
> webrev: http://cr.openjdk.java.net/~neliasso/8142464/webrev_jdk.01/ (JDK)
>
> Regards,
> Nils Eliasson