Aleandro/Progetto-ingegneria-del-software
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: BoardTest

Browse Source
This commit is contained in:
rubenpirreram
2026-05-02 17:31:46 +02:00
parent c95e529caa
commit 6d5a52f2a4
1 changed files with 14 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/test/java/it/polimi/ingsw/gc14/Model/GamePackage/BoardTest.java
+14 -16
View File
@@ -125,11 +125,11 @@ class BoardTest {
void removeUpperTribeCard() { void removeUpperTribeCard() {
Board bd = new Board(3); Board bd = new Board(3);
List<TribeCard> before = bd.upperListTribe; List<TribeCard> before = new ArrayList<>( bd.upperListTribe);
TribeCard cardToRemove = before.get(2); TribeCard cardToRemove = before.get(0);
before.remove(2); assertEquals(cardToRemove, before.remove(0));
bd.removeUpperTribeCard(cardToRemove); assertTrue( bd.removeUpperTribeCard(cardToRemove));
assertEquals(before, bd.upperListTribe); assertEquals(before, bd.upperListTribe);
} }
@@ -139,11 +139,11 @@ class BoardTest {
void removeLowerTribeCard() { void removeLowerTribeCard() {
Board bd = new Board(3); Board bd = new Board(3);
List<TribeCard> before = bd.lowerListTribe; List<TribeCard> before = new ArrayList<>(bd.lowerListTribe);
TribeCard cardToRemove = before.get(2); TribeCard cardToRemove = before.get(0);
before.remove(2); assertEquals(cardToRemove, before.remove(0));
bd.removeLowerTribeCard(cardToRemove); assertTrue( bd.removeLowerTribeCard(cardToRemove));
assertEquals(before, bd.lowerListTribe); assertEquals(before, bd.lowerListTribe);
} }
@@ -153,11 +153,11 @@ class BoardTest {
void removeUpperBuildingCard() { void removeUpperBuildingCard() {
Board bd = new Board(3); Board bd = new Board(3);
List<BuildingCard> before = bd.upperListBuilding; List<BuildingCard> before = new ArrayList<>(bd.upperListBuilding);
BuildingCard cardToRemove = before.get(0); BuildingCard cardToRemove = before.get(0);
before.remove(0); assertEquals(cardToRemove, before.remove(0));
bd.removeUpperBuildingCard(cardToRemove); assertTrue( bd.removeUpperBuildingCard(cardToRemove));
assertEquals(before, bd.upperListBuilding); assertEquals(before, bd.upperListBuilding);
} }
@@ -171,12 +171,10 @@ class BoardTest {
bd.nextRound(); // Skip to era 2 bd.nextRound(); // Skip to era 2
} }
List<BuildingCard> before = bd.lowerListBuilding; List<BuildingCard> before = new ArrayList<>( bd.lowerListBuilding);
BuildingCard cardToRemove = before.get(0); BuildingCard cardToRemove = before.get(0);
assertEquals(cardToRemove, before.remove(0));
before.remove(0); assertTrue(bd.removeLowerBuildingCard(cardToRemove));
bd.removeLowerBuildingCard(cardToRemove);
assertEquals(before, bd.lowerListBuilding); assertEquals(before, bd.lowerListBuilding);
} }