summaryrefslogtreecommitdiff
path: root/internal
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2024-09-12 21:55:26 +0300
committerPaul Buetow <paul@buetow.org>2024-09-12 21:55:26 +0300
commitd586966b0ac5d245b8198c4e9dcbb516fa8828f7 (patch)
treed916cd54b0fd15c073a3e54f18e0ea9f1d1ec802 /internal
parentc2671a57fea7457fb24a3d65f5f4a2a1cc65bebe (diff)
fix
Diffstat (limited to 'internal')
-rw-r--r--internal/server/repository/pending.go2
-rw-r--r--internal/server/repository/pending_test.go8
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)
}
}