change behaviors list to array list

Author: lucasstarsz

src/main/java/tech/fastj/graphics/game/GameObject.java

@@ -3,10 +3,10 @@
 import tech.fastj.graphics.Drawable;
 import tech.fastj.systems.behaviors.Behavior;
 import tech.fastj.systems.behaviors.BehaviorHandler;
-import tech.fastj.systems.collections.ManagedList;
 import tech.fastj.systems.control.GameHandler;
 
 import java.awt.Graphics2D;
+import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 
@@ -22,11 +22,11 @@
  */
 public abstract class GameObject extends Drawable {
 
-    private final ManagedList<Behavior> behaviors;
+    private final List<Behavior> behaviors;
 
     /** Initializes {@link GameObject} internals. */
     protected GameObject() {
-        behaviors = new ManagedList<>();
+        behaviors = new ArrayList<>();
     }
 
     /** {@return the gmae object's {@link Behavior behaviors}} */
@@ -36,34 +36,27 @@ public List<Behavior> getBehaviors() {
 
     /** Calls the {@link Behavior#init(GameObject)} method for each of the {@link GameObject game object}'s behaviors. */
     public void initBehaviors() {
-        behaviors.run(list -> {
-            for (Behavior behavior : list) {
-                behavior.init(this);
-            }
-        });
+        for (Behavior behavior : behaviors) {
+            behavior.init(this);
+        }
     }
 
     /** Calls the {@link Behavior#fixedUpdate(GameObject)} method for each of the {@link GameObject game object}'s behaviors. */
     public void fixedUpdateBehaviors() {
-        behaviors.run(list -> {
-            for (Behavior behavior : list) {
-                behavior.fixedUpdate(this);
-            }
-        });
+        for (Behavior behavior : behaviors) {
+            behavior.fixedUpdate(this);
+        }
     }
 
     /** Calls the {@link Behavior#update(GameObject)} method for each of the {@link GameObject game object}'s behaviors. */
     public void updateBehaviors() {
-        behaviors.run(list -> {
-            for (Behavior behavior : list) {
-                behavior.update(this);
-            }
-        });
+        for (Behavior behavior : behaviors) {
+            behavior.update(this);
+        }
     }
 
     /** Calls the {@link Behavior#destroy()} method for each of the {@link GameObject game object}'s behaviors. */
     public synchronized void destroyAllBehaviors() {
-        behaviors.shutdownNow();
         for (Behavior behavior : behaviors) {
             behavior.destroy();
         }

src/test/java/unittest/testcases/graphics/game/GameObjectTests.java

@@ -196,9 +196,7 @@ void tryUpdateBehaviorWithoutInitializing_shouldThrowNullPointerException() {
         MockBehavior mockBehavior = new MockBehavior();
         Scene mockScene = new MockEmptyScene();
 
-        gameObject.addBehavior(mockBehavior, mockScene);
-
-        Throwable exception = assertThrows(IllegalStateException.class, gameObject::fixedUpdateBehaviors, "Trying to update the behavior without initializing it should throw a null pointer exception on the Pointf, wrapped in an illegal state exception.");
-        assertEquals(NullPointerException.class, exception.getCause().getClass(), "The underlying exception should be a null pointer exception.");
+        gameObject.addBehavior(mockBehavior, mockScene); // pointf is null here
+        assertThrows(NullPointerException.class, gameObject::fixedUpdateBehaviors, "Trying to update the behavior without initializing it should throw a null pointer exception on the Pointf.");
     }
 }