Closed jkomoros closed 7 years ago
Actually JK, Delegate should be renamed GameManager. It should have all of the current delegate's methods, as well as Chest(), Storage(), NewGame(), and lose the Game bit
Should game manager be the thing that applies moves to the canonical games?
Should NewGame be on manager? Currently newGame just creates a game object with things init'd. But the NewGame() package methods return a new, set up game. Most of the overhead is in new manager.
... Yeah, I think that for testing the boardgame package I still want newGame so I can call setUp later. But in general usage manager.NewGame() returning a SetUp'd game is more useful and less error prone.
Nah, it's not useful to have manager.NewGame, because SetUp() returns its own error, and it's weird and clunky to wire that back, too.
So now everything's done on this ticket.
To underline that it's not actually a game that should be modified, just have e.g. its chest used as an example.
Important after we fix #81 and #94