bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \| \|	Sync tests: test the return value of _send_prompt	MarkKoz	2020-02-12	-7/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: use channel fixtures with subtests in add reaction test	MarkKoz	2020-02-12	-4/+15
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: use channel fixtures with subtests	MarkKoz	2020-02-12	-25/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Merge test_send_prompt_fetches_channel_if_cache_miss into test_send_prompt_gets_channel_from_cache * Rename test_send_prompt_gets_channel_from_cache * Test test_send_prompt_sends_new_message_if_not_given with fetch_channel too
\| * \| \| \| \| \| \|	Sync tests: rename channel fixtures	MarkKoz	2020-02-12	-2/+2
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: reset mocks in channel fixtures	MarkKoz	2020-02-12	-0/+4
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: add fixtures to mock dev core channel get and fetch	MarkKoz	2020-02-12	-0/+21
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test _send_prompt adds reactions	MarkKoz	2020-02-12	-0/+8
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test _send_prompt sends a new message if one isn't given	MarkKoz	2020-02-12	-0/+11
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test _send_prompt returns None if channel fetch fails	MarkKoz	2020-02-12	-0/+12
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test _send_prompt fetches channel on a cache miss	MarkKoz	2020-02-12	-0/+11
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test that _send_prompt gets channel from cache	MarkKoz	2020-02-12	-0/+11
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test that _send_prompt edits message contents	MarkKoz	2020-02-12	-0/+8
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: create a Syncer subclass for testing	MarkKoz	2020-02-12	-0/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test instantiation fails without abstract methods	MarkKoz	2020-02-12	-0/+17
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix wait_until_ready in duck pond tests	MarkKoz	2020-02-12	-2/+2
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix docstring for UserSyncerSyncTests	MarkKoz	2020-02-12	-1/+1
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: create and use a fake_role fixture	MarkKoz	2020-02-12	-33/+31
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: add tests for API requests for syncing users	MarkKoz	2020-02-12	-1/+40
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff is empty when DB has a user not in the guild	MarkKoz	2020-02-12	-9/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for all 3 changes simultaneously	MarkKoz	2020-02-12	-8/+12
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for users which leave the guild	MarkKoz	2020-02-12	-7/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for new users	MarkKoz	2020-02-12	-8/+10
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for updated users	MarkKoz	2020-02-12	-9/+11
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix get_guild modifying the original member dicts	MarkKoz	2020-02-12	-0/+1
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test empty diff for identical users	MarkKoz	2020-02-12	-8/+8
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: work around @everyone role being added by MockMember	MarkKoz	2020-02-12	-2/+3
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix mismatched attributes when creating a mock user	MarkKoz	2020-02-12	-0/+3
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix fake_user fixture	MarkKoz	2020-02-12	-7/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test empty diff for no users	MarkKoz	2020-02-12	-6/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: add fixture to get a guild with members	MarkKoz	2020-02-12	-1/+19
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: rename user sync test case	MarkKoz	2020-02-12	-2/+2
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test syncs with multiple roles	MarkKoz	2020-02-12	-15/+37
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test API requests for role syncing	MarkKoz	2020-02-12	-1/+34
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: create separate role test cases for diff and sync tests	MarkKoz	2020-02-12	-2/+10
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: remove guild_roles lists and assign roles to variables	MarkKoz	2020-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 together	MarkKoz	2020-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 simultaneously	MarkKoz	2020-02-12	-21/+17
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for deleted roles	MarkKoz	2020-02-12	-17/+10
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for new roles	MarkKoz	2020-02-12	-20/+15
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: create a role in setUp to use as a constant	MarkKoz	2020-02-12	-6/+5
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test diff for updated roles	MarkKoz	2020-02-12	-27/+16
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: test empty diff for identical roles	MarkKoz	2020-02-12	-8/+11
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: fix creation of MockRoles	MarkKoz	2020-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 roles	MarkKoz	2020-02-12	-1/+21
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync tests: rename the role syncer test case	MarkKoz	2020-02-12	-1/+1
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Tests: add a MockAPIClient	MarkKoz	2020-02-12	-0/+18
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync: handle API errors gracefully	MarkKoz	2020-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 space	MarkKoz	2020-02-12	-6/+8
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Sync: split _confirm() into two functions	MarkKoz	2020-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 prompt	MarkKoz	2020-02-12	-4/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This makes it clearer to users where the notification came from.