UndefinedOperationError for game.player.{actions,infosets} on non-tree

Author: rahulsavani

src/pygambit/player.pxi

@@ -228,15 +228,34 @@ class Player:
            previously, information sets were (expensively) re-sorted after every change
            to the game tree.
 
+        Raises
+        ------
+        UndefinedOperationError
+            If the game does not have a tree representation.
+
         See also
         --------
         Game.sort_infosets
         """
+        if not self.game.is_tree:
+            raise UndefinedOperationError(
+                "Operation only defined for games with a tree representation"
+            )
         return PlayerInfosets.wrap(self.player)
 
     @property
     def actions(self) -> PlayerActions:
-        """Returns the set of actions available to the player at some information set."""
+        """Returns the set of actions available to the player at some information set.
+
+        Raises
+        ------
+        UndefinedOperationError
+            If the game does not have a tree representation.
+        """
+        if not self.game.is_tree:
+            raise UndefinedOperationError(
+                "Operation only defined for games with a tree representation"
+            )
         return PlayerActions.wrap(self)
 
     @property

tests/test_strategic.py

@@ -11,12 +11,24 @@ def test_strategic_game_actions():
         _ = game.actions
 
 
+def test_strategic_game_player_actions():
+    game = gbt.Game.new_table([2, 2])
+    with pytest.raises(gbt.UndefinedOperationError):
+        _ = game.players[0].actions
+
+
 def test_strategic_game_infosets():
     game = gbt.Game.new_table([2, 2])
     with pytest.raises(gbt.UndefinedOperationError):
         _ = game.infosets
 
 
+def test_strategic_game_player_infosets():
+    game = gbt.Game.new_table([2, 2])
+    with pytest.raises(gbt.UndefinedOperationError):
+        _ = game.players[0].infosets
+
+
 def test_strategic_game_root():
     game = gbt.Game.new_table([2, 2])
     with pytest.raises(gbt.UndefinedOperationError):