From 38e6604f500423cff212f2da48fa41afb2f7aaa8 Mon Sep 17 00:00:00 2001 From: findthelorax Date: Mon, 20 Apr 2026 12:50:43 -0400 Subject: [PATCH] align canvasStore expectations with container-mode behavior --- frontend/src/stores/__tests__/canvasStore.test.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/frontend/src/stores/__tests__/canvasStore.test.ts b/frontend/src/stores/__tests__/canvasStore.test.ts index 65af38e..860dd67 100644 --- a/frontend/src/stores/__tests__/canvasStore.test.ts +++ b/frontend/src/stores/__tests__/canvasStore.test.ts @@ -99,7 +99,7 @@ describe('canvasStore', () => { it('updateNode clearing parent_id converts position to absolute and clears parentId', () => { const proxmox = { ...makeNode('px1', { type: 'proxmox', container_mode: true }), position: { x: 100, y: 100 } } - const lxc = { ...makeNode('lxc1', { type: 'lxc', parent_id: 'px1' }), position: { x: 30, y: 40 }, parentId: 'px1', extent: 'parent' as const } + const lxc = { ...makeNode('lxc1', { type: 'lxc', parent_id: 'px1' }), position: { x: 130, y: 140 }, parentId: 'px1', extent: 'parent' as const } useCanvasStore.getState().addNode(proxmox) useCanvasStore.getState().addNode(lxc) useCanvasStore.getState().updateNode('lxc1', { parent_id: undefined }) @@ -229,7 +229,7 @@ describe('canvasStore', () => { }) it('deleteNode also removes children with matching parentId', () => { - useCanvasStore.getState().addNode(makeNode('parent')) + useCanvasStore.getState().addNode(makeNode('parent', { container_mode: true })) useCanvasStore.getState().addNode(makeNode('child', { parent_id: 'parent' })) useCanvasStore.getState().deleteNode('parent') const { nodes } = useCanvasStore.getState() @@ -237,8 +237,8 @@ describe('canvasStore', () => { expect(nodes.find((n) => n.id === 'child')).toBeUndefined() }) - it('addNode with parent_id sets parentId and extent', () => { - useCanvasStore.getState().addNode(makeNode('parent')) + it('addNode with parent_id sets parentId and extent when parent is in container mode', () => { + useCanvasStore.getState().addNode(makeNode('parent', { container_mode: true })) useCanvasStore.getState().addNode(makeNode('child', { parent_id: 'parent' })) const child = useCanvasStore.getState().nodes.find((n) => n.id === 'child') expect(child?.parentId).toBe('parent')