test(split): cover tick callback cleanup
- commit
- a784a5312720b4e7e0894f218eb8de71adf20df8
- author
- brice · · 2026-07-14 06:16 UTC
- committer
- brice · · 2026-07-14 07:06 UTC
- parent
- 9c48930
- signature
- GPG Verified
CD88F9E564E0102646FB6077A1A07C7C7C3E349D· arch-gpg Verified for the committer email
1 file · +29 -0
A internal/ui/layout/split_cleanup_test.go +29 -0
| @@ -0,0 +1,29 @@ | |||
| 1 | + | package layout_test | |
| 2 | + | ||
| 3 | + | import ( | |
| 4 | + | "context" | |
| 5 | + | "testing" | |
| 6 | + | ||
| 7 | + | "github.com/stretchr/testify/mock" | |
| 8 | + | ||
| 9 | + | "github.com/bnema/dumber/internal/ui/layout" | |
| 10 | + | "github.com/bnema/dumber/internal/ui/layout/mocks" | |
| 11 | + | ) | |
| 12 | + | ||
| 13 | + | func TestSplitViewCleanupReleasesTickCallbackExactlyOnce(t *testing.T) { | |
| 14 | + | factory := mocks.NewMockWidgetFactory(t) | |
| 15 | + | paned := mocks.NewMockPanedWidget(t) | |
| 16 | + | factory.EXPECT().NewPaned(layout.OrientationHorizontal).Return(paned).Once() | |
| 17 | + | paned.EXPECT().SetResizeStartChild(true).Once() | |
| 18 | + | paned.EXPECT().SetResizeEndChild(true).Once() | |
| 19 | + | paned.EXPECT().SetVisible(true).Once() | |
| 20 | + | paned.EXPECT().GetAllocatedWidth().Return(0).Once() | |
| 21 | + | paned.EXPECT().ConnectNotifyPosition(mock.Anything).Return(uint(0)).Once() | |
| 22 | + | paned.EXPECT().ConnectMap(mock.Anything).Return(uint(0)).Once() | |
| 23 | + | paned.EXPECT().AddTickCallback(mock.Anything).Return(uint(42)).Once() | |
| 24 | + | paned.EXPECT().RemoveTickCallback(uint(42)).Once() | |
| 25 | + | ||
| 26 | + | sv := layout.NewSplitView(context.Background(), factory, layout.OrientationHorizontal, nil, nil, 0.5) | |
| 27 | + | sv.Cleanup() | |
| 28 | + | sv.Cleanup() | |
| 29 | + | } | |
| @@ -0,0 +1,29 @@ | |||||
| 1 | + | package layout_test | |||
| 2 | + | ||||
| 3 | + | import ( | |||
| 4 | + | "context" | |||
| 5 | + | "testing" | |||
| 6 | + | ||||
| 7 | + | "github.com/stretchr/testify/mock" | |||
| 8 | + | ||||
| 9 | + | "github.com/bnema/dumber/internal/ui/layout" | |||
| 10 | + | "github.com/bnema/dumber/internal/ui/layout/mocks" | |||
| 11 | + | ) | |||
| 12 | + | ||||
| 13 | + | func TestSplitViewCleanupReleasesTickCallbackExactlyOnce(t *testing.T) { | |||
| 14 | + | factory := mocks.NewMockWidgetFactory(t) | |||
| 15 | + | paned := mocks.NewMockPanedWidget(t) | |||
| 16 | + | factory.EXPECT().NewPaned(layout.OrientationHorizontal).Return(paned).Once() | |||
| 17 | + | paned.EXPECT().SetResizeStartChild(true).Once() | |||
| 18 | + | paned.EXPECT().SetResizeEndChild(true).Once() | |||
| 19 | + | paned.EXPECT().SetVisible(true).Once() | |||
| 20 | + | paned.EXPECT().GetAllocatedWidth().Return(0).Once() | |||
| 21 | + | paned.EXPECT().ConnectNotifyPosition(mock.Anything).Return(uint(0)).Once() | |||
| 22 | + | paned.EXPECT().ConnectMap(mock.Anything).Return(uint(0)).Once() | |||
| 23 | + | paned.EXPECT().AddTickCallback(mock.Anything).Return(uint(42)).Once() | |||
| 24 | + | paned.EXPECT().RemoveTickCallback(uint(42)).Once() | |||
| 25 | + | ||||
| 26 | + | sv := layout.NewSplitView(context.Background(), factory, layout.OrientationHorizontal, nil, nil, 0.5) | |||
| 27 | + | sv.Cleanup() | |||
| 28 | + | sv.Cleanup() | |||
| 29 | + | } | |||