Convert start game command to event sourcing

dshaneg · Mar 1, 2018 · d9e86d1 · d9e86d1
1 parent 9b3b7f2
commit d9e86d1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/src/commands/start-game-command.ts b/src/commands/start-game-command.ts
@@ -3,7 +3,7 @@
 import { Command } from './command';
 import { EventPublisher } from '../domain/event-publisher';
 import { Voice } from '../domain/voice';
-import { GameState } from '../state/game-state';
+import { ReadOnlyGameState } from '../state/game-state';
 import { CommandFactory } from './command-factory';
 
 import { ItemRepository } from '../item-repository';
@@ -33,7 +33,7 @@ export class StartGameCommand {
   private gameDefinitionRepository: GameDefinitionRepository;
   private mapNodeRepository: MapNodeRepository;
 
-  execute(gameState: GameState, publisher: EventPublisher): void {
+  execute(gameState: ReadOnlyGameState, publisher: EventPublisher): void {
     if (gameState.isStarted) {
       return;
     }
@@ -57,8 +57,6 @@ export class StartGameCommand {
       voice: Voice.herald
     });
 
-    gameState.start();
-
     publisher.publish({
       topic: 'game.started',
       message: this.gameDefinitionRepository.getGameDefinition().opening,

diff --git a/src/domain/game-state-event-subscriber.ts b/src/domain/game-state-event-subscriber.ts
@@ -18,6 +18,9 @@ export class GameStateEventSubscriber implements EventSubscriber {
       case 'player.inventory.item-equipped':
         this.gameState.equip(event.item);
         break;
+      case 'game.started':
+        this.gameState.start();
+        break;
     }
   }
 }
diff --git a/src/test/commands/start-game.spec.ts b/src/test/commands/start-game.spec.ts
@@ -218,11 +218,6 @@ describe('StartGameCommand', () => {
       it('Should execute TeleportCommand.', () => {
         mockito.verify(TeleportCommandMock.execute(gameState, publisher)).once();
       });
-
-      // gameState start
-      it('Should execute start on gameState.', () => {
-        mockito.verify(GameStateMock.start()).once();
-      });
     });
 
     describe('Starting an already started game', () => {