GH-442: Do not remove reactions if not configured

utarwyn · Mar 1, 2023 · 0ec165e · 0ec165e
1 parent 357deb5
commit 0ec165e
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/src/__tests__/GameBoard.test.ts b/src/__tests__/GameBoard.test.ts
@@ -225,6 +225,22 @@ describe('GameBoard', () => {
                 await gameBoard.attemptNextTurn();
                 expect(tunnel.end).toHaveBeenCalledTimes(1);
             });
+
+            it.each`
+                gameBoardReactions | calledTimes | description
+                ${false}           | ${0}        | ${'not delete'}
+                ${true}            | ${1}        | ${'delete'}
+            `(
+                'should $description reactions at the end of the game if reactions are $gameBoardReactions',
+                async ({ gameBoardReactions, calledTimes }) => {
+                    configuration.gameBoardReactions = gameBoardReactions;
+                    const reply = { reactions: { removeAll: jest.fn() } as any } as Message;
+                    Object.assign(tunnel, { reply });
+                    jest.spyOn(ai, 'operate').mockReturnValue({ move: 5, score: 1 });
+                    await gameBoard.attemptNextTurn();
+                    expect(reply.reactions.removeAll).toHaveBeenCalledTimes(calledTimes);
+                }
+            );
         });
 
         describe('Human: wait for a reaction', () => {

diff --git a/src/bot/entity/GameBoard.ts b/src/bot/entity/GameBoard.ts
@@ -240,7 +240,9 @@ export default class GameBoard {
                 const options = this.createBuilder().withEndingMessage(winner).toMessageOptions();
                 await this.tunnel.end(options);
             } else {
-                await this.tunnel.reply?.reactions?.removeAll();
+                if (this.configuration.gameBoardReactions) {
+                    await this.tunnel.reply?.reactions?.removeAll();
+                }
                 await this.update(interaction);
             }