refactor: change tests that used @NotNull

src/test/java/edu/ntnu/idatt2001/group_30/actions/GoldActionTest.java

@@ -41,11 +41,11 @@ class GoldActionTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_executing_with_null_argument() {
+        public void throws_NullPointerException_executing_with_null_argument() {
             GoldAction goldAction = new GoldAction(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> goldAction.execute(player));
+            assertThrows(NullPointerException.class, () -> goldAction.execute(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/actions/HealthActionTest.java

@@ -41,11 +41,11 @@ class HealthActionTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_executing_with_null_argument() {
+        public void throws_NullPointerException_executing_with_null_argument() {
             HealthAction healthAction = new HealthAction(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> healthAction.execute(player));
+            assertThrows(NullPointerException.class, () -> healthAction.execute(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/actions/InventoryActionTest.java

@@ -44,11 +44,11 @@ class InventoryActionTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_executing_with_null_argument() {
+        public void throws_NullPointerException_executing_with_null_argument() {
             InventoryAction inventoryAction = new InventoryAction("Apple");
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> inventoryAction.execute(player));
+            assertThrows(NullPointerException.class, () -> inventoryAction.execute(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/actions/ScoreActionTest.java

@@ -41,11 +41,11 @@ class ScoreActionTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_executing_with_null_argument() {
+        public void throws_NullPointerException_executing_with_null_argument() {
             ScoreAction scoreAction = new ScoreAction(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> scoreAction.execute(player));
+            assertThrows(NullPointerException.class, () -> scoreAction.execute(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/goals/GoldGoalTest.java

@@ -52,11 +52,11 @@ class GoldGoalTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_checking_with_null_as_player() {
+        public void throws_NullPointerException_checking_with_null_as_player() {
             GoldGoal goldGoal = new GoldGoal(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> goldGoal.isFulfilled(player));
+            assertThrows(NullPointerException.class, () -> goldGoal.isFulfilled(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/goals/HealthGoalTest.java

@@ -52,11 +52,11 @@ class HealthGoalTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_checking_with_null_as_player() {
+        public void throws_NullPointerException_checking_with_null_as_player() {
             HealthGoal healthGoal = new HealthGoal(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> healthGoal.isFulfilled(player));
+            assertThrows(NullPointerException.class, () -> healthGoal.isFulfilled(player));
         }
 
         @Test

src/test/java/edu/ntnu/idatt2001/group_30/goals/InventoryGoalTest.java

@@ -58,11 +58,11 @@ class InventoryGoalTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_checking_with_null_as_player() {
+        public void throws_NullPointerException_checking_with_null_as_player() {
             InventoryGoal inventoryGoal = new InventoryGoal(getMandatoryInventory());
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> inventoryGoal.isFulfilled(player));
+            assertThrows(NullPointerException.class, () -> inventoryGoal.isFulfilled(player));
         }
     }
 

src/test/java/edu/ntnu/idatt2001/group_30/goals/ScoreGoalTest.java

@@ -51,11 +51,11 @@ class ScoreGoalTest {
         }
 
         @Test
-        public void throws_IllegalArgumentException_checking_with_null_as_player() {
+        public void throws_NullPointerException_checking_with_null_as_player() {
             ScoreGoal scoreGoal = new ScoreGoal(10);
             Player player = null;
 
-            assertThrows(IllegalArgumentException.class, () -> scoreGoal.isFulfilled(player));
+            assertThrows(NullPointerException.class, () -> scoreGoal.isFulfilled(player));
         }
 
     }