From d586966b0ac5d245b8198c4e9dcbb516fa8828f7 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Thu, 12 Sep 2024 21:55:26 +0300 Subject: fix --- internal/server/repository/pending.go | 2 +- internal/server/repository/pending_test.go | 8 ++------ 2 files changed, 3 insertions(+), 7 deletions(-) (limited to 'internal/server') 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) } } -- cgit v1.2.3