diff options
| author | Paul Buetow <paul@buetow.org> | 2024-09-12 21:55:26 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2024-09-12 21:55:26 +0300 |
| commit | d586966b0ac5d245b8198c4e9dcbb516fa8828f7 (patch) | |
| tree | d916cd54b0fd15c073a3e54f18e0ea9f1d1ec802 /internal/server | |
| parent | c2671a57fea7457fb24a3d65f5f4a2a1cc65bebe (diff) | |
fix
Diffstat (limited to 'internal/server')
| -rw-r--r-- | internal/server/repository/pending.go | 2 | ||||
| -rw-r--r-- | internal/server/repository/pending_test.go | 8 |
2 files changed, 3 insertions, 7 deletions
diff --git a/internal/server/repository/pending.go b/internal/server/repository/pending.go index f03c8fd..bf8a571 100644 --- a/internal/server/repository/pending.go +++ b/internal/server/repository/pending.go @@ -33,5 +33,5 @@ func (p pending) delete(platform types.PlatformName, id types.EntryID) { func (p pending) get(platform types.PlatformName) (pendingEntries, bool) { pe, ok := p.platforms[platform] - return pe, ok + return pe, ok && len(pe) > 0 } diff --git a/internal/server/repository/pending_test.go b/internal/server/repository/pending_test.go index e74771c..44fe84c 100644 --- a/internal/server/repository/pending_test.go +++ b/internal/server/repository/pending_test.go @@ -31,10 +31,6 @@ func TestPendingAdd(t *testing.T) { func TestPendingDelete(t *testing.T) { pending := newPending() - if entries, ok := pending.get(types.LinkedIn); ok { - t.Error("expected not an ok", entries) - } - pending.add(types.LinkedIn, "fooid") entries, ok := pending.get(types.LinkedIn) @@ -46,7 +42,7 @@ func TestPendingDelete(t *testing.T) { } pending.delete(types.LinkedIn, "fooid") - if entries, _ = pending.get(types.LinkedIn); len(entries) > 0 { - t.Error("expected zero results", entries) + if entries, ok = pending.get(types.LinkedIn); ok { + t.Error("expected not an ok", entries) } } |
