diff options
| author | Paul Buetow <paul@buetow.org> | 2026-06-23 06:40:44 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-06-23 06:40:44 +0300 |
| commit | f75924987a1685055d4eb4517a0f2443c6f7d6be (patch) | |
| tree | bc4712b81fbee8fd7fe4b3a82b95b4df8c03adec /internal/ui/table_test.go | |
| parent | c525eb4c200a1bc20a8d866efaa3eaa3ce564376 (diff) | |
Reject typed nil Taskwarrior injection for vq0
Diffstat (limited to 'internal/ui/table_test.go')
| -rw-r--r-- | internal/ui/table_test.go | 22 |
1 files changed, 17 insertions, 5 deletions
diff --git a/internal/ui/table_test.go b/internal/ui/table_test.go index a9348ba..14e78ca 100644 --- a/internal/ui/table_test.go +++ b/internal/ui/table_test.go @@ -188,12 +188,24 @@ func TestNewWithTaskwarriorUsesFakeForAddTask(t *testing.T) { } func TestNewWithTaskwarriorRejectsNilClient(t *testing.T) { - _, err := NewWithTaskwarrior(nil, "firefox", nil) - if err == nil { - t.Fatalf("NewWithTaskwarrior with nil client succeeded") + tests := []struct { + name string + tw task.Taskwarrior + }{ + {name: "plain nil", tw: nil}, + {name: "typed nil", tw: (*fakeTaskwarrior)(nil)}, } - if got, want := err.Error(), "taskwarrior client is nil"; got != want { - t.Fatalf("NewWithTaskwarrior error = %q, want %q", got, want) + + for _, tc := range tests { + t.Run(tc.name, func(t *testing.T) { + _, err := NewWithTaskwarrior(nil, "firefox", tc.tw) + if err == nil { + t.Fatalf("NewWithTaskwarrior with nil client succeeded") + } + if got, want := err.Error(), "taskwarrior client is nil"; got != want { + t.Fatalf("NewWithTaskwarrior error = %q, want %q", got, want) + } + }) } } |
