From 93ed0c1d8b225e4f86cc2d522da8fb9372923da5 Mon Sep 17 00:00:00 2001
From: Manu Artero Anguita <manutero.developer@gmail.com>
Date: Fri, 25 Nov 2022 14:12:14 +0100
Subject: [PATCH] feat: declare greatest empire

 - function declareGreatestEmpire(player) on usePlayer()
 - test usePlayer.declareGreatestEmpire
 - function: empireSize(board): Record<Player, number>
 - test empire-size
 - refactor: move isConquering({ ... }) from useBoard()
 - check if player is creating greatest empire on build action
 - TODO: check if player is creating greatest empire on conquer!
---
 src/contexts/game-context/use-board.ts        | 14 +---
 .../game-context/use-game-context.tsx         | 18 +++--
 .../game-context/use-players.test.tsx         | 67 ++++++++++++++++---
 src/contexts/game-context/use-players.ts      | 18 ++++-
 src/models/empire-size.test.ts                | 40 +++++++++++
 src/models/empire-size.ts                     | 18 +++++
 src/models/score.ts                           | 35 ++++++++++
 7 files changed, 184 insertions(+), 26 deletions(-)
 create mode 100644 src/models/empire-size.test.ts
 create mode 100644 src/models/empire-size.ts
 create mode 100644 src/models/score.ts

diff --git a/src/contexts/game-context/use-board.ts b/src/contexts/game-context/use-board.ts
index 92e934f..c76f2d0 100644
--- a/src/contexts/game-context/use-board.ts
+++ b/src/contexts/game-context/use-board.ts
@@ -26,14 +26,6 @@ export function useBoard() {
     });
   };
 
