Skip to content

Commit 040a3c5

Browse files
committed
Finished tests for road development card
1 parent aa0a098 commit 040a3c5

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/DevelopmentCard.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,10 @@ void RoadBuildingCard::playCard()
109109
}
110110

111111
void RoadBuildingCard::playCard(Coordinate start1, Coordinate end1, Coordinate start2, Coordinate end2){
112-
if (!getOwner()->getBoard()->PlaceRoad(start1, end1, *getOwner())){
112+
if (!(getOwner()->getBoard()->PlaceRoad(start1, end1, *getOwner()))){
113113
throw std::invalid_argument("The first road passed was not valid, no roads placed");
114114
}
115-
if (!getOwner()->getBoard()->PlaceRoad(start2, end2, *getOwner())){
115+
if ((!getOwner()->getBoard()->PlaceRoad(start2, end2, *getOwner()))){
116116
throw std::invalid_argument("The second road passed was not valid, only the first road was placed");
117117
}
118118
}

tests/test_DevelopmentCards.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ TEST(RoadBuildingCard_good){
1717
GameBoard * test_board = new GameBoard(std::move(players));
1818
test_player.setBoard(test_board);
1919

20-
20+
test_board->PlaceSettlement(Coordinate(0,0), test_player);
2121

2222
RoadBuildingCard test_card(&test_player);
2323

@@ -28,7 +28,6 @@ TEST(RoadBuildingCard_good){
2828
std::cout << e.what();
2929
CHECK(false);
3030
}
31-
3231
}
3332

3433
TEST(RoadBuildingCard_bad){
@@ -38,6 +37,8 @@ TEST(RoadBuildingCard_bad){
3837
GameBoard * test_board = new GameBoard(std::move(players));
3938
test_player.setBoard(test_board);
4039

40+
test_board->PlaceSettlement(Coordinate(0,0), test_player);
41+
4142
RoadBuildingCard test_card(&test_player);
4243

4344
try{

0 commit comments

Comments
 (0)