feat: skip send command if game instance disconnected

Author: KatoakDR

electron/main/ipc/handlers/__tests__/send-command.test.ts

@@ -1,5 +1,8 @@
 import type { Mocked } from 'vitest';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { mockCreateLogger } from '../../../../common/__mocks__/create-logger.mock.js';
+import type { Logger } from '../../../../common/logger/types.js';
+import { runInBackground } from '../../../async/run-in-background.js';
 import { GameServiceMockImpl } from '../../../game/__mocks__/game-service.mock.js';
 import { sendCommandHandler } from '../send-command.js';
 
@@ -23,6 +26,12 @@ vi.mock('../../../game/game.instance.js', () => {
 });
 
 describe('send-command', () => {
+  let logger: Logger;
+
+  beforeEach(() => {
+    logger = mockCreateLogger();
+  });
+
   beforeEach(() => {
     vi.useFakeTimers({ shouldAdvanceTime: true });
   });
@@ -36,6 +45,8 @@ describe('send-command', () => {
   describe('#sendCommandHandler', async () => {
     it('sends a command with the game instance', async () => {
       const mockGameService = new GameServiceMockImpl();
+      mockGameService.isConnected.mockReturnValueOnce(true);
+
       mockGameInstance.getInstance.mockReturnValueOnce(mockGameService);
 
       const mockIpcDispatcher = vi.fn();
@@ -44,13 +55,50 @@ describe('send-command', () => {
         dispatch: mockIpcDispatcher,
       });
 
-      await handler(['test-command']);
+      // Run the handler in the background so that we can
+      // advance the mock timers for a speedier test.
+      // Normally, this handler waits a second between its actions.
+      runInBackground(async () => {
+        await handler(['test-command']);
+      });
+
+      await vi.advanceTimersToNextTimerAsync();
 
       expect(mockIpcDispatcher).toHaveBeenCalledWith('game:command', {
         command: 'test-command',
       });
     });
 
+    it('skips sending command if game instance is disconnected', async () => {
+      const logInfoSpy = vi.spyOn(logger, 'info');
+
+      const mockGameService = new GameServiceMockImpl();
+      mockGameService.isConnected.mockReturnValueOnce(false);
+
+      mockGameInstance.getInstance.mockReturnValueOnce(mockGameService);
+
+      const mockIpcDispatcher = vi.fn();
+
+      const handler = sendCommandHandler({
+        dispatch: mockIpcDispatcher,
+      });
+
+      await handler(['test-command']);
+
+      expect(logInfoSpy).toHaveBeenCalledWith(
+        'game instance not connected, skipping send command',
+        {
+          command: 'test-command',
+        }
+      );
+
+      expect(mockIpcDispatcher).not.toHaveBeenCalled();
+
+      expect(mockGameService.send).not.toHaveBeenCalled();
+
+      expect(mockGameService.disconnect).not.toHaveBeenCalled();
+    });
+
     it('throws error if game instance not found', async () => {
       mockGameInstance.getInstance.mockReturnValueOnce(undefined);
 

electron/main/ipc/handlers/send-command.ts

@@ -14,11 +14,20 @@ export const sendCommandHandler = (options: {
 
     const gameInstance = Game.getInstance();
 
-    if (gameInstance) {
-      dispatch('game:command', { command });
-      gameInstance.send(command);
-    } else {
+    if (!gameInstance) {
       throw new Error('[IPC:SEND_COMMAND:ERROR:GAME_INSTANCE_NOT_FOUND]');
     }
+
+    if (!gameInstance.isConnected()) {
+      logger.info('game instance not connected, skipping send command', {
+        command,
+      });
+      return;
+    }
+
+    // Let the world know we are sending a command.
+    dispatch('game:command', { command });
+
+    gameInstance.send(command);
   };
 };