Kata refactoring

@@ -41,7 +41,7 @@

 </td>

 <td>

 <div class="infoBox" id="duration">

-<div class="counter">0.221s</div>

+<div class="counter">0.163s</div>

 <p>duration</p>

 </div>

 </td>

@@ -76,7 +76,7 @@

 </thead>

 <tr>

 <td class="success">testAffichageBase()</td>

-<td class="success">0.221s</td>

+<td class="success">0.163s</td>

 <td class="success">passed</td>

 </tr>

 </table>

@@ -89,7 +89,7 @@

 <input id="line-wrapping-toggle" type="checkbox" autocomplete="off"/>

 </label>

 </div>Generated by 

-<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 09:16:49</p>

+<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 11:48:15</p>

 </div>

 </div>

 </body>

@@ -23,7 +23,7 @@

 <tr>

 <td>

 <div class="infoBox" id="tests">

-<div class="counter">10</div>

+<div class="counter">12</div>

 <p>tests</p>

 </div>

 </td>

@@ -82,21 +82,39 @@

 <td class="success">passed</td>

 </tr>

 <tr>

+<td class="success">Item and player</td>

+<td class="success">testBuySell()</td>

+<td class="success">0.002s</td>

+<td class="success">passed</td>

+</tr>

+<tr>

 <td class="success">Heal and hurt</td>

 <td class="success">testHealAndHurt()</td>

-<td class="success">0.002s</td>

+<td class="success">0.001s</td>

+<td class="success">passed</td>

+</tr>

+<tr>

+<td class="success">Item tests</td>

+<td class="success">testItem()</td>

+<td class="success">0.010s</td>

 <td class="success">passed</td>

 </tr>

 <tr>

 <td class="success">Main</td>

 <td class="success">testMain()</td>

-<td class="success">0.008s</td>

+<td class="success">0.002s</td>

 <td class="success">passed</td>

 </tr>

 <tr>

 <td class="success">majDeFinDeTour</td>

 <td class="success">testMajFinTour()</td>

-<td class="success">0.004s</td>

+<td class="success">0.003s</td>

+<td class="success">passed</td>

+</tr>

+<tr>

+<td class="success">Construcor Manager</td>

+<td class="success">testManager()</td>

+<td class="success">0.001s</td>

 <td class="success">passed</td>

 </tr>

 <tr>

@@ -108,7 +126,7 @@

 <tr>

 <td class="success">Constructor Player</td>

 <td class="success">testPlayerConstructor()</td>

-<td class="success">0.002s</td>

+<td class="success">0.001s</td>

 <td class="success">passed</td>

 </tr>

 <tr>

@@ -120,19 +138,13 @@

 <tr>

 <td class="success">removeMoney</td>

 <td class="success">testRemoveMoney()</td>

-<td class="success">0.009s</td>

+<td class="success">0.008s</td>

 <td class="success">passed</td>

 </tr>

 <tr>

 <td class="success">ex &amp; lvl</td>

 <td class="success">testRetrieveLevel()</td>

-<td class="success">0.005s</td>

-<td class="success">passed</td>

-</tr>

-<tr>

-<td class="success">Construcor UpdatePlayer</td>

-<td class="success">testUpdatePlayer()</td>

-<td class="success">0.001s</td>

+<td class="success">0.003s</td>

 <td class="success">passed</td>

 </tr>

 </table>

@@ -145,7 +157,7 @@

 <input id="line-wrapping-toggle" type="checkbox" autocomplete="off"/>

 </label>

 </div>Generated by 

-<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 09:16:49</p>

+<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 11:48:15</p>

 </div>

 </div>

 </body>

@@ -20,7 +20,7 @@

 <tr>

 <td>

 <div class="infoBox" id="tests">

-<div class="counter">11</div>

+<div class="counter">13</div>

 <p>tests</p>

 </div>

 </td>

@@ -38,7 +38,7 @@

 </td>

 <td>

 <div class="infoBox" id="duration">

-<div class="counter">0.258s</div>

+<div class="counter">0.200s</div>

 <p>duration</p>

 </div>

 </td>

@@ -82,10 +82,10 @@

 <td class="success">

 <a href="packages/re.forestier.edu.html">re.forestier.edu</a>

 </td>

-<td>11</td>

+<td>13</td>

 <td>0</td>

 <td>0</td>

