diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2025-08-05 21:52:15 +0200 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2025-08-06 09:55:41 +0000 |
| commit | 745510422fa0fd508f681b06c98e3a74d41344c5 (patch) | |
| tree | aa2563d9870e90c081bd481a7046b5bc2964bd77 /pkg/manager/diff_test.go | |
| parent | 5ba0fed13435213276f29e3d9e39d926f04ac1a8 (diff) | |
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.
Diffstat (limited to 'pkg/manager/diff_test.go')
| -rw-r--r-- | pkg/manager/diff_test.go | 16 |
1 files changed, 8 insertions, 8 deletions
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) } } |