-  const isConquering = ({ piece, to }: { to: TileID; piece: Piece }) => {
-    if (board[to].building && board[to].building?.owner !== piece.owner) {
-      console.info(`GameContext.isConquering({ ... }): YES`);
-      return true;
-    }
-    return false;
-  };
-
   /** direct update on the board; no validity check */
   const movePiece = ({ piece, from, to }: MoveAction) => {
     console.info(
@@ -48,7 +40,6 @@ export function useBoard() {
         ...currentBoard[to],
         piece,
       };
-      // is it better to use isConquering() ?
       if (targetTile.building) {
         targetTile.building = {
           ...targetTile.building,
@@ -79,7 +70,7 @@ export function useBoard() {
     });
   };
 
-  return { board, buildOnTile, movePiece, recruitOnTile, isConquering };
+  return { board, buildOnTile, movePiece, recruitOnTile };
 }
 
 /**
@@ -90,6 +81,7 @@ export const emptyBoard: Board = tiles.reduce((acc, key) => {
   acc[key] = {
     terrain: "field",
     piece: undefined,
+    building: undefined,
   };
   return acc;
-}, {} as any);
+}, {} as Record<TileID, Tile>);
diff --git a/src/contexts/game-context/use-game-context.tsx b/src/contexts/game-context/use-game-context.tsx
index 785f154..319cbad 100644
--- a/src/contexts/game-context/use-game-context.tsx
+++ b/src/contexts/game-context/use-game-context.tsx
@@ -3,6 +3,8 @@ import { logRender } from "utils/console";
 import { useBoard, emptyBoard } from "./use-board";
 import usePlayers from "./use-players";
 import { emptyTimeline, useTimeline } from "./use-timeline";
+import { isConquering, isCreatingGreatesEmpire } from "models/score";
+import { empireSize } from "models/empire-size";
 
 const GameContext = createContext<GameContext>({
   phase: "setup",
@@ -46,10 +48,10 @@ export function GameContextProvider({ children }: Props): JSX.Element {
   logRender("GameContextProvider");
 
   const [phase, setPhase] = useState<PhaseType>("planification"); // will be setup
-  const { board, isConquering, buildOnTile, movePiece, recruitOnTile } =
-    useBoard();
+  const { board, buildOnTile, movePiece, recruitOnTile } = useBoard();
   const { timeline, nextCard, planification, newTurn } = useTimeline();
-  const { players, firstPlayer, scorePoint } = usePlayers();
+  const { players, firstPlayer, scorePoint, declareGreatestEmpire } =
+    usePlayers();
 
   /* derived state */
   const activeCard = timeline.current;
@@ -98,13 +100,19 @@ export function GameContextProvider({ children }: Props): JSX.Element {
         players,
 
         build(action: BuildAction) {
+          if (
+            isCreatingGreatesEmpire({ ...action, empires: empireSize(board) })
+          ) {
+            declareGreatestEmpire(action.building.owner);
+          }
           buildOnTile(action);
           resolveActionCard();
         },
 
         move(action: MoveAction) {
-          if (isConquering(action)) {
-            scorePoint(action.piece.owner);
+          const player = action.piece.owner;
+          if (isConquering({ player, targetTile: board[action.to] })) {
+            scorePoint(player);
           }
           movePiece(action);
           resolveActionCard();
diff --git a/src/contexts/game-context/use-players.test.tsx b/src/contexts/game-context/use-players.test.tsx
index f663096..a2a616f 100644
--- a/src/contexts/game-context/use-players.test.tsx
+++ b/src/contexts/game-context/use-players.test.tsx
@@ -2,21 +2,37 @@ import { render, screen, fireEvent } from "@testing-library/react";
 import usePlayers from "./use-players";
 
 function TestingComponent() {
-  const { players, firstPlayer, scorePoint } = usePlayers();
+  const { players, firstPlayer, scorePoint, declareGreatestEmpire } =
+    usePlayers();
   return (
     <>
       <ul>
         {players.map((player, i) => (
-          <li
-            key={`player-#${i}`}
-          >{`${player.player} - ${player.points} points`}</li>
+          <li key={`player-#${i}`}>{`${player.player} - ${
+            player.points
+          } points${player.greatestEmpirePoint ? "*" : ""}`}</li>
         ))}
       </ul>
-      <button data-testid="enemy1-scores-point" onClick={() => scorePoint("enemy1")} />
+      <button
+        data-testid="enemy1-scores-point"
+        onClick={() => scorePoint("enemy1")}
+      />
       <button
         data-testid="enemy2-first-player"
         onClick={() => firstPlayer("enemy2")}
       />
+      <button
+        data-testid="player-first-player"
+        onClick={() => firstPlayer("player")}
+      />
+      <button
+        data-testid="enemy3-greatest-empire"
+        onClick={() => declareGreatestEmpire("enemy3")}
+      />
+      <button
+        data-testid="player-greatest-empire"
+        onClick={() => declareGreatestEmpire("player")}
+      />
     </>
   );
 }
@@ -24,8 +40,8 @@ function TestingComponent() {
 const getPlayerList = () =>
   Array.from(screen.getByRole("list").children).map((li) => li.textContent);
 
-describe("game-context", () => {
-  test("usePlayers() returns players[]", () => {
+describe("gameContext.usePlayers()", () => {
+  it("returns players[]", () => {
     render(<TestingComponent />);
     expect(getPlayerList()).toEqual([
       "player - 0 points",
@@ -35,7 +51,7 @@ describe("game-context", () => {
     ]);
   });
 
-  test("usePlayers() returns scorePoint()", () => {
+  test("returns scorePoint()", () => {
     render(<TestingComponent />);
     fireEvent.click(screen.getByTestId("enemy1-scores-point"));
     expect(getPlayerList()).toEqual([
@@ -44,9 +60,17 @@ describe("game-context", () => {
       "enemy2 - 0 points",
       "enemy3 - 0 points",
     ]);
+    fireEvent.click(screen.getByTestId("enemy1-scores-point"));
+    fireEvent.click(screen.getByTestId("enemy1-scores-point"));
+    expect(getPlayerList()).toEqual([
+      "player - 0 points",
+      "enemy1 - 3 points",
+      "enemy2 - 0 points",
+      "enemy3 - 0 points",
+    ]);
   });
 
-  test("usePlayers() returns firstPlayer()", () => {
+  test("returns firstPlayer()", () => {
     render(<TestingComponent />);
     fireEvent.click(screen.getByTestId("enemy2-first-player"));
     expect(getPlayerList()).toEqual([
@@ -55,5 +79,30 @@ describe("game-context", () => {
       "enemy1 - 0 points",
       "enemy3 - 0 points",
     ]);
+    fireEvent.click(screen.getByTestId("player-first-player"));
+    expect(getPlayerList()).toEqual([
+      "player - 0 points",
+      "enemy2 - 0 points",
+      "enemy1 - 0 points",
+      "enemy3 - 0 points",
+    ]);
+  });
+
+  test("returns declareGreatesEmpire()", () => {
+    render(<TestingComponent />);
+    fireEvent.click(screen.getByTestId("enemy3-greatest-empire"));
+    expect(getPlayerList()).toEqual([
+      "player - 0 points",
+      "enemy1 - 0 points",
+      "enemy2 - 0 points",
+      "enemy3 - 0 points*",
+    ]);
+    fireEvent.click(screen.getByTestId("player-greatest-empire"));
+    expect(getPlayerList()).toEqual([
+      "player - 0 points*",
+      "enemy1 - 0 points",
+      "enemy2 - 0 points",
+      "enemy3 - 0 points",
+    ]);
   });
 });
diff --git a/src/contexts/game-context/use-players.ts b/src/contexts/game-context/use-players.ts
index 3eefbd9..796412b 100644
--- a/src/contexts/game-context/use-players.ts
+++ b/src/contexts/game-context/use-players.ts
@@ -8,6 +8,11 @@ const initialPlayerStatus: PlayerStatus[] = [
   { player: "enemy3", points: 0, greatestEmpirePoint: false },
 ];
 
+/**
+ * plain react state + named update methods
+ *
+ * **no game logic here**
+ */
 function usePlayers() {
   const [players, setPlayers] = useState(initialPlayerStatus);
 
@@ -29,7 +34,18 @@ function usePlayers() {
     );
   };
 
-  return { players, firstPlayer, scorePoint };
+  const declareGreatestEmpire = (player: Player) => {
+    console.info(`GameContext.declareGreatestEmpire({ player: ${player} })`);
+    setPlayers((currentPlayers) =>
+      currentPlayers.map((p) =>
+        p.player === player
+          ? { ...p, greatestEmpirePoint: true }
+          : { ...p, greatestEmpirePoint: false }
+      )
+    );
+  };
+
+  return { players, firstPlayer, scorePoint, declareGreatestEmpire };
 }
 
 export default usePlayers;
diff --git a/src/models/empire-size.test.ts b/src/models/empire-size.test.ts
new file mode 100644
index 0000000..87553d2
--- /dev/null
+++ b/src/models/empire-size.test.ts
@@ -0,0 +1,40 @@
+import { empireSize } from "./empire-size";
+
+const board = {
+  "-2,3": {
+    terrain: "field",
+    building: { owner: "player", type: "town" },
+    piece: undefined,
+  },
+  "-1,3": {
+    terrain: "field",
+    building: undefined,
+    piece: undefined,
+  },
+  "0,3": {
+    terrain: "field",
+    building: { owner: "enemy3", type: "village" },
+    piece: undefined,
+  },
+  "1,3": {
+    terrain: "field",
+    building: { owner: "enemy3", type: "city" },
+    piece: undefined,
+  },
+  "2,3": {
+    terrain: "field",
+    building: { owner: "enemy1", type: "city" },
+    piece: undefined,
+  },
+} as Board;
+
+describe("empireSize()", () => {
+  it("returns number of buildings grouped by player", () => {
+    expect(empireSize(board)).toEqual({
+      player: 1,
+      enemy1: 1,
+      enemy2: 0,
+      enemy3: 2,
+    });
+  });
+});
diff --git a/src/models/empire-size.ts b/src/models/empire-size.ts
new file mode 100644
index 0000000..7edefda
--- /dev/null
+++ b/src/models/empire-size.ts
@@ -0,0 +1,18 @@
+export function empireSize(board: Board) {
+  return Object.entries(board).reduce(
+    (acc, [_, tile]) => {
+      if (tile.building) {
+        const player = tile.building.owner;
+        acc[player] = acc[player] + 1;
+      }
+      return acc;
+    },
+    {
+      player: 0,
+      enemy1: 0,
+      enemy2: 0,
+      enemy3: 0,
+    }
+  );
+}
+
diff --git a/src/models/score.ts b/src/models/score.ts
new file mode 100644
index 0000000..12b2ecd
--- /dev/null
+++ b/src/models/score.ts
@@ -0,0 +1,35 @@
+export function isConquering({
+  targetTile,
+  player,
+}: {
+  targetTile: Tile;
+  player: Player;
+}) {
+  if (targetTile.building && targetTile.building.owner !== player) {
+    console.info(`Score: ${player} is conquering a settlement`);
+    return true;
+  }
+  return false;
+}
+
+export function isCreatingGreatesEmpire({
+  building,
+  empires,
+}: {
+  building: Building;
+  empires: Record<Player, number>;
+}) {
+  const player = building.owner;
+  const newEmpireSize = empires[player] + 1;
+  if (newEmpireSize < 3) {
+    return false;
+  }
+  const isGreatestEmpire = Object.values(empires).every(
+    (size) => size < newEmpireSize
+  );
+  if (isGreatestEmpire) {
+    console.info(`Score: ${player} is creating the greatest empire`);
+    return true;
+  }
+  return false;
+}