Skip to content

Commit

Permalink
add turn order to player summary
Browse files Browse the repository at this point in the history
fix test
  • Loading branch information
evanofslack committed Feb 5, 2023
1 parent a228465 commit e4b9631
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 5 deletions.
3 changes: 2 additions & 1 deletion pyminion/game.py
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ def summerize_game(self) -> GameResult:
player_summaries = []
winners = self.get_winners()

for player in self.players:
for order, player in enumerate(self.players):

# player won
if player in winners and len(winners) == 1:
Expand All @@ -301,6 +301,7 @@ def summerize_game(self) -> GameResult:
score=player.get_victory_points(),
turns=player.turns,
shuffles=player.shuffles,
turn_order=order + 1,
deck=DeckCounter(player.get_all_cards()),
)
player_summaries.append(summary)
Expand Down
18 changes: 17 additions & 1 deletion pyminion/result.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,17 +31,33 @@ class PlayerSummary:
score: int
turns: int
shuffles: int
turn_order: int
deck: "DeckCounter"

def __repr__(self):

order_format = "None"
if self.turn_order == 1:
order_format = "1st"

elif self.turn_order == 2:
order_format = "2nd"

elif self.turn_order == 3:
order_format = "3rd"

elif self.turn_order == 4:
order_format = "4th"

player = f"Player: {self.player.player_id}"
result = f"Result: {self.result.name}"
score = f"Score: {self.score}"
turns = f"Turns: {self.turns}"
shuffles = f"Shuffles: {self.shuffles}"
order = f"Turn Order: {order_format}"
deck = f"Deck: {self.deck}"

return f"{player}\n{result}\n{score}\n{turns}\n{shuffles}\n{deck}"
return f"{player}\n{result}\n{score}\n{turns}\n{shuffles}\n{order}\n{deck}"


@dataclass
Expand Down
3 changes: 0 additions & 3 deletions tests/test_players/test_bots/test_bot_game.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ def test_game_single_player_play(bm_bot: BigMoney):
players=[bm_bot],
expansions=[base_set],
kingdom_cards=[smithy],
use_logger=False,
)
game.play()
assert game.get_winners() == [bm_bot]
Expand All @@ -20,7 +19,6 @@ def test_game_2_player_play(bm_bot: BigMoney):
players=[bm_bot, bm_bot],
expansions=[base_set],
kingdom_cards=[smithy],
use_logger=False,
)
game.play()
assert len(game.get_winners()) >= 1
Expand All @@ -32,7 +30,6 @@ def test_game_2_player_with_actions():
players=[bot, bot],
expansions=[base_set],
kingdom_cards=[smithy],
use_logger=False,
)
game.play()
assert len(game.get_winners()) >= 1

0 comments on commit e4b9631

Please sign in to comment.