Updated ArchUnit and added check for application/domain code accessin…

…g random/shuffle code.

Fixed RandomShuffler by moving it to the web admin adapter where it's used.

pom.xml

@@ -139,7 +139,7 @@
     <dependency>
       <groupId>com.tngtech.archunit</groupId>
       <artifactId>archunit-junit5</artifactId>
-      <version>1.2.1</version>
+      <version>1.3.0</version>
       <scope>test</scope>
     </dependency>
 

src/main/java/com/jitterted/mobreg/adapter/in/web/admin/EnsembleTimerLifecycleController.java

@@ -1,7 +1,6 @@
 package com.jitterted.mobreg.adapter.in.web.admin;
 
 import com.jitterted.mobreg.application.EnsembleTimerHolder;
-import com.jitterted.mobreg.application.port.RandomShuffler;
 import com.jitterted.mobreg.domain.EnsembleId;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Controller;

src/main/java/com/jitterted/mobreg/application/port/RandomShuffler.java → src/main/java/com/jitterted/mobreg/adapter/in/web/admin/RandomShuffler.java

@@ -1,5 +1,6 @@
-package com.jitterted.mobreg.application.port;
+package com.jitterted.mobreg.adapter.in.web.admin;
 
+import com.jitterted.mobreg.application.port.Shuffler;
 import com.jitterted.mobreg.domain.Member;
 
 import java.util.Collections;

src/test/java/com/jitterted/mobreg/HexagonalArchitectureTest.java

@@ -13,6 +13,9 @@
 import java.time.Instant;
 import java.time.LocalDateTime;
 import java.time.ZonedDateTime;
+import java.util.Collections;
+import java.util.List;
+import java.util.Random;
 
 import static com.tngtech.archunit.lang.syntax.ArchRuleDefinition.noClasses;
 import static com.tngtech.archunit.library.dependencies.SlicesRuleDefinition.slices;
@@ -64,6 +67,21 @@ void domainAndApplicationMustNotAccessSystemTime() {
                                .importPackages("com.jitterted.mobreg"));
     }
 
+    @Test
+    void domainAndApplicationMustNotAccessRandom() {
+        noClasses()
+                .that().resideInAPackage("..domain..")
+                .or().resideInAPackage("..application..")
+
+                .should().callMethod(Collections.class, "shuffle", List.class)
+                .orShould().dependOnClassesThat().areAssignableTo(Random.class)
+
+                .as("Neither Application nor Domain should access Random or Random-related code (e.g., Collections.shuffle).")
+                .check(new ClassFileImporter()
+                               .withImportOption(ImportOption.Predefined.DO_NOT_INCLUDE_TESTS)
+                               .importPackages("com.jitterted.mobreg"));
+    }
+
     private JavaClasses productionAndTestClasses() {
         return new ClassFileImporter().importPackages("com.jitterted.mobreg");
     }