Make split take priority over I18

mhluska · Jan 29, 2023 · a72b961 · a72b961
1 parent bfec99a
commit a72b961
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 2 deletions.
diff --git a/src/game.ts b/src/game.ts
@@ -221,12 +221,14 @@ export default class Game extends EventEmitter {
     const hiLoResult = HiLoDeviationChecker.check(this, hand, input);
     const basicStrategyResult = BasicStrategyChecker.check(this, hand, input);
 
-    // If surrender is correct, it should take priority over I18 deviations.
+    // If surrender/split is correct, it should take priority over I18
+    // deviations.
     // TODO: After we add a game mode for Fab4, we should split those checks out
     // into their own checker since we do want to still run those in the
     // surrender case.
     const checkerResult =
-      input === Move.Surrender && basicStrategyResult === true
+      (input === Move.Surrender || input === Move.Split) &&
+      basicStrategyResult === true
         ? true
         : hiLoResult || basicStrategyResult;
 

diff --git a/test/src/game.ts b/test/src/game.ts
@@ -327,6 +327,32 @@ describe('Game', function () {
       });
     });
 
+    context('when splitting is allowed', function () {
+      before(function () {
+        game = setupGame({
+          settings: {
+            deckCount: 6,
+            mode: GameMode.Illustrious18,
+          },
+          dealerCards: [Rank.Two],
+          playerCards: [Rank.Six, Rank.Six],
+        });
+
+        runGame(game, {
+          input: [
+            {
+              [GameStep.WaitingForPlayInput]: Move.Split,
+            },
+          ],
+        });
+      });
+
+      it('should take priority over illustrious 18', function () {
+        expect(game.state.sessionMovesTotal).to.equal(1);
+        expect(game.state.sessionMovesCorrect).to.equal(1);
+      });
+    });
+
     context('when resplit aces is disabled', function () {
       before(function () {
         game = setupGame({