-<td>0.258s</td>

+<td>0.200s</td>

 <td class="success">100%</td>

 </tr>

 </tbody>

@@ -112,14 +112,14 @@

 <td>1</td>

 <td>0</td>

 <td>0</td>

-<td>0.221s</td>

+<td>0.163s</td>

 <td class="success">100%</td>

 </tr>

 <tr>

 <td class="success">

 <a href="classes/re.forestier.edu.UnitTests.html">re.forestier.edu.UnitTests</a>

 </td>

-<td>10</td>

+<td>12</td>

 <td>0</td>

 <td>0</td>

 <td>0.037s</td>

@@ -136,7 +136,7 @@

 <input id="line-wrapping-toggle" type="checkbox" autocomplete="off"/>

 </label>

 </div>Generated by 

-<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 09:16:49</p>

+<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 11:48:15</p>

 </div>

 </div>

 </body>

@@ -22,7 +22,7 @@

 <tr>

 <td>

 <div class="infoBox" id="tests">

-<div class="counter">11</div>

+<div class="counter">13</div>

 <p>tests</p>

 </div>

 </td>

@@ -40,7 +40,7 @@

 </td>

 <td>

 <div class="infoBox" id="duration">

-<div class="counter">0.258s</div>

+<div class="counter">0.200s</div>

 <p>duration</p>

 </div>

 </td>

@@ -83,14 +83,14 @@

 <td>1</td>

 <td>0</td>

 <td>0</td>

-<td>0.221s</td>

+<td>0.163s</td>

 <td class="success">100%</td>

 </tr>

 <tr>

 <td class="success">

 <a href="../classes/re.forestier.edu.UnitTests.html">UnitTests</a>

 </td>

-<td>10</td>

+<td>12</td>

 <td>0</td>

 <td>0</td>

 <td>0.037s</td>

@@ -106,7 +106,7 @@

 <input id="line-wrapping-toggle" type="checkbox" autocomplete="off"/>

 </label>

 </div>Generated by 

-<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 09:16:49</p>

+<a href="http://www.gradle.org">Gradle 8.8</a> at 13 déc. 2024 à 11:48:15</p>

 </div>

 </div>

 </body>

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

 <?xml version="1.0" encoding="UTF-8"?>

-<testsuite name="re.forestier.edu.GlobalTest" tests="1" skipped="0" failures="0" errors="0" timestamp="2024-12-13T08:16:49" hostname="clement-G551JM" time="0.221">

+<testsuite name="re.forestier.edu.GlobalTest" tests="1" skipped="0" failures="0" errors="0" timestamp="2024-12-13T10:48:15" hostname="clement-G551JM" time="0.163">

   <properties/>

-  <testcase name="testAffichageBase()" classname="re.forestier.edu.GlobalTest" time="0.221"/>

+  <testcase name="testAffichageBase()" classname="re.forestier.edu.GlobalTest" time="0.163"/>

   <system-out><![CDATA[]]></system-out>

   <system-err><![CDATA[]]></system-err>

 </testsuite>

@@ -1,14 +1,16 @@

 <?xml version="1.0" encoding="UTF-8"?>

-<testsuite name="re.forestier.edu.UnitTests" tests="10" skipped="0" failures="0" errors="0" timestamp="2024-12-13T08:16:49" hostname="clement-G551JM" time="0.047">

+<testsuite name="re.forestier.edu.UnitTests" tests="12" skipped="0" failures="0" errors="0" timestamp="2024-12-13T10:48:15" hostname="clement-G551JM" time="0.05">

   <properties/>

-  <testcase name="Main" classname="re.forestier.edu.UnitTests" time="0.008"/>

-  <testcase name="removeMoney" classname="re.forestier.edu.UnitTests" time="0.009"/>

-  <testcase name="ex &amp; lvl" classname="re.forestier.edu.UnitTests" time="0.005"/>

+  <testcase name="Item tests" classname="re.forestier.edu.UnitTests" time="0.01"/>

+  <testcase name="Main" classname="re.forestier.edu.UnitTests" time="0.002"/>

+  <testcase name="removeMoney" classname="re.forestier.edu.UnitTests" time="0.008"/>

+  <testcase name="ex &amp; lvl" classname="re.forestier.edu.UnitTests" time="0.003"/>

   <testcase name="addMoney" classname="re.forestier.edu.UnitTests" time="0.001"/>

