Geoffery10 / Discord-Role-Manager / 27889998957

90%

push

github

Add test_update.py — update.py now at 98% coverage

update.py went from 0% to 98%. 13 new tests cover the daily-task
runner plus its per-member sync logic.

update_database branches:
  - adds new user when member has no DB row
  - updates discriminator when stale (Discord username migration)
  - updates username when stale
  - updates avatar when stale
  - handles member.avatar = None without crashing
  - is a no-op when member is fully synced
  - links to new guild without duplicating existing rows
  - handles empty members list

update() orchestrator:
  - iterates each guild, calls check_birthday once per guild,
    spawns update_database tasks and gathers them
  - empty guild list path

check_if_update_needed() gate:
  - runs update() and overwrites last_update.json when day changed
  - skips update() when last_update.json matches today
  - overwrites last_update.json BEFORE running update() so a crash
    inside update() doesn't loop infinitely on next startup

The one uncovered line (49) is dead code: a hardcoded
'DEBUG = False' inside check_if_update_needed makes the else branch
unreachable without modifying source. Worth flagging for cleanup
in a follow-up.

165 of 165 new or added lines in 1 file covered. (100.0%)

1475 of 2276 relevant lines covered (64.81%)

0.65 hits per line