don't run bot once game is over

boardgameio · Feb 14, 2020 · aed5cd1 · aed5cd1
1 parent 7c65046
commit aed5cd1
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/client/transport/local.js b/src/client/transport/local.js
@@ -16,6 +16,10 @@ import { Transport } from './transport';
  * Otherwise, returns a playerID of a bot that may play now.
  */
 export function GetBotPlayer(state, bots) {
+  if (state.ctx.gameover !== undefined) {
+    return null;
+  }
+
   if (state.ctx.stage) {
     for (const key of Object.keys(bots)) {
       if (key in state.ctx.stage) {

diff --git a/src/client/transport/local.test.js b/src/client/transport/local.test.js
@@ -106,6 +106,19 @@ describe('GetBotPlayer', () => {
     );
     expect(result).toEqual(null);
   });
+
+  test('gameover', () => {
+    const result = GetBotPlayer(
+      {
+        ctx: {
+          currentPlayer: '0',
+          gameover: true,
+        },
+      },
+      { '0': {} }
+    );
+    expect(result).toEqual(null);
+  });
 });
 
 describe('LocalMaster', () => {