Address feedback

ls1intum · Oct 21, 2024 · 20b9767 · 20b9767
1 parent ec8a350
commit 20b9767
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/...java/de/tum/cit/ase/ares/api/architecture/java/archunit/JavaArchUnitSecurityTestCase.java b/...java/de/tum/cit/ase/ares/api/architecture/java/archunit/JavaArchUnitSecurityTestCase.java
@@ -96,7 +96,9 @@ public void executeArchitectureTestCase(JavaClasses classes) {
                 default -> throw new UnsupportedOperationException("Not implemented yet");
             }
         } catch (AssertionError e) {
-            assert e.getMessage().split("\n").length > 1;
+            if (e.getMessage() == null || e.getMessage().split("\n").length < 2) {
+                throw new SecurityException(localized("security.archunit.illegal.execution", e.getMessage()));
+            }
             throw new SecurityException(localized("security.archunit.illegal.execution", e.getMessage().split("\n")[1]));
         }
     }

diff --git a/...e/ares/api/architecture/java/archunit/postcompile/JavaArchitectureTestCaseCollection.java b/...e/ares/api/architecture/java/archunit/postcompile/JavaArchitectureTestCaseCollection.java
@@ -101,8 +101,7 @@ public static ArchRule noClassesShouldImportForbiddenPackages(Set<String> allowe
                     public boolean test(JavaClass javaClass) {
                         return allowedPackages.stream().noneMatch(allowedPackage -> javaClass.getPackageName().startsWith(allowedPackage));
                     }
-                })
-                .as("TEST TEST");
+                });
     }
 
     private static ArchRule createNoClassShouldHaveMethodRule(

diff --git a/...java/de/tum/cit/ase/ares/api/securitytest/java/JavaSecurityTestCaseFactoryAndBuilder.java b/...java/de/tum/cit/ase/ares/api/securitytest/java/JavaSecurityTestCaseFactoryAndBuilder.java
@@ -30,7 +30,6 @@
 
 import static com.google.common.collect.Iterables.isEmpty;
 import static de.tum.cit.ase.ares.api.aop.java.instrumentation.advice.JavaInstrumentationAdviceToolbox.localize;
-import static de.tum.cit.ase.ares.api.jupiter.JupiterSecurityExtension.resetSettings;
 //</editor-fold>
 
 /**
@@ -197,10 +196,6 @@ public JavaSecurityTestCaseFactoryAndBuilder(
      * </p>
      */
     private void createSecurityTestCases() {
-        //<editor-fold desc="Delete old rules code">
-        // Reset
-        //</editor-fold>
-
         //<editor-fold desc="Create fixed rules code">
         Set<SecurityPolicy.PackagePermission> allowedPackages = new HashSet<>(resourceAccesses.regardingPackageImports());
         // Add default imports needed for the execution