changed test

OGP1718-Worms/src/worms/model/Worm.java

@@ -102,17 +102,21 @@ public class Worm {
 
         if (!isValidMinRadius(minRadius))
             throw new IllegalArgumentException("Invalid min radius"); // TODO add decent exception msg
-        if (!canHaveAsMinRadius(radius))
+        this.minRadius = minRadius;
+
+        if (!canHaveAsRadius(radius))
             throw new IllegalArgumentException("Invalid radius");
 
         setRadius(radius);
-        this.minRadius = minRadius;
+
         setActionPoints(getMaxActionPoints());
 
         int validName = isValidName(name);
         if (validName != -1)
             throw new IllegalNameException(validName, name);
         this.name = name;
+
+        Tuple<Integer, Integer> test = null;
     }
 
     //===================================================================================
@@ -231,13 +235,13 @@ public class Worm {
     /**
      * check whether the given radius is a valid minimum radius for the worm
      *
-     * @param radius
+     * @param minRadius
      * 		  the radius to check
      * @return True if and only if the radius is a number and the radius is bigger then 0
      * 		   |result == ((!Double.isNaN(radius)) && (radius > 0))
      */
-    public static boolean isValidMinRadius(double radius) {
-        return !Double.isNaN(radius) && radius > 0;
+    public static boolean isValidMinRadius(double minRadius) {
+        return !Double.isNaN(minRadius) && minRadius > 0;
     }
 
     /**
@@ -252,7 +256,7 @@ public class Worm {
      * 		   |! canHaveAsMinRadius(radius)
      */
     public void setRadius(double radius) throws IllegalArgumentException {
-        if (!canHaveAsMinRadius(radius))
+        if (!canHaveAsRadius(radius))
             throw new IllegalArgumentException("Invalid radius");
 
         this.radius = radius;
@@ -290,8 +294,8 @@ public class Worm {
      * 	       (or equal) and the radius is a number
      * 		   |result == (radius >= this.minRadius && !Double.isNaN(radius))
      */
-    private boolean canHaveAsMinRadius(double radius) {
-        return radius >= getMinRadius() && !Double.isNaN(radius);
+    private boolean canHaveAsRadius(double radius) {
+        return !Double.isNaN(radius) && radius >= getMinRadius();
     }
 
     /**

OGP1718-Worms/tests/worms/model/WormTest.java

@@ -1,13 +1,13 @@
 package worms.model;
 
-import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.*;
+
 import static org.junit.jupiter.api.Assertions.*;
 import worms.util.Tuple;
 
 class WormTest {
 
-    private Worm worm;
+    /*private Worm worm;
 
     @BeforeEach
     void setUp() {
@@ -36,11 +36,6 @@ class WormTest {
     	assertEquals(2, worm.getRadius());
     }
 
-    /**@Test	//TODO mag niet voorkomen => exception (hoe doe je dat?)
-    void setRadius2() {
-    	worm.setRadius(0.02);
-    }*/
-    
     @Test
     void getMinimumRadius() {
     	assertEquals(0.25, worm.getMinRadius());
@@ -54,14 +49,6 @@ class WormTest {
     	//Moet dit round of ceil zijn? Ik dacht round, maar ben niet zeker
     }
 
-    /** @Test	TODO was een idee voor een testfunctie, maar setOrientation
-     * 			is private, dus misschien moet dit niet
-    void getMass2() {
-    	worm.setOrientation(Math.PI/2);
-    	assertEquals(Math.PI/2, worm.getOrientation());
-    	assertEquals(17241, worm.getMass());
-    }*/
-
     @Test
     void getActionPoints() {
     	assertEquals(4448.0, worm.getActionPoints());
@@ -98,25 +85,32 @@ class WormTest {
     void setName2() {
     	worm.setName("Jefke 'anssens");
     	assertEquals("Jefke 'anssens", worm.getName());
+    }*/
+
+    private static Worm worm;
+
+    @BeforeAll
+    static void setUp() {
+        worm = new Worm(Tuple.create(0.0, 0.0), 0, "Test", 1);
     }
 
     @Test
     void move() {
+        assertEquals("Test", worm.getName());
     }
 
     @Test
     void turn() {
     }
 
-    @Test
-    void jump() {
+    @RepeatedTest(10)
+    void jump(RepetitionInfo repetitionInfo) {
+        if (repetitionInfo.getCurrentRepetition() > 1){
+            assertThrows(IllegalStateException.class, () -> worm.jump());
+        }
+        else {
+            worm.jump();
         }
 
-    @Test
-    void jumpTime() {
-    }
-
-    @Test
-    void jumpStep() {
     }
 }