Skip to content

Commit 58f059c

Browse files
committed
Fixed memory leak in tests
1 parent 8b764d6 commit 58f059c

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

tests/test_DevelopmentCards.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ TEST(RoadBuildingCard_good){
1414
std::vector<std::unique_ptr<Player>> players {};
1515
players.emplace_back(new Player("tester"));
1616
Player& test_player = *players[0];
17-
GameBoard * test_board = new GameBoard(std::move(players));
18-
test_player.setBoard(test_board);
17+
GameBoard test_board(std::move(players));
18+
test_player.setBoard(&test_board);
1919

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

2222
RoadBuildingCard test_card(&test_player);
2323

@@ -34,10 +34,10 @@ TEST(RoadBuildingCard_bad){
3434
std::vector<std::unique_ptr<Player>> players {};
3535
players.emplace_back(new Player("tester"));
3636
Player& test_player = *players[0];
37-
GameBoard * test_board = new GameBoard(std::move(players));
38-
test_player.setBoard(test_board);
37+
GameBoard test_board(std::move(players));
38+
test_player.setBoard(&test_board);
3939

40-
test_board->PlaceSettlement(Coordinate(0,0), test_player);
40+
test_board.PlaceSettlement(Coordinate(0,0), test_player);
4141

4242
RoadBuildingCard test_card(&test_player);
4343

0 commit comments

Comments
 (0)