brick-engine-js 1.0.1 → 1.0.2

package/src/core/module/control/GameControlKeyBinding.test.ts

@@ -1,59 +0,0 @@
-/** @vitest-environment jsdom */
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import GameControlKeyBinding from './GameControlKeyBinding';
-import { Control } from '../../types/modules';
-import { ControlKey } from '../../types/enums';
-
-describe('GameControlKeyBinding', () => {
-    let binding: GameControlKeyBinding;
-    let mockControl: Control;
-
-    beforeEach(() => {
-        mockControl = {
-            notify: vi.fn(),
-        } as unknown as Control;
-
-        binding = new GameControlKeyBinding(mockControl);
-    });
-
-    it('should bind and unbind window events', () => {
-        const addSpy = vi.spyOn(window, 'addEventListener');
-        const removeSpy = vi.spyOn(window, 'removeEventListener');
-
-        binding.bindControls();
-        expect(addSpy).toHaveBeenCalledWith('keydown', expect.any(Function));
-
-        binding.unbindControls();
-        expect(removeSpy).toHaveBeenCalledWith('keydown', expect.any(Function));
-    });
-
-    it('should trigger control press on valid key down', () => {
-        binding.bindControls();
-
-        // Simulating KeyboardEvent
-        const event = new KeyboardEvent('keydown', { code: 'ArrowUp' });
-        window.dispatchEvent(event);
-
-        expect(mockControl.notify).toHaveBeenCalledWith(ControlKey.UP, 'pressed');
-    });
-
-    it('should ignore repeated native key down events', () => {
-        binding.bindControls();
-
-        const event = new KeyboardEvent('keydown', { code: 'ArrowUp', repeat: true } as KeyboardEventInit);
-        window.dispatchEvent(event);
-
-        expect(mockControl.notify).not.toHaveBeenCalled();
-    });
-
-    it('should trigger control release on key up', () => {
-        binding.bindControls();
-
-        const event = new KeyboardEvent('keyup', { code: 'ArrowLeft' });
-        window.dispatchEvent(event);
-
-        // ControlInputHandler doesn't notify on release directly but we can verify it doesn't crash
-        // Actually, let's verify if handleRelease was called on the handler
-        // Since it's private, we just check if it doesn't error and maybe mock the handler if we could
-    });
-});

package/src/core/module/control/GameControlKeyBinding.ts

@@ -1,92 +0,0 @@
-import { ControlKey } from '../../types/enums';
-import { Control } from '../../types/modules';
-import ControlInputHandler from '../../helpers/ControlInputHandlerHelper';
-
-const DEFAULT_KEY_MAP: Record<string, ControlKey> = {
-    // Arrows
-    ArrowUp: ControlKey.UP,
-    ArrowDown: ControlKey.DOWN,
-    ArrowLeft: ControlKey.LEFT,
-    ArrowRight: ControlKey.RIGHT,
-
-    // WASD
-    KeyW: ControlKey.UP,
-    KeyS: ControlKey.DOWN,
-    KeyA: ControlKey.LEFT,
-    KeyD: ControlKey.RIGHT,
-
-    // Actions
-    KeyJ: ControlKey.ACTION,
-
-    // System
-    Digit1: ControlKey.POWER,
-    Digit2: ControlKey.START_PAUSE,
-    Digit3: ControlKey.SOUND,
-    Digit4: ControlKey.RESET,
-    Digit5: ControlKey.EXIT,
-    Digit6: ControlKey.COLOR,
-};
-
-/**
- * Handles the binding of physical keyboard events to logical control actions.
- * Translates Browser `KeyboardEvent`s into engine `ControlKey`s.
- */
-export default class GameControlKeyBinding {
-    private _inputHandler: ControlInputHandler;
-
-    private _boundHandleKeyDown: (event: KeyboardEvent) => void;
-    private _boundHandleKeyUp: (event: KeyboardEvent) => void;
-
-    /**
-     * Creates an instance of GameControlKeyBinding.
-     *
-     * @param {Control} control - The parent control module.
-     */
-    constructor(control: Control) {
-        this._inputHandler = new ControlInputHandler(control);
-        this._boundHandleKeyDown = this._handleKeyDown.bind(this);
-        this._boundHandleKeyUp = this._handleKeyUp.bind(this);
-    }
-
-    /**
-     * Binds keyboard events to the global window object.
-     */
-    bindControls() {
-        window.addEventListener('keydown', this._boundHandleKeyDown);
-        window.addEventListener('keyup', this._boundHandleKeyUp);
-    }
-
-    /**
-     * Unbinds keyboard events from the global window object.
-     */
-    unbindControls() {
-        window.removeEventListener('keydown', this._boundHandleKeyDown);
-        window.removeEventListener('keyup', this._boundHandleKeyUp);
-    }
-
-    /**
-     * Internal handler for key down events.
-     *
-     * @param {KeyboardEvent} event - The browser keyboard event.
-     */
-    private _handleKeyDown(event: KeyboardEvent) {
-        const controlKey = DEFAULT_KEY_MAP[event.code];
-        if (!controlKey) return;
-
-        if (event.repeat) return; // Ignore native repeat, fully delegating repeat logic to InputHandler
-
-        this._inputHandler.handlePress(controlKey);
-    }
-
-    /**
-     * Internal handler for key up events.
-     *
-     * @param {KeyboardEvent} event - The browser keyboard event.
-     */
-    private _handleKeyUp(event: KeyboardEvent) {
-        const controlKey = DEFAULT_KEY_MAP[event.code];
-        if (!controlKey) return;
-
-        this._inputHandler.handleRelease(controlKey);
-    }
-}

