feat(poller): skip finished series, keep polling archived ones

backend/store_test.go

@@ -838,3 +838,37 @@ func TestMigrationAddsStatusToLegacyDatabase(t *testing.T) {
 		t.Fatalf("Status = %q, want %q", b.Status, statusReading)
 	}
 }
+
+// Archiving is the reason to keep polling — the point is to come back to a
+// series that has moved on. A finished series has nothing left to publish.
+func TestDueForLatestCheckSkipsFinishedKeepsArchived(t *testing.T) {
+	store := newTestStore(t)
+	for _, tc := range []struct{ key, status string }{
+		{"asura:reading", statusReading},
+		{"asura:archived", statusArchived},
+		{"asura:finished", statusFinished},
+	} {
+		if _, err := store.Upsert(Bookmark{
+			Key: tc.key, Site: "asura", SeriesID: tc.key,
+			SeriesURL: "https://asurascans.com/comics/" + tc.key,
+			Status:    tc.status, UpdatedAt: time.Now().UnixMilli(),
+		}); err != nil {
+			t.Fatalf("seed %s: %v", tc.key, err)
+		}
+	}
+
+	due, err := store.DueForLatestCheck(time.Now().UnixMilli(), 10)
+	if err != nil {
+		t.Fatalf("DueForLatestCheck: %v", err)
+	}
+	got := map[string]bool{}
+	for _, b := range due {
+		got[b.Key] = true
+	}
+	if !got["asura:reading"] || !got["asura:archived"] {
+		t.Fatalf("due = %v, want reading and archived present", got)
+	}
+	if got["asura:finished"] {
+		t.Fatal("due included a finished series")
+	}
+}