test(tree): cover split cleanup ownership
- commit
- 82da5b8ffb6ab283050b37be65d70e7da55b214f
- author
- brice · · 2026-07-14 06:21 UTC
- committer
- brice · · 2026-07-14 07:06 UTC
- parent
- 5a776d4
- signature
- GPG Verified
CD88F9E564E0102646FB6077A1A07C7C7C3E349D· arch-gpg Verified for the committer email
1 file · +48 -0
A internal/ui/layout/tree_cleanup_test.go +48 -0
| @@ -0,0 +1,48 @@ | |||
| 1 | + | package layout_test | |
| 2 | + | ||
| 3 | + | import ( | |
| 4 | + | "context" | |
| 5 | + | "testing" | |
| 6 | + | ||
| 7 | + | "github.com/bnema/dumber/internal/domain/entity" | |
| 8 | + | "github.com/bnema/dumber/internal/ui/layout" | |
| 9 | + | "github.com/bnema/dumber/internal/ui/layout/mocks" | |
| 10 | + | "github.com/stretchr/testify/mock" | |
| 11 | + | "github.com/stretchr/testify/require" | |
| 12 | + | ) | |
| 13 | + | ||
| 14 | + | func TestTreeRendererClearReleasesOwnedSplitTickExactlyOnce(t *testing.T) { | |
| 15 | + | ctx := context.Background() | |
| 16 | + | factory := mocks.NewMockWidgetFactory(t) | |
| 17 | + | paneFactory := mocks.NewMockPaneViewFactory(t) | |
| 18 | + | paned := mocks.NewMockPanedWidget(t) | |
| 19 | + | leftStack, leftContainer := setupStackedLeafMocks(t, factory) | |
| 20 | + | rightStack, rightContainer := setupStackedLeafMocks(t, factory) | |
| 21 | + | ||
| 22 | + | left := &entity.PaneNode{ID: "left", Pane: entity.NewPane("left")} | |
| 23 | + | right := &entity.PaneNode{ID: "right", Pane: entity.NewPane("right")} | |
| 24 | + | root := &entity.PaneNode{ID: "split", SplitDir: entity.SplitHorizontal, SplitRatio: 0.5, Children: []*entity.PaneNode{left, right}} | |
| 25 | + | ||
| 26 | + | paneFactory.EXPECT().CreatePaneView(left).Return(leftContainer).Once() | |
| 27 | + | paneFactory.EXPECT().CreatePaneView(right).Return(rightContainer).Once() | |
| 28 | + | factory.EXPECT().NewPaned(layout.OrientationHorizontal).Return(paned).Once() | |
| 29 | + | paned.EXPECT().SetResizeStartChild(true).Once() | |
| 30 | + | paned.EXPECT().SetResizeEndChild(true).Once() | |
| 31 | + | paned.EXPECT().SetVisible(true).Once() | |
| 32 | + | leftStack.EXPECT().SetVisible(true).Once() | |
| 33 | + | rightStack.EXPECT().SetVisible(true).Once() | |
| 34 | + | paned.EXPECT().SetStartChild(leftStack).Once() | |
| 35 | + | paned.EXPECT().SetEndChild(rightStack).Once() | |
| 36 | + | paned.EXPECT().ConnectNotifyPosition(mock.Anything).Return(uint(0)).Once() | |
| 37 | + | paned.EXPECT().GetAllocatedWidth().Return(0).Once() | |
| 38 | + | paned.EXPECT().ConnectMap(mock.Anything).Return(uint(0)).Once() | |
| 39 | + | paned.EXPECT().AddTickCallback(mock.Anything).Return(uint(42)).Once() | |
| 40 | + | paned.EXPECT().RemoveTickCallback(uint(42)).Once() | |
| 41 | + | ||
| 42 | + | renderer := layout.NewTreeRenderer(ctx, factory, paneFactory) | |
| 43 | + | _, err := renderer.Build(ctx, root) | |
| 44 | + | require.NoError(t, err) | |
| 45 | + | ||
| 46 | + | renderer.Clear() | |
| 47 | + | renderer.Clear() | |
| 48 | + | } | |
| @@ -0,0 +1,48 @@ | |||||
| 1 | + | package layout_test | |||
| 2 | + | ||||
| 3 | + | import ( | |||
| 4 | + | "context" | |||
| 5 | + | "testing" | |||
| 6 | + | ||||
| 7 | + | "github.com/bnema/dumber/internal/domain/entity" | |||
| 8 | + | "github.com/bnema/dumber/internal/ui/layout" | |||
| 9 | + | "github.com/bnema/dumber/internal/ui/layout/mocks" | |||
| 10 | + | "github.com/stretchr/testify/mock" | |||
| 11 | + | "github.com/stretchr/testify/require" | |||
| 12 | + | ) | |||
| 13 | + | ||||
| 14 | + | func TestTreeRendererClearReleasesOwnedSplitTickExactlyOnce(t *testing.T) { | |||
| 15 | + | ctx := context.Background() | |||
| 16 | + | factory := mocks.NewMockWidgetFactory(t) | |||
| 17 | + | paneFactory := mocks.NewMockPaneViewFactory(t) | |||
| 18 | + | paned := mocks.NewMockPanedWidget(t) | |||
| 19 | + | leftStack, leftContainer := setupStackedLeafMocks(t, factory) | |||
| 20 | + | rightStack, rightContainer := setupStackedLeafMocks(t, factory) | |||
| 21 | + | ||||
| 22 | + | left := &entity.PaneNode{ID: "left", Pane: entity.NewPane("left")} | |||
| 23 | + | right := &entity.PaneNode{ID: "right", Pane: entity.NewPane("right")} | |||
| 24 | + | root := &entity.PaneNode{ID: "split", SplitDir: entity.SplitHorizontal, SplitRatio: 0.5, Children: []*entity.PaneNode{left, right}} | |||
| 25 | + | ||||
| 26 | + | paneFactory.EXPECT().CreatePaneView(left).Return(leftContainer).Once() | |||
| 27 | + | paneFactory.EXPECT().CreatePaneView(right).Return(rightContainer).Once() | |||
| 28 | + | factory.EXPECT().NewPaned(layout.OrientationHorizontal).Return(paned).Once() | |||
| 29 | + | paned.EXPECT().SetResizeStartChild(true).Once() | |||
| 30 | + | paned.EXPECT().SetResizeEndChild(true).Once() | |||
| 31 | + | paned.EXPECT().SetVisible(true).Once() | |||
| 32 | + | leftStack.EXPECT().SetVisible(true).Once() | |||
| 33 | + | rightStack.EXPECT().SetVisible(true).Once() | |||
| 34 | + | paned.EXPECT().SetStartChild(leftStack).Once() | |||
| 35 | + | paned.EXPECT().SetEndChild(rightStack).Once() | |||
| 36 | + | paned.EXPECT().ConnectNotifyPosition(mock.Anything).Return(uint(0)).Once() | |||
| 37 | + | paned.EXPECT().GetAllocatedWidth().Return(0).Once() | |||
| 38 | + | paned.EXPECT().ConnectMap(mock.Anything).Return(uint(0)).Once() | |||
| 39 | + | paned.EXPECT().AddTickCallback(mock.Anything).Return(uint(42)).Once() | |||
| 40 | + | paned.EXPECT().RemoveTickCallback(uint(42)).Once() | |||
| 41 | + | ||||
| 42 | + | renderer := layout.NewTreeRenderer(ctx, factory, paneFactory) | |||
| 43 | + | _, err := renderer.Build(ctx, root) | |||
| 44 | + | require.NoError(t, err) | |||
| 45 | + | ||||
| 46 | + | renderer.Clear() | |||
| 47 | + | renderer.Clear() | |||
| 48 | + | } | |||