kierankihn/uno-game
1
0
Fork 0
mirror of https://github.com/kierankihn/uno-game.git synced 2025-12-27 02:13:18 +08:00
Code Issues Packages Projects Releases Wiki Activity

test(game): simplify card play and player initialization in tests

Browse Source 
- Updated `HandCard::play` usage to accept a dereferenced card.
- Replaced sequential `addPlayer` calls with `init` method in `GameStateTest`.
This commit is contained in:
Kieran Kihn
2025-12-07 19:33:29 +08:00
parent d266f1c514
commit 9ae3b9cc89
2 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
test/unit/game/GameStateTest.cpp
View File

@@ -20,10 +20,7 @@ TEST(game_state_test, game_state_test_1)
UNO::GAME::ClientPlayerState playerState3("qkp", 100, false); UNO::GAME::ClientPlayerState playerState3("qkp", 100, false);
UNO::GAME::ClientPlayerState playerState4("lzh", 100, false); UNO::GAME::ClientPlayerState playerState4("lzh", 100, false);
clientGameState.addPlayer(playerState1); clientGameState.init({playerState1, playerState2, playerState3, playerState4}, {}, {}, 0, 3);
clientGameState.addPlayer(playerState2);
clientGameState.addPlayer(playerState3);
clientGameState.addPlayer(playerState4);
const auto &players = clientGameState.getPlayers(); const auto &players = clientGameState.getPlayers();
ASSERT_EQ(players[0].getName(), "pkq"); ASSERT_EQ(players[0].getName(), "pkq");

2
test/unit/game/PlayerTest.cpp
View File

@@ -25,7 +25,7 @@ TEST(player_test, player_test_1)
ASSERT_EQ(handCard.getCards().begin()->getColor(), UNO::GAME::CardColor::RED); ASSERT_EQ(handCard.getCards().begin()->getColor(), UNO::GAME::CardColor::RED);
ASSERT_EQ(handCard.getCards().begin()->getType(), UNO::GAME::CardType::NUM1); ASSERT_EQ(handCard.getCards().begin()->getType(), UNO::GAME::CardType::NUM1);
handCard.play(handCard.getCards().begin()); handCard.play(*handCard.getCards().begin());
ASSERT_EQ(handCard.getCards().begin()->getColor(), UNO::GAME::CardColor::RED); ASSERT_EQ(handCard.getCards().begin()->getColor(), UNO::GAME::CardColor::RED);
ASSERT_EQ(handCard.getCards().begin()->getType(), UNO::GAME::CardType::SKIP); ASSERT_EQ(handCard.getCards().begin()->getType(), UNO::GAME::CardType::SKIP);