package/src/core/module/grid/GameGrid.test.ts

@@ -1,83 +0,0 @@
-import { describe, it, expect, beforeEach } from 'vitest';
-import GameGrid from './GameGrid';
-import { Color } from '../../types/enums';
-import CellHelper from '../../helpers/CellHelper';
-
-describe('GameGrid', () => {
-    let grid: GameGrid;
-
-    beforeEach(() => {
-        grid = new GameGrid();
-        grid.setup();
-    });
-
-    describe('Core & Utilities', () => {
-        it('should initialize with empty cells', () => {
-            const data = grid.getGrid();
-            expect(data.length).toBe(grid.height);
-            expect(data[0].length).toBe(grid.width);
-            expect(data[0][0].value).toBe(0);
-        });
-
-        it('should validate coordinates', () => {
-            expect(grid.isValidCoordinate({ x: 0, y: 0 })).toBe(true);
-            expect(grid.isValidCoordinate({ x: -1, y: 0 })).toBe(false);
-            expect(grid.isValidCoordinate({ x: grid.width, y: 0 })).toBe(false);
-        });
-
-        it('should get and set cell values/colors', () => {
-            const coord = { x: 5, y: 5 };
-            grid.setCellValue(coord, 1);
-            grid.setCellColor(coord, Color.RED);
-
-            expect(grid.isCellActive(coord)).toBe(true);
-            expect(grid.getCell(coord)?.color).toBe(Color.RED);
-        });
-
-        it('should return null for out of bounds cell access', () => {
-            expect(grid.getCell({ x: -1, y: 0 })).toBeNull();
-        });
-    });
-
-    describe('Iteration', () => {
-        it('should iterate over all cells', () => {
-            let count = 0;
-            grid.forEach(() => count++);
-            expect(count).toBe(grid.width * grid.height);
-        });
-    });
-
-    describe('Delegation', () => {
-        // We verify that calling grid methods calls the internal engines
-        // Since engines are private, we can't easily mock them without reaching into internals
-        // but given the implementation, we can just check if the methods exist and work as expected
-        // as high-level integration tests.
-
-        it('should clear full rows (delegated to line engine)', () => {
-            // Fill a row
-            for (let x = 0; x < grid.width; x++) grid.setCellValue({ x, y: 0 }, 1);
-
-            const cleared = grid.clearFullRows();
-            expect(cleared).toBe(1);
-            expect(grid.isRowEmpty(0)).toBe(true);
-        });
-
-        it('should move piece (delegated to movement engine)', () => {
-            const piece = [CellHelper.emptyCell({ x: 5, y: 5 })];
-            piece[0].value = 1;
-
-            const moved = grid.movePiece(piece, { x: 1, y: 0 });
-            expect(moved![0].coordinate).toEqual({ x: 6, y: 5 });
-        });
-    });
-
-    describe('Debug', () => {
-        it('should return correct debug data', () => {
-            grid.setCellValue({ x: 0, y: 0 }, 1);
-            const debug = grid.getDebugData();
-            expect(debug.activeCells).toBe(1);
-            expect(debug.width).toBe(grid.width);
-            expect(debug.height).toBe(grid.height);
-        });
-    });
-});