dev-shimada / discord-rss-bot

84%

Build Type

push

github

Committed by dev-shimada

Commit Message

Fix crash and correctness bugs found in code review

- Fall back to nil PublishedParsed/UpdatedParsed instead of panicking on
  feed items without a parseable date (crashed the whole process)
- Bail out cleanly instead of panicking when DB init fails (main.go,
  database.CloseDB)
- Keep the background poll loop alive after a transient FindAll error
  instead of returning and killing it permanently
- Use log/slog instead of the unconfigured golang.org/x/exp/slog in
  usecase/subscription.go
- RssEntryPersistence.Find now filters by entry link instead of loading
  the entire table every tick
- Fetch each subscribed URL once per tick instead of once per channel
- Surface List() errors to the user instead of silently showing an
  empty list

Adds test coverage for the date-fallback logic, the once-per-URL fetch
behavior, and the Find filtering.

Coverage Stats

40 of 43 new or added lines in 2 files covered. (93.02%)

4 existing lines in 1 file now uncovered.

162 of 194 relevant lines covered (83.51%)

3.94 hits per line