Geoffery10 / Discord-Role-Manager / 27889600890

90%

push

github

Add conftest and test_idiscord.py — 31 tests for the DB layer

iDiscord.py went from 26% to 99% covered. Total project coverage
climbed from 26% to 48% (53 tests passing, up from 22).

conftest.py
  - isolated_cwd session fixture: chdir into a tmp dir for the whole
    test session and pre-create the discord.db schema so importing
    iDiscord is a no-op against the schema migration.
  - fresh_db function fixture: wipe rows between tests.
  - reload_idiscord_in context-manager fixture: chdir + importlib.reload
    so _ensure_schema() can be re-exercised against legacy / fresh /
    current schemas for migration-branch coverage.
  - birthday_json / last_update_json: pre-stage the JSON files birthday.py
    and update.py open() relative to cwd.

test_idiscord.py
  - Schema migration: 5 tests for the legacy/current/fresh branches of
    _ensure_schema including the guilds PK rebuild with MIN(name) dedupe.
  - connect(): PRAGMA busy_timeout verification.
  - CRUD: add/get/get_users/update/delete for the users table, both
    signatures of update_user.
  - Granular update_user_*/tag/username/birthday/avatar writers.
  - Guild membership: add_user_to_guild idempotency, remove_user_from_guild,
    is_user_in_guild, get_guild_users scoping.
  - Guilds table: add_guild_to_table (INSERT OR REPLACE), remove,
    sync_guilds_table iterating client.guilds.

394 of 402 new or added lines in 2 files covered. (98.01%)

853 of 1770 relevant lines covered (48.19%)

0.48 hits per line