Fix SecurityManager.

Author: albertus82

src/main/java/io/github/albertus82/jface/sysinfo/SystemInformationDialog.java

@@ -1,6 +1,5 @@
 package io.github.albertus82.jface.sysinfo;
 
-import java.lang.management.ManagementPermission;
 import java.lang.reflect.InvocationTargetException;
 import java.util.Collections;
 import java.util.Map;
@@ -96,32 +95,7 @@ public class SystemInformationDialog extends Dialog {
 	 *         {@code false}.
 	 */
 	public static boolean isAvailable() {
-		final SecurityManager sm = System.getSecurityManager();
-		if (sm == null) {
-			return true;
-		}
-		try {
-			sm.checkPropertiesAccess(); // system properties
-			return true;
-		}
-		catch (final SecurityException e) {
-			log.log(Level.FINE, "The calling thread does not have permission to access the system properties:", e);
-		}
-		try {
-			sm.checkPermission(new RuntimePermission("getenv.*")); // environment variables
-			return true;
-		}
-		catch (final SecurityException e) {
-			log.log(Level.FINE, "Access to the environment variables is not permitted based on the current security policy:", e);
-		}
-		try {
-			sm.checkPermission(new ManagementPermission("monitor")); // jvm args
-			return true;
-		}
-		catch (final SecurityException e) {
-			log.log(Level.FINE, "Access to the JVM arguments is not permitted based on the current security policy:", e);
-		}
-		return false;
+		return true;
 	}
 
 	/**

src/test/java/io/github/albertus82/jface/sysinfo/SystemInformationDialogTest.java

@@ -1,13 +1,8 @@
 package io.github.albertus82.jface.sysinfo;
 
-import java.lang.management.ManagementPermission;
-import java.security.Permission;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
-import org.junit.AfterClass;
 import org.junit.Assert;
-import org.junit.BeforeClass;
 import org.junit.Test;
 
 import io.github.albertus82.util.logging.LoggerFactory;
@@ -16,104 +11,9 @@ public class SystemInformationDialogTest {
 
 	private static final Logger log = LoggerFactory.getLogger(SystemInformationDialogTest.class);
 
-	private static boolean skip = false;
-
-	@BeforeClass
-	public static void beforeAll() {
-		if (System.getSecurityManager() != null) {
-			log.log(Level.WARNING, "SecurityManager detected, ignoring test {0}.", SystemInformationDialogTest.class);
-			skip = true;
-		}
-		else {
-			try {
-				System.setSecurityManager(new SecurityManager());
-			}
-			catch (final UnsupportedOperationException e) {
-				log.log(Level.WARNING, "SecurityManager deprecated, ignoring test {0}.", SystemInformationDialogTest.class);
-				skip = true;
-			}
-		}
-	}
-
-	@AfterClass
-	public static void afterAll() {
-		if (!skip) {
-			System.setSecurityManager(null);
-		}
-	}
-
 	@Test
 	public void testIsAvailable() {
-		if (skip) {
-			return;
-		}
-
-		Assert.assertTrue(SystemInformationDialog.isAvailable());
-
-		System.setSecurityManager(new SecurityManager() {
-			@Override
-			public void checkPropertiesAccess() {
-				throw new SecurityException("1");
-			}
-
-			@Override
-			public void checkPermission(final Permission perm) { /* NOOP */ }
-		});
-		Assert.assertTrue(SystemInformationDialog.isAvailable());
-
-		System.setSecurityManager(new SecurityManager() {
-			@Override
-			public void checkPropertiesAccess() { /* NOOP */ }
-
-			@Override
-			public void checkPermission(final Permission perm) {
-				if (new RuntimePermission("getenv.*").equals(perm)) {
-					throw new SecurityException("2");
-				}
-			}
-		});
-		Assert.assertTrue(SystemInformationDialog.isAvailable());
-
-		System.setSecurityManager(new SecurityManager() {
-			@Override
-			public void checkPropertiesAccess() { /* NOOP */ }
-
-			@Override
-			public void checkPermission(final Permission perm) {
-				if (new ManagementPermission("monitor").equals(perm)) {
-					throw new SecurityException("3");
-				}
-			}
-		});
 		Assert.assertTrue(SystemInformationDialog.isAvailable());
-
-		System.setSecurityManager(new SecurityManager() {
-			@Override
-			public void checkPropertiesAccess() { /* NOOP */ }
-
-			@Override
-			public void checkPermission(final Permission perm) {
-				if (new RuntimePermission("getenv.*").equals(perm) || new ManagementPermission("monitor").equals(perm)) {
-					throw new SecurityException("4");
-				}
-			}
-		});
-		Assert.assertTrue(SystemInformationDialog.isAvailable());
-
-		System.setSecurityManager(new SecurityManager() {
-			@Override
-			public void checkPropertiesAccess() {
-				throw new SecurityException("5");
-			}
-
-			@Override
-			public void checkPermission(final Permission perm) {
-				if (new RuntimePermission("getenv.*").equals(perm) || new ManagementPermission("monitor").equals(perm)) {
-					throw new SecurityException("6");
-				}
-			}
-		});
-		Assert.assertFalse(SystemInformationDialog.isAvailable());
 	}
 
 }