MushroomObserver / mushroom-observer / 27493343625

98%

main: 99%

Pull #4521

github

admin: close remaining coverage gaps on per-file build 80015173

Coveralls per-file check on build 80015173 surfaced three lines still
uncovered after the prior pass:

- `Admin::Users::Controller#update_user_contribution` line 62
  (`contrib -= points` inside the `bonuses&.each_key` subtract loop)
  — the fixture has no prior bonuses, so the subtract branch never
  fired. Added `test_update_subtracts_existing_bonuses`, which
  pre-populates `user_stats.bonuses` with a Hash before posting an
  update.

- `Admin::BlockedIps::Edit::IpStats` line 61 (`plain("bogus!")` in
  the api-key branch) + `IpSummary` line 100 (same `"bogus!"` branch
  for the right-column summary) — exercised by an IP whose
  `:api_key` string doesn't resolve to an APIKey record.

- `Admin::BlockedIps::Edit::IpStats` line 94
  (`"most recent 50 of N requests"` heading in `render_activity_count`)
  — exercised by logging 51 activity entries for one IP before
  rendering its stats panel.

Both new fixture IPs (4.4.4.4 bogus-key, 5.5.5.5 with 51 activity
rows) are added to the existing `test_report_ip_renders_stats_panels`
so the setup/render assertions stay grouped.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

286 of 286 new or added lines in 21 files covered. (100.0%)

46994 of 48085 relevant lines covered (97.73%)

704.09 hits per line