align canvasStore expectations with container-mode behavior

This commit is contained in:
findthelorax
2026-04-20 12:50:43 -04:00
committed by Remy
parent 0e70b45e8a
commit 38e6604f50
@@ -99,7 +99,7 @@ describe('canvasStore', () => {
it('updateNode clearing parent_id converts position to absolute and clears parentId', () => { 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 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(proxmox)
useCanvasStore.getState().addNode(lxc) useCanvasStore.getState().addNode(lxc)
useCanvasStore.getState().updateNode('lxc1', { parent_id: undefined }) useCanvasStore.getState().updateNode('lxc1', { parent_id: undefined })
@@ -229,7 +229,7 @@ describe('canvasStore', () => {
}) })
it('deleteNode also removes children with matching parentId', () => { 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().addNode(makeNode('child', { parent_id: 'parent' }))
useCanvasStore.getState().deleteNode('parent') useCanvasStore.getState().deleteNode('parent')
const { nodes } = useCanvasStore.getState() const { nodes } = useCanvasStore.getState()
@@ -237,8 +237,8 @@ describe('canvasStore', () => {
expect(nodes.find((n) => n.id === 'child')).toBeUndefined() expect(nodes.find((n) => n.id === 'child')).toBeUndefined()
}) })
it('addNode with parent_id sets parentId and extent', () => { it('addNode with parent_id sets parentId and extent when parent is in container mode', () => {
useCanvasStore.getState().addNode(makeNode('parent')) useCanvasStore.getState().addNode(makeNode('parent', { container_mode: true }))
useCanvasStore.getState().addNode(makeNode('child', { parent_id: 'parent' })) useCanvasStore.getState().addNode(makeNode('child', { parent_id: 'parent' }))
const child = useCanvasStore.getState().nodes.find((n) => n.id === 'child') const child = useCanvasStore.getState().nodes.find((n) => n.id === 'child')
expect(child?.parentId).toBe('parent') expect(child?.parentId).toBe('parent')