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

test(game): refine GameStateTest while-loop logic and assertions

Browse Source 
- Adjusted while-loop conditions to streamline test execution.
- Enhanced `canBePlayedOn` validation by passing `drawCount`.
- Added assertion to validate card count consistency during gameplay.
This commit is contained in:
Kieran Kihn
2025-11-17 22:11:14 +08:00
parent 423c237a39
commit 23cd94e656
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -115,10 +115,6 @@ TEST(game_state_test, game_state_test_2)
} }
while (true) { while (true) {
if (serverGameState.getCurrentPlayer()->getRemainingCardCount() == 0) {
break;
}
for (auto it = serverGameState.getCurrentPlayer()->getCards().begin();; it++) { for (auto it = serverGameState.getCurrentPlayer()->getCards().begin();; it++) {
const auto player = serverGameState.getCurrentPlayer(); const auto player = serverGameState.getCurrentPlayer();
auto prevCards = player->getCards(); auto prevCards = player->getCards();
@@ -140,7 +136,8 @@ TEST(game_state_test, game_state_test_2)
break; break;
} }
if (it->canBePlayedOn(serverGameState.getDiscardPile().getFront())) {
if (it->canBePlayedOn(serverGameState.getDiscardPile().getFront(), serverGameState.getDrawCount())) {
auto card = *it; auto card = *it;
size_t prevCount = player->getCards().count(card); size_t prevCount = player->getCards().count(card);
@@ -159,5 +156,11 @@ TEST(game_state_test, game_state_test_2)
break; break;
} }
} }
ASSERT_EQ(serverGameState.getCurrentPlayer()->getRemainingCardCount(), serverGameState.getCurrentPlayer()->getCards().size());
if (serverGameState.getCurrentPlayer()->getRemainingCardCount() == 0) {
break;
}
} }
} }