Reimplement get_all_possible_moves function

game/player.py

@@ -1,9 +1,11 @@
 from __future__ import annotations
 
-from typing import List, Optional, TYPE_CHECKING
+from itertools import combinations
+from typing import List, TYPE_CHECKING
 
 if TYPE_CHECKING:
     from game.card import Card
+    from game.table import Table
 
 
 class Player:
@@ -13,3 +15,11 @@ def __init__(self):
         self.hand: List[Card] = []
         self.player_id: int = Player._player_id
         Player._player_id += 1
+
+    def get_all_possible_moves(self, table: Table) -> [[Card]]:
+        possible_moves = []
+        for amount_of_cards in range(len(table.last_move()[0]) if table.last_move() else 1, len(self.hand) + 1):
+            for potential_move in combinations(self.hand, amount_of_cards):
+                if table.game.valid_move(list(potential_move)):
+                    possible_moves.append(list(potential_move))
+        return possible_moves

game/president.py

@@ -68,13 +68,13 @@ def on_move(self, agent: Agent, cards: List[Card]) -> Tuple[int, bool]:
             return -5, False  # TODO fix reward
 
         # Previous value should be lower
-        if self._valid_move(cards):
+        if self.valid_move(cards):
             self.table.do_move(agent, cards)
             return 10, False  # TODO fix reward
         else:
             return -10, False  # TODO fix reward
 
-    def _valid_move(self, cards: List[Card]) -> bool:
+    def valid_move(self, cards: List[Card]) -> bool:
         last_move: Tuple[List[Card], Agent] = self.table.last_move()
 
         # If multiple cards are played length should be at least the same.