BAEL-3732: Move core-java-security-manager into core-java-security (eugenp#8460)

kwoyke · pivovarit · commit 04a571ea02d4 · 2020-01-03T08:59:33.000+02:00
* BAEL-3732: Move core-java-security-manager into core-java-security

* BAEL-3732: Teardown the security manager to not interfere with other tests
diff --git a/core-java-modules/core-java-security-manager/README.md b/core-java-modules/core-java-security-manager/README.md
diff --git a/core-java-modules/core-java-security-manager/pom.xml b/core-java-modules/core-java-security-manager/pom.xml
diff --git a/core-java-modules/core-java-security-manager/src/test/java/com/baeldung/security/manager/SecurityManagerUnitTest.java b/core-java-modules/core-java-security-manager/src/test/java/com/baeldung/security/manager/SecurityManagerUnitTest.java
diff --git a/core-java-modules/core-java-security/README.md b/core-java-modules/core-java-security/README.md
@@ -15,4 +15,5 @@ This module contains articles about core Java Security
 - [The Java SecureRandom Class](https://www.baeldung.com/java-secure-random)
 - [An Introduction to Java SASL](https://www.baeldung.com/java-sasl)
 - [A Guide to Java GSS API](https://www.baeldung.com/java-gss)
+- [Intro to the Java SecurityManager](https://www.baeldung.com/java-security-manager)
 
diff --git a/core-java-modules/core-java-security/src/main/java/com/baeldung/securitymanager/CustomPermission.java b/core-java-modules/core-java-security/src/main/java/com/baeldung/securitymanager/CustomPermission.java
@@ -1,4 +1,4 @@
-package com.baeldung.security.manager;
+package com.baeldung.securitymanager;
 
 import java.security.BasicPermission;
 
diff --git a/core-java-modules/core-java-security/src/main/java/com/baeldung/securitymanager/Service.java b/core-java-modules/core-java-security/src/main/java/com/baeldung/securitymanager/Service.java
@@ -1,4 +1,4 @@
-package com.baeldung.security.manager;
+package com.baeldung.securitymanager;
 
 public class Service {
 
diff --git a/core-java-modules/core-java-security/src/test/java/com/baeldung/java/md5/JavaMD5UnitTest.java b/core-java-modules/core-java-security/src/test/java/com/baeldung/java/md5/JavaMD5UnitTest.java
@@ -1,4 +1,4 @@
-package org.baeldung.java.md5;
+package com.baeldung.java.md5;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
diff --git a/core-java-modules/core-java-security/src/test/java/com/baeldung/securitymanager/SecurityManagerUnitTest.java b/core-java-modules/core-java-security/src/test/java/com/baeldung/securitymanager/SecurityManagerUnitTest.java
@@ -0,0 +1,35 @@
+package com.baeldung.securitymanager;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+import java.io.IOException;
+import java.net.URL;
+import java.security.AccessControlException;
+
+public class SecurityManagerUnitTest {
+
+    private static final String TESTING_SECURITY_POLICY = "file:src/test/resources/testing.policy";
+
+    @Before
+    public void setUp() {
+        System.setProperty("java.security.policy", TESTING_SECURITY_POLICY);
+        System.setSecurityManager(new SecurityManager());
+    }
+
+    @After
+    public void tearDown() {
+        System.setSecurityManager(null);
+    }
+
+    @Test(expected = AccessControlException.class)
+    public void whenSecurityManagerIsActive_thenNetworkIsNotAccessibleByDefault() throws IOException {
+        new URL("http://www.google.com").openConnection().connect();
+    }
+
+    @Test(expected = AccessControlException.class)
+    public void whenUnauthorizedClassTriesToAccessProtectedOperation_thenAnExceptionIsThrown() {
+        new Service().operation();
+    }
+}
diff --git a/core-java-modules/core-java-security/src/test/resources/testing.policy b/core-java-modules/core-java-security/src/test/resources/testing.policy
@@ -0,0 +1,5 @@
+grant {
+        // This is for testing purposes only.
+        // It allows us to properly reset the security manager after the unit test completes.
+        permission java.lang.RuntimePermission "setSecurityManager";
+};
diff --git a/core-java-modules/pom.xml b/core-java-modules/pom.xml
@@ -18,7 +18,6 @@
         <module>core-java-optional</module>
         <module>core-java-lang-operators</module>
         <module>core-java-networking-2</module>
-        <module>core-java-security-manager</module>
         <module>core-java-date-operations</module>
     </modules>
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-package com.baeldung.security.manager;`
	`1`	`+package com.baeldung.securitymanager;`
`2`	`2`
`3`	`3`	`import java.security.BasicPermission;`
`4`	`4`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-package org.baeldung.java.md5;`
	`1`	`+package com.baeldung.java.md5;`
`2`	`2`
`3`	`3`	`import static org.assertj.core.api.Assertions.assertThat;`
`4`	`4`