Merge pull request wildfly#18128 from khermano/WFLY-15289_from_main

[WFLY-19717] Remove unused Elytron permissions in MDBRoleTestCase, RunAsPrincipalTestCase, RunAsTestCase, RunAsEjbMdbTestCase, RunAsMDBUnitTestCase and LegacyCompliantPrincipalPropagationTestCase
bstansberry · Sep 19, 2024 · 1377d5f · 1377d5f
2 parents 6d13fbf + a5093f1
commit 1377d5f
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 3 deletions.
diff --git a/...ation/basic/src/test/java/org/jboss/as/test/integration/ejb/security/MDBRoleTestCase.java b/...ation/basic/src/test/java/org/jboss/as/test/integration/ejb/security/MDBRoleTestCase.java
@@ -74,7 +74,6 @@ public static Archive<?> deployment() {
       // TODO WFLY-15289 The Elytron permissions need to be checked, should a deployment really need these?
       deployment.addAsResource(createPermissionsXmlAsset(new PropertyPermission("ts.timeout.factor", "read"),
                                                          new ElytronPermission("setRunAsPrincipal"),
-                                                         new ElytronPermission("handleSecurityEvent"),
                                                          new ChangeRoleMapperPermission("ejb")), "META-INF/jboss-permissions.xml");
       return deployment;
    }

diff --git a/...st/java/org/jboss/as/test/integration/ejb/security/runas/ejb2mdb/RunAsEjbMdbTestCase.java b/...st/java/org/jboss/as/test/integration/ejb/security/runas/ejb2mdb/RunAsEjbMdbTestCase.java
@@ -81,7 +81,6 @@ public static Archive<?> runAsEJB3() {
         // TODO WFLY-15289 Should these permissions be required?
         jar.addAsResource(createPermissionsXmlAsset(new PropertyPermission("ts.timeout.factor", "read"),
                                                     new ElytronPermission("setRunAsPrincipal"),
-                                                    new ElytronPermission("handleSecurityEvent"),
                                                     new ChangeRoleMapperPermission("ejb")), "META-INF/jboss-permissions.xml");
         jar.addAsManifestResource(new StringAsset("Dependencies: deployment.runasmdbejb-ejb2.jar  \n"), "MANIFEST.MF");
         return jar;

diff --git a/.../test/java/org/jboss/as/test/integration/ejb/security/runas/mdb/RunAsMDBUnitTestCase.java b/.../test/java/org/jboss/as/test/integration/ejb/security/runas/mdb/RunAsMDBUnitTestCase.java
@@ -87,7 +87,6 @@ public static Archive<?> deploy() {
         jar.addPackage(CommonCriteria.class.getPackage());
         // TODO WFLY-15289 Should these permissions be required?
         jar.addAsResource(createPermissionsXmlAsset(new ElytronPermission("setRunAsPrincipal"),
-                new ElytronPermission("handleSecurityEvent"),
                 new ChangeRoleMapperPermission("ejb")), "META-INF/jboss-permissions.xml");
         return jar;
     }