-  <testcase name="Heal and hurt" classname="re.forestier.edu.UnitTests" time="0.002"/>

-  <testcase name="Construcor UpdatePlayer" classname="re.forestier.edu.UnitTests" time="0.001"/>

-  <testcase name="majDeFinDeTour" classname="re.forestier.edu.UnitTests" time="0.004"/>

-  <testcase name="Constructor Player" classname="re.forestier.edu.UnitTests" time="0.002"/>

+  <testcase name="Heal and hurt" classname="re.forestier.edu.UnitTests" time="0.001"/>

+  <testcase name="Item and player" classname="re.forestier.edu.UnitTests" time="0.002"/>

+  <testcase name="Construcor Manager" classname="re.forestier.edu.UnitTests" time="0.001"/>

+  <testcase name="majDeFinDeTour" classname="re.forestier.edu.UnitTests" time="0.003"/>

+  <testcase name="Constructor Player" classname="re.forestier.edu.UnitTests" time="0.001"/>

   <testcase name="Natural Tests" classname="re.forestier.edu.UnitTests" time="0.004"/>

   <testcase name="Sample test" classname="re.forestier.edu.UnitTests" time="0.001"/>

   <system-out><![CDATA[]]></system-out>

@@ -8,7 +8,7 @@ import java.util.ArrayList;

 public class Main {

     public static void main(String[] args) {

-        Player firstPlayer = new Player("Florian", "Ruzberg de Rivehaute", Jobs.DWARF, 200, new ArrayList<>());

+        Player firstPlayer = new Player("Florian", "Ruzberg de Rivehaute", Jobs.DWARF, 200, new ArrayList<>(),10);

         firstPlayer.addMoney(400);

         firstPlayer.addXp(15);

@@ -2,7 +2,9 @@ package re.forestier.edu.rpg;

 public class Affichage {

-    public static void afficherJoueur(Player player) {

+    public static void afficherJoueur(Player player) 

+    {

         System.out.println(player.toString());

     }

+

 }

@@ -27,16 +27,18 @@ public class Player {

     private Natural maxHealthPoint;

     private Natural currentHealthPoints;

     private Natural xp;

+    private Natural maxWeight;

+    private Natural weight;

     private final HashMap<Ability, Integer> abilities; //Ability = stat

                                                        //SHOULDDO : have another hashmap to have contextual modifier (items,...)

     private ArrayList<Item> inventory;

-    public Player(String playerName, String avatar_name, Jobs avatarClass, int money, ArrayList<Item> inventory) {

-        this(playerName,avatar_name,avatarClass,money,inventory,Player.defaultMaxHp.toInt());

+    public Player(String playerName, String avatar_name, Jobs avatarClass, int money, ArrayList<Item> inventory, int maxWeight) {

+        this(playerName,avatar_name,avatarClass,money,inventory,Player.defaultMaxHp.toInt(),maxWeight);

     }

-    public Player(String playerName, String avatar_name, Jobs avatarClass, int money, ArrayList<Item> inventory, int maxHp)

+    public Player(String playerName, String avatar_name, Jobs avatarClass, int money, ArrayList<Item> inventory, int maxHp, int maxWeight)

     {

         this.playerName = playerName;

         this.avatarName = avatar_name;

@@ -49,7 +51,16 @@ public class Player {

         avatarClass.getAbilityPerLevel().forEach((ability,value)->

             {this.abilities.put(ability,value[0]);}

         );

-

+        this.maxWeight = Natural.valueOf(maxWeight);

+        this.weight = Natural.valueOf(0);

+        for(Item i : inventory)

+        {

+            this.weight.add(i.getWeight());

+        }

+        if(this.weight.compareTo(this.maxWeight) == -1)

+        {

+            throw  new UnsupportedOperationException("The player have too much to carry in theyr inventory");

+        }

         this.maxHealthPoint = Natural.valueOf(maxHp);

         this.currentHealthPoints = Natural.valueOf(maxHp);

     }

@@ -111,6 +122,30 @@ public class Player {

         }

     }

+    public void addItem(Item i)

+    {

+        Natural w = (Natural)this.weight.clone();

+        w.add(i.getWeight());

+        if(w.compareTo(this.maxWeight) == -1)

+        {

+            throw new UnsupportedOperationException("Player can't carry more.");

+

+        }

+

+        this.weight.add(i.getWeight());

+        this.inventory.add(i);

+    }

+

+    public void removeItem(Item i)

+    {

+        if(!this.inventory.contains(i))

+        {

+            throw new UnsupportedOperationException("Can't remove what player don't have");

+        }

+        this.weight.substract(i.getWeight());

+        this.inventory.remove(i);

+    }

+

     public void buy(Item i)

     {

         if(this.money.compareTo(i.getValue()) == 1)

@@ -118,7 +153,16 @@ public class Player {

             throw new UnsupportedOperationException("Player doesn't have enough money.");

         }

+        Natural w = (Natural)this.weight.clone();

+        w.add(i.getWeight());

+        if(w.compareTo(this.maxWeight) == -1)

+        {

+            throw new UnsupportedOperationException("Player can't carry more.");

+

+        }

+

         this.removeMoney(i.getValue().toInt());

+        this.weight.add(i.getWeight());

         this.inventory.add(i);

     }

@@ -130,6 +174,7 @@ public class Player {

         }

         this.inventory.remove(i);

+        this.weight.substract(i.getWeight());

         this.addMoney(i.getValue().toInt());

     }

@@ -5,6 +5,7 @@ import org.junit.jupiter.api.Test;

 import re.forestier.edu.rpg.Affichage;

 import re.forestier.edu.rpg.Manager;

 import re.forestier.edu.rpg.Player;

+import re.forestier.edu.rpg.Item;

 import re.forestier.edu.rpg.Jobs;

 import java.util.ArrayList;

@@ -43,11 +44,12 @@ public class GlobalTest {

         System.setErr(originalErr);

     }

+

     @Test

     void testAffichageBase() {

-        Player player = new Player("Florian", "Gnognak le Barbare", Jobs.ADVENTURER, 200, new ArrayList<>());

+        Player player = new Player("Florian", "Gnognak le Barbare", Jobs.ADVENTURER, 200, new ArrayList<Item>(),10);

         player.addXp(20);

-        player.inventory = new ArrayList<>();

+        player.removeItem(player.getInventory().get(0));

         Affichage.afficherJoueur(player);

         verify(outContent );

     }

@@ -38,7 +38,7 @@ public class UnitTests {

     @Test

     @DisplayName("Sample test")

     void testPlayerName() {

-        Player Player = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player Player = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         assertThat(Player.getPlayerName(), is("Florian"));

     }

@@ -47,11 +47,11 @@ public class UnitTests {

     @DisplayName("Constructor Player")

     void testPlayerConstructor()

     {

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         assertThat(p.getPlayerName(), is("Florian"));

-        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>());

+        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>(),10);

         assertThat(p.getPlayerName(), is("Florian"));

-        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, new ArrayList<>());

+        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, new ArrayList<>(),10);

         assertThat(p.getPlayerName(), is("Florian"));

         assertThat(p.getAvatarName(), is("Grognak le barbare"));

     }

@@ -59,7 +59,7 @@ public class UnitTests {

     @Test

     @DisplayName("removeMoney")

     void testRemoveMoney() {

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         assertThrows(IllegalArgumentException.class, () -> p.removeMoney(200));

         p.removeMoney(50);

@@ -73,7 +73,7 @@ public class UnitTests {

     @DisplayName("Heal and hurt")

     void testHealAndHurt()

     {

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         assertThrows(IllegalArgumentException.class, () -> p.heal(-1));

         assertThrows(IllegalArgumentException.class, () -> p.hurt(-1));

@@ -83,7 +83,7 @@ public class UnitTests {

     @Test

     @DisplayName("addMoney")

     void testAddMoney() {

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         p.addMoney(50);

         assertThat(p.getMoney(), is(150));

     }

@@ -98,7 +98,7 @@ public class UnitTests {

     @Test

     @DisplayName("ex & lvl")

     void testRetrieveLevel() {

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

         boolean b;

         p.addXp(5);

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

@@ -112,13 +112,13 @@ public class UnitTests {

         p.addXp(100);

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

-        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>());

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

+        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

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

         p.addXp(10);

         /*fail(p.avatarClass.getAbilityPerLevel().toString());

         fail(p.getAbilities().toString());*/

         int atk = p.getAbilities().get(Ability.ATK);

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

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

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

         p.addXp(17);

         assertThat(p.getLevel(),is(3));

@@ -134,7 +134,7 @@ public class UnitTests {

     @DisplayName("majDeFinDeTour")

     void testMajFinTour() {

         assertEquals(Ability.ATK.toString(),"ATK");

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

+        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10,10);

         p.hurt(10);

         Manager.majFinDeTour(p);

         assertEquals("Le joueur est KO !", outContent.toString().trim());

@@ -150,45 +150,45 @@ public class UnitTests {

         Manager.majFinDeTour(p);

         assertThat(p.getCurrentHealthPoints(),is(p.getMaxHealthPoints()));

-        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>(),10);

+        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>(),10,10);

         p.hurt(9);

         Manager.majFinDeTour(p);

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

-        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>(),10);

+        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, new ArrayList<>(),10,10);

         p.hurt(5);

         Manager.majFinDeTour(p);

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

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

-        inv.add("Holy Elixir");

-        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, inv,10);

+        ArrayList<Item> inv = new ArrayList<Item>();

+        inv.add(new Item("Holy Elixir","",Natural.valueOf(0),Natural.valueOf(0)))   ;

+        p = new Player("Florian", "Grognak le barbare", Jobs.DWARF, 100, inv,10,10);

         p.hurt(9);

         Manager.majFinDeTour(p);

         assertThat(p.getCurrentHealthPoints(),is(3));

-        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, new ArrayList<>(),10);

+        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, new ArrayList<>(),10,10);

         p.hurt(9);

         Manager.majFinDeTour(p);

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

-        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

+        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10,10);

         p.hurt(9);

         p.addXp(28);

         Manager.majFinDeTour(p);

         assertThat(p.getCurrentHealthPoints(),is(3));

-        inv.add("Magic Bow");

-        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, inv,10);

+        inv.add(new Item("Magic Bow","",Natural.valueOf(0),Natural.valueOf(0)));

+        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, inv,10,10);

         p.hurt(9);;

         Manager.majFinDeTour(p);

         assertThat(p.getCurrentHealthPoints(),is(2)); //Suite a discussion avec le client, le "Magic bow" ne peut que soigner

-        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, inv,40);

