aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Sync tests: test _send_prompt fetches channel on a cache missGravatar MarkKoz2020-02-12-0/+11
|
* Sync tests: test that _send_prompt gets channel from cacheGravatar MarkKoz2020-02-12-0/+11
|
* Sync tests: test that _send_prompt edits message contentsGravatar MarkKoz2020-02-12-0/+8
|
* Sync tests: create a Syncer subclass for testingGravatar MarkKoz2020-02-12-0/+9
|
* Sync tests: test instantiation fails without abstract methodsGravatar MarkKoz2020-02-12-0/+17
|
* Sync tests: fix wait_until_ready in duck pond testsGravatar MarkKoz2020-02-12-2/+2
|
* Sync tests: fix docstring for UserSyncerSyncTestsGravatar MarkKoz2020-02-12-1/+1
|
* Sync tests: create and use a fake_role fixtureGravatar MarkKoz2020-02-12-33/+31
|
* Sync tests: add tests for API requests for syncing usersGravatar MarkKoz2020-02-12-1/+40
|
* Sync tests: test diff is empty when DB has a user not in the guildGravatar MarkKoz2020-02-12-9/+9
|
* Sync tests: test diff for all 3 changes simultaneouslyGravatar MarkKoz2020-02-12-8/+12
|
* Sync tests: test diff for users which leave the guildGravatar MarkKoz2020-02-12-7/+9
|
* Sync tests: test diff for new usersGravatar MarkKoz2020-02-12-8/+10
|
* Sync tests: test diff for updated usersGravatar MarkKoz2020-02-12-9/+11
|
* Sync tests: fix get_guild modifying the original member dictsGravatar MarkKoz2020-02-12-0/+1
|
* Sync tests: test empty diff for identical usersGravatar MarkKoz2020-02-12-8/+8
|
* Sync tests: work around @everyone role being added by MockMemberGravatar MarkKoz2020-02-12-2/+3
|
* Sync tests: fix mismatched attributes when creating a mock userGravatar MarkKoz2020-02-12-0/+3
|
* Sync tests: fix fake_user fixtureGravatar MarkKoz2020-02-12-7/+9
|
* Sync tests: test empty diff for no usersGravatar MarkKoz2020-02-12-6/+9
|
* Sync tests: add fixture to get a guild with membersGravatar MarkKoz2020-02-12-1/+19
|
* Sync tests: rename user sync test caseGravatar MarkKoz2020-02-12-2/+2
|
* Sync tests: test syncs with multiple rolesGravatar MarkKoz2020-02-12-15/+37
|
* Sync tests: test API requests for role syncingGravatar MarkKoz2020-02-12-1/+34
|
* Sync tests: create separate role test cases for diff and sync testsGravatar MarkKoz2020-02-12-2/+10
|
* Sync tests: remove guild_roles lists and assign roles to variablesGravatar MarkKoz2020-02-12-14/+8
| | | | | Makes the creation of the expected diff clearer since the variable has a name compared to accessing some index of a list.
* Sync tests: remove diff test for updated and new roles togetherGravatar MarkKoz2020-02-12-19/+0
| | | | | Redundant since test_diff_for_new_updated_and_deleted_roles tests all 3 types together.
* Sync tests: test diff for all 3 role changes simultaneouslyGravatar MarkKoz2020-02-12-21/+17
|
* Sync tests: test diff for deleted rolesGravatar MarkKoz2020-02-12-17/+10
|
* Sync tests: test diff for new rolesGravatar MarkKoz2020-02-12-20/+15
|
* Sync tests: create a role in setUp to use as a constantGravatar MarkKoz2020-02-12-6/+5
|
* Sync tests: test diff for updated rolesGravatar MarkKoz2020-02-12-27/+16
|
* Sync tests: test empty diff for identical rolesGravatar MarkKoz2020-02-12-8/+11
|
* Sync tests: fix creation of MockRolesGravatar MarkKoz2020-02-12-3/+4
| | | | Role was being accessed like a class when it is actually a dict.
* Sync tests: add fixture to create a guild with rolesGravatar MarkKoz2020-02-12-1/+21
|
* Sync tests: rename the role syncer test caseGravatar MarkKoz2020-02-12-1/+1
|
* Tests: add a MockAPIClientGravatar MarkKoz2020-02-12-0/+18
|
* Sync: handle API errors gracefullyGravatar MarkKoz2020-02-12-10/+21
| | | | | | | The whole sync is aborted when an error is caught for simplicity's sake. The sync message is edited to display the error and the traceback is logged. To distinguish an error from an abort/timeout, the latter now uses a warning emoji while the former uses the red cross.
* Sync: mention core devs when results are shown & fix missing spaceGravatar MarkKoz2020-02-12-6/+8
|
* Sync: split _confirm() into two functionsGravatar MarkKoz2020-02-12-10/+32
| | | | | | One is responsible for sending the confirmation prompt while the other waits for the reaction. The split allows for the confirmation prompt to be edited with the results of automatic syncs too.
* Sync: keep the mention for all edits of the confirmation promptGravatar MarkKoz2020-02-12-4/+6
| | | | This makes it clearer to users where the notification came from.
* Sync: add trace and debug loggingGravatar MarkKoz2020-02-12-1/+16
|
* Sync: fix confirmation reaction checkGravatar MarkKoz2020-02-12-5/+10
| | | | | | | * Ignore bot reactions * Check for core dev role if sync is automatic * Require author as an argument to _confirm() so it can be compared against the reaction author
* Sync: remove author mention from confirm promptGravatar MarkKoz2020-02-12-2/+2
|
* Sync: fix overwriting message with None after editing itGravatar MarkKoz2020-02-12-1/+1
|
* Sync: fix passing context instead of message to _confirm()Gravatar MarkKoz2020-02-12-4/+5
| | | | * Mention possibility of timing out as a reason for aborting a sync
* Sync: fix missing await for fetch_channelGravatar MarkKoz2020-02-12-1/+1
|
* Sync: allow for None values in DiffsGravatar MarkKoz2020-02-12-4/+4
|
* Sync: make Role, User, and Diff privateGravatar MarkKoz2020-02-12-15/+15
|
* Sync: remove generic type from DiffGravatar MarkKoz2020-02-12-19/+10
| | | | | | | | | | It doesn't play along well with NamedTuple due to metaclass conflicts. The workaround involved created a NamedTuple-only base class, which does work but at the cost of confusing some static type checkers. Since Diff is now an internal data structure, it no longer really needs to have precise type annotations. Therefore, a normal namedtuple is adequate.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-26 17:17:05 +0000