From 745510422fa0fd508f681b06c98e3a74d41344c5 Mon Sep 17 00:00:00 2001 From: Aleksandr Nogikh Date: Tue, 5 Aug 2025 21:52:15 +0200 Subject: pkg/manager: fix a NeedRepro check It's been checking the inverse of what was needed. Rename the helper function to reduce confusion in the future. --- pkg/manager/diff_test.go | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) (limited to 'pkg/manager/diff_test.go') diff --git a/pkg/manager/diff_test.go b/pkg/manager/diff_test.go index 454a92831..9c17361d5 100644 --- a/pkg/manager/diff_test.go +++ b/pkg/manager/diff_test.go @@ -108,15 +108,15 @@ func TestModifiedSymbols(t *testing.T) { }) } -func TestSkipDiffRepro(t *testing.T) { +func TestNeedReproForTitle(t *testing.T) { for title, skip := range map[string]bool{ - "no output from test machine": true, - "SYZFAIL: read failed": true, - "lost connection to test machine": true, - "INFO: rcu detected stall in clone": true, - "WARNING in arch_install_hw_breakpoint": false, - "KASAN: slab-out-of-bounds Write in __bpf_get_stackid": false, + "no output from test machine": false, + "SYZFAIL: read failed": false, + "lost connection to test machine": false, + "INFO: rcu detected stall in clone": false, + "WARNING in arch_install_hw_breakpoint": true, + "KASAN: slab-out-of-bounds Write in __bpf_get_stackid": true, } { - assert.Equal(t, skip, skipDiffRepro(title), "title=%q", title) + assert.Equal(t, skip, needReproForTitle(title), "title=%q", title) } } -- cgit mrf-deployment