diff --git a/src/client/graphics/layers/RadialMenuElements.ts b/src/client/graphics/layers/RadialMenuElements.ts index ae4c29d9d..29e5bd379 100644 --- a/src/client/graphics/layers/RadialMenuElements.ts +++ b/src/client/graphics/layers/RadialMenuElements.ts @@ -583,17 +583,11 @@ export const rootMenuElement: MenuElement = { const menuItems: (MenuElement | null)[] = [ infoMenuElement, - boatMenuElement, - ally, + ...(isOwnTerritory + ? [deleteUnitElement, ally, buildMenuElement] + : [boatMenuElement, ally, attackMenuElement]), ]; - if (isOwnTerritory) { - menuItems.push(buildMenuElement); - menuItems.push(deleteUnitElement); - } else { - menuItems.push(attackMenuElement); - } - return menuItems.filter((item): item is MenuElement => item !== null); }, }; diff --git a/src/core/configuration/DefaultConfig.ts b/src/core/configuration/DefaultConfig.ts index 3632e4f60..748aa2d62 100644 --- a/src/core/configuration/DefaultConfig.ts +++ b/src/core/configuration/DefaultConfig.ts @@ -586,7 +586,7 @@ export class DefaultConfig implements Config { return 15 * 10; } deleteUnitCooldown(): Tick { - return 5 * 10; + return 15 * 10; } emojiMessageDuration(): Tick { return 5 * 10; diff --git a/tests/DeleteUnitExecution.test.ts b/tests/DeleteUnitExecution.test.ts index a931c140b..04034a82f 100644 --- a/tests/DeleteUnitExecution.test.ts +++ b/tests/DeleteUnitExecution.test.ts @@ -53,6 +53,8 @@ describe("DeleteUnitExecution Security Tests", () => { game.executeNextTick(); } + executeTicks(game, game.config().deleteUnitCooldown() + 1); + player = game.player(player1Info.id); enemyPlayer = game.player(player2Info.id); diff --git a/tests/client/graphics/RadialMenuElements.test.ts b/tests/client/graphics/RadialMenuElements.test.ts index df8c796d4..67021fee9 100644 --- a/tests/client/graphics/RadialMenuElements.test.ts +++ b/tests/client/graphics/RadialMenuElements.test.ts @@ -299,16 +299,18 @@ describe("RadialMenuElements", () => { expect(rootMenuElement.disabled(mockParams)).toBe(false); }); - it("should show build menu on own territory", () => { + it("should show build and delete menu on own territory", () => { const subMenu = rootMenuElement.subMenu!(mockParams); const buildMenu = subMenu.find((item) => item.id === Slot.Build); const attackMenu = subMenu.find((item) => item.id === Slot.Attack); + const deleteMenu = subMenu.find((item) => item.id === Slot.Delete); expect(buildMenu).toBeDefined(); expect(attackMenu).toBeUndefined(); + expect(deleteMenu).toBeDefined(); }); - it("should show attack menu on enemy territory", () => { + it("should show attack and boat menu on enemy territory", () => { const enemyPlayer = { id: () => 2, isPlayer: jest.fn(() => true), @@ -318,18 +320,18 @@ describe("RadialMenuElements", () => { const subMenu = rootMenuElement.subMenu!(mockParams); const buildMenu = subMenu.find((item) => item.id === Slot.Build); const attackMenu = subMenu.find((item) => item.id === Slot.Attack); + const boatMenu = subMenu.find((item) => item.id === Slot.Boat); expect(attackMenu).toBeDefined(); expect(buildMenu).toBeUndefined(); + expect(boatMenu).toBeDefined(); }); - it("should include info and boat menus in both cases", () => { + it("should include info menu in both cases", () => { const subMenu = rootMenuElement.subMenu!(mockParams); const infoMenu = subMenu.find((item) => item.id === Slot.Info); - const boatMenu = subMenu.find((item) => item.id === Slot.Boat); expect(infoMenu).toBeDefined(); - expect(boatMenu).toBeDefined(); }); it("should handle ally menu correctly", () => {