+        p = new Player("Florian", "Grognak le barbare", Jobs.ARCHER, 100, inv,40,10);

         p.hurt(24);

         Manager.majFinDeTour(p);

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

-        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),40);

+        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),40,10);

         p.hurt(20);

         Manager.majFinDeTour(p);

         assertEquals(p.getCurrentHealthPoints(),20);

@@ -217,18 +217,18 @@ public class UnitTests {

     @DisplayName("Item and player")

     void testBuySell()

     {   

-        Item i = new Item("1kg de plume","Pese plus lourd que le plomb de part le poids du destin de ces pauvres oiseaux", Natural.valueOf(1),Natural.valueOf(2));

-        Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10);

+        final Item i = new Item("1kg de plume","Pese plus lourd que le plomb de part le poids du destin de ces pauvres oiseaux", Natural.valueOf(1),Natural.valueOf(2));

+        final Player p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 100, new ArrayList<>(),10,10);

         p.buy(i);

-        assertEquals(p.getInventory().size(),1);

-        assertEquals(p.getMoney(),Natural.valueOf(98));

+        assertTrue(p.getInventory().contains(i));

+        assertEquals(p.getMoney(),98);

         p.sell(i);

         assertEquals(p.getInventory().size(),0);

-        assertEquals(p.getMoney(),Natural.valueOf(100));

-        assertThrows(UnsupportedOperationExceptionn.class, () -> p.sell(i));

-        p = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 0, new ArrayList<>(),10);

-        assertThrows(UnsupportedOperationExceptionn.class, () -> p.buy(i));

-        assertThrows(UnsupportedOperationExceptionn.class, () -> p.sell(i));

+        assertEquals(p.getMoney(),100);

+        assertThrows(UnsupportedOperationException.class, () -> p.sell(i));

+        Player p2 = new Player("Florian", "Grognak le barbare", Jobs.ADVENTURER, 0, new ArrayList<>(),10,10);

+        assertThrows(UnsupportedOperationException.class, () -> p2.buy(i));

+        assertThrows(UnsupportedOperationException.class, () -> p2.sell(i));

     }