From f12756eecf2ea791ad894cc63380ed78f22f8797 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Sat, 23 May 2026 20:16:15 +0300 Subject: 9c move Non-IO grouping policy from core stats/types into dashboard Snapshot.NonIOFamilies, Snapshot.NonIOFamiliesCount, and types.IsNonIOSyscallFamily encoded a TUI tab concept in core packages. Move this filtering into internal/tui/dashboard/nonio.go as unexported helpers so the dashboard owns its own grouping policy and Snapshot.Families remains the neutral core API. Co-Authored-By: Claude Opus 4.7 --- internal/statsengine/engine_test.go | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'internal/statsengine/engine_test.go') diff --git a/internal/statsengine/engine_test.go b/internal/statsengine/engine_test.go index 0500d20..6eb19e8 100644 --- a/internal/statsengine/engine_test.go +++ b/internal/statsengine/engine_test.go @@ -115,12 +115,13 @@ func TestEngineAggregatesSyscallFamilies(t *testing.T) { t.Fatalf("FS family = %+v, want count=1 bytes=4", families["FS"]) } - nonIO := familyRowsByName(snap.NonIOFamilies()) - if _, ok := nonIO["FS"]; ok { - t.Fatalf("NonIOFamilies should not include FS: %+v", nonIO["FS"]) + // Verify that non-FS families exist and FS is present in the full + // family list — Non-IO filtering has moved to the dashboard package. + if _, ok := families["Polling"]; !ok { + t.Fatalf("Families missing Polling row: %+v", families) } - if nonIO["Polling"].Count != 2 || nonIO["Process"].Count != 1 { - t.Fatalf("NonIOFamilies missing expected rows: %+v", nonIO) + if _, ok := families["Process"]; !ok { + t.Fatalf("Families missing Process row: %+v", families) } } -- cgit v1.2.3