diff options
| -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) } } |