From 172839f24f63d81efad1b44a47b99d02a5f4c3bc Mon Sep 17 00:00:00 2001 From: Daniel Fuchs Date: Wed, 11 May 2016 19:23:58 +0200 Subject: [PATCH] 8156153: java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java fails with java.lang.RuntimeException Runs each scenario in its own VM to avoid unwanted interaction and start from a clean slate. Reviewed-by: mchung --- .../DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/jdk/test/java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java b/jdk/test/java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java index 7118101cada..c796cd1888d 100644 --- a/jdk/test/java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java +++ b/jdk/test/java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java @@ -59,7 +59,9 @@ import java.lang.reflect.Module; * @modules java.base/sun.util.logging * java.base/jdk.internal.logger * java.logging/sun.util.logging.internal - * @run main/othervm DefaultLoggerBridgeTest + * @run main/othervm DefaultLoggerBridgeTest NOSECURITY + * @run main/othervm DefaultLoggerBridgeTest NOPERMISSIONS + * @run main/othervm DefaultLoggerBridgeTest WITHPERMISSIONS * @author danielfuchs */ public class DefaultLoggerBridgeTest { @@ -275,7 +277,6 @@ public class DefaultLoggerBridgeTest { }; Stream.of(args).map(TestCases::valueOf).forEach((testCase) -> { - LoggerFinder provider; switch (testCase) { case NOSECURITY: System.out.println("\n*** Without Security Manager\n"); @@ -396,10 +397,10 @@ public class DefaultLoggerBridgeTest { final boolean old = allowAll.get().get(); allowAll.get().set(true); try { - sysSink = LoggingProviderImpl.getLogManagerAccess().demandLoggerFor( - LogManager.getLogManager(), "foo", Thread.class.getModule()); appSink = LoggingProviderImpl.getLogManagerAccess().demandLoggerFor( LogManager.getLogManager(), "foo", DefaultLoggerBridgeTest.class.getModule()); + sysSink = LoggingProviderImpl.getLogManagerAccess().demandLoggerFor( + LogManager.getLogManager(), "foo", Thread.class.getModule()); if (appSink == sysSink) { throw new RuntimeException("identical backend loggers"); }