Kata refactoring

@@ -1,7 +1,7 @@

 plugins {

     id 'java'

     id "jacoco"

-    id 'info.solidsoft.pitest' version '1.7.4'

+    id 'info.solidsoft.pitest' version '1.15.0'

 }

 group = 're.forestier.edu'

@@ -15,7 +15,7 @@ dependencies {

     testImplementation platform('org.junit:junit-bom:5.10.0')

     testImplementation 'org.junit.jupiter:junit-jupiter-api:5.3.1'

     testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine:5.3.1'

-    testImplementation 'org.pitest:pitest-junit5-plugin:0.12'

+    testImplementation 'org.pitest:pitest-junit5-plugin:1.1.1'

     testImplementation("com.approvaltests:approvaltests:15.6.0")

     implementation 'org.hamcrest:hamcrest:2.2'

     implementation("org.freemarker:freemarker:2.3.30")

@@ -1,6 +1,6 @@

 #Wed Jul 24 15:43:23 CEST 2024

 distributionBase=GRADLE_USER_HOME

 distributionPath=wrapper/dists

-distributionUrl=https\://services.gradle.org/distributions/gradle-8.5-bin.zip

+distributionUrl=https\://services.gradle.org/distributions/gradle-8.8-bin.zip

 zipStoreBase=GRADLE_USER_HOME

 zipStorePath=wrapper/dists

@@ -10,6 +10,7 @@ import static org.junit.jupiter.api.Assertions.fail;

 import static org.junit.jupiter.api.Assertions.assertThrows;

 import static org.junit.jupiter.api.Assertions.assertEquals;

 import static org.junit.jupiter.api.Assertions.assertNotNull;

+import static org.junit.jupiter.api.Assertions.assertNotEquals;

 import org.junit.jupiter.api.BeforeEach;

 import org.junit.jupiter.api.AfterEach;

@@ -60,6 +61,8 @@ public class UnitTests {

         p.removeMoney(50);

         assertThat(p.money, is(50));

         assertThat(p.money, not(150));

+        p.removeMoney(50);

+        assertThat(p.money, is(0));

     }

     @Test

@@ -93,6 +96,23 @@ public class UnitTests {

         assertThat(p.retrieveLevel(),is(4));

         b = UpdatePlayer.addXp(p,100);

         assertThat(p.retrieveLevel(),is(5));

+

+        p = new player("Florian", "Grognak le barbare", "ADVENTURER", 100, new ArrayList<>());

+        int tailleinv = p.inventory.size();

+        b = UpdatePlayer.addXp(p,10);

+        int atk = p.abilities.get("ATK");

+        assertNotEquals(p.inventory.size(),tailleinv);

+        assertThat(p.retrieveLevel(),is(2));

+        assertThat(b,is(true));

+        b = UpdatePlayer.addXp(p,17);

+        assertThat(p.retrieveLevel(),is(3));

+        b = UpdatePlayer.addXp(p,30);

+        assertThat(p.retrieveLevel(),is(4));

+        b = UpdatePlayer.addXp(p,54);

+        assertThat(p.retrieveLevel(),is(5));

+        b = UpdatePlayer.addXp(p,1);

+        assertThat(b,is(false));

+        assertNotEquals(p.abilities.get("ATK"),atk);

     }

     @Test

@@ -121,6 +141,12 @@ public class UnitTests {

         UpdatePlayer.majFinDeTour(p);

         assertThat(p.currenthealthpoints,is(2));

+        p = new player("Florian", "Grognak le barbare", "DWARF", 100, new ArrayList<>());

+        p.currenthealthpoints = 5;

+        p.healthpoints = 10;

+        UpdatePlayer.majFinDeTour(p);

+        assertThat(p.currenthealthpoints,is(5));

+

         ArrayList<String> inv = new ArrayList<String>();

         inv.add("Holy Elixir");

         p = new player("Florian", "Grognak le barbare", "DWARF", 100, inv);

@@ -135,6 +161,13 @@ public class UnitTests {

         UpdatePlayer.majFinDeTour(p);

         assertThat(p.currenthealthpoints,is(2));

+        p = new player("Florian", "Grognak le barbare", "ADVENTURER", 100, new ArrayList<>());

+        p.currenthealthpoints = 1;

+        p.healthpoints = 10;

+        UpdatePlayer.addXp(p,28);

+        UpdatePlayer.majFinDeTour(p);

+        assertThat(p.currenthealthpoints,is(3));

+

         inv.add("Magic Bow");

         p = new player("Florian", "Grognak le barbare", "ARCHER", 100, inv);

         p.currenthealthpoints = 1;

@@ -146,6 +179,12 @@ public class UnitTests {

         p.healthpoints = 40;

         UpdatePlayer.majFinDeTour(p);

         assertThat(p.currenthealthpoints,is(17 + 17/8-1));

+

+        p = new player("Florian", "Grognak le barbare", "ADVENTURER", 100, new ArrayList<>());

+        p.currenthealthpoints = 20;

+        p.healthpoints = 40;

+        UpdatePlayer.majFinDeTour(p);

+        assertEquals(p.currenthealthpoints,20);

     }

     @Test