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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \|	Sync tests: test Sync cog's on_guild_role_delete listener	MarkKoz	2020-02-12	-0/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A DELETE request should be sent.
\| \| * \| \| \| \| \| \|	Sync tests: test Sync cog's on_guild_role_create listener	MarkKoz	2020-02-12	-0/+29
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A POST request should be sent with the new role's data. * Add a fixture to create a MockRole
\| \| * \| \| \| \| \| \|	Sync tests: create a test case for listener tests	MarkKoz	2020-02-12	-0/+7
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: create a base TestCase class for Sync cog tests	MarkKoz	2020-02-12	-2/+6
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: add helper function for testing patch_user	MarkKoz	2020-02-12	-8/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Reduces redundancy in the tests by taking care of the mocks, calling of the function, and the assertion.
\| \| * \| \| \| \| \| \|	Sync tests: test patch_user	MarkKoz	2020-02-12	-0/+26
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A PATCH request should be sent. The error should only be raised if it is not a 404. * Add a fixture to create ResponseCodeErrors with a specific status
\| \| * \| \| \| \| \| \|	Sync tests: test sync_guild	MarkKoz	2020-02-12	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Roles and users should be synced only if a guild is successfully retrieved.
\| \| * \| \| \| \| \| \|	Sync tests: fix Syncer mocks not having async methods	MarkKoz	2020-02-12	-2/+23
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	While on 3.7, the CustomMockMixin needs to be leveraged so that coroutine members are replace with AsyncMocks instead.
\| \| * \| \| \| \| \| \|	Sync tests: use mock.patch for sync_guild	MarkKoz	2020-02-12	-2/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This prevents persistence of changes to the cog instance; sync_guild would otherwise remain as a mock object for any subsequent tests.
\| \| * \| \| \| \| \| \|	Sync tests: instantiate a Sync cog in setUp	MarkKoz	2020-02-12	-5/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Move patches to setUp
\| \| * \| \| \| \| \| \|	Sync tests: test Sync cog __init__	MarkKoz	2020-02-12	-0/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should instantiate syncers and run a sync for the guild.
\| \| * \| \| \| \| \| \|	Sync tests: test the extension setup	MarkKoz	2020-02-12	-0/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The Sync cog should be added.
\| \| * \| \| \| \| \| \|	Sync tests: ignore coverage for abstract methods	MarkKoz	2020-02-12	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's impossible to create an instance of the base class which does not have the abstract methods implemented, so it doesn't really matter what they do.
\| \| * \| \| \| \| \| \|	Sync tests: test _get_confirmation_result for large diffs	MarkKoz	2020-02-12	-0/+29
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should return True if confirmed and False if _send_prompt fails or aborted.
\| \| * \| \| \| \| \| \|	Sync tests: test _get_confirmation_result for small diffs	MarkKoz	2020-02-12	-0/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should always return True and the given message if the diff size is too small.
\| \| * \| \| \| \| \| \|	Sync tests: test sync edits message even if there's an API error	MarkKoz	2020-02-12	-4/+12
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: test sync redirects confirmation message to given context	MarkKoz	2020-02-12	-6/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If ctx is given, a new message should be sent and author should be ctx's author. test_sync_confirmation_author was re-worked to include a test for the message being sent and passed.
\| \| * \| \| \| \| \| \|	Sync tests: test sync passes correct author for confirmation	MarkKoz	2020-02-12	-1/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Author should be the bot or the ctx author, if a ctx is given.
\| \| * \| \| \| \| \| \|	Sync tests: test sync edits the message if one was sent	MarkKoz	2020-02-12	-0/+15
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: test diff size calculation	MarkKoz	2020-02-12	-0/+22
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: adjust sync test to account for _get_confirmation_result	MarkKoz	2020-02-12	-23/+13
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync: create a separate function to get the confirmation result	MarkKoz	2020-02-12	-9/+32
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: test sync sends a confirmation prompt	MarkKoz	2020-02-12	-12/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The prompt should be sent only if the diff is large and should fail if not confirmed. The empty diff test was integrated into this new test.
\| \| * \| \| \| \| \| \|	Sync tests: test sync with an empty diff	MarkKoz	2020-02-12	-1/+25
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A confirmation prompt should not be sent if the diff is too small.
\| \| * \| \| \| \| \| \|	Sync tests: test _wait_for_confirmation	MarkKoz	2020-02-12	-0/+38
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The message should always be edited and only return True if the emoji is a check mark.
\| \| * \| \| \| \| \| \|	Tests: add a return value for MockReaction.__str__	MarkKoz	2020-02-12	-0/+1
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: add messages to _reaction_check subtests	MarkKoz	2020-02-12	-4/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The message will be displayed by the test runner when a subtest fails.
\| \| * \| \| \| \| \| \|	Sync: fix precedence of conditional expression in _reaction_check	MarkKoz	2020-02-12	-1/+1
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync tests: test _reaction_check for invalid reactions	MarkKoz	2020-02-12	-0/+43
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should return False for invalid reaction events.
\| \| * \| \| \| \| \| \|	Sync tests: test _reaction_check for valid emoji and authors	MarkKoz	2020-02-12	-0/+27
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should return True if authors are identical or are a bot and a core dev, respectively. * Create a mock core dev role in the setup fixture * Create a fixture to create a mock message and reaction from an emoji
\| \| * \| \| \| \| \| \|	Sync tests: create a test suite for confirmation tests	MarkKoz	2020-02-12	-1/+9
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Sync: make the reaction check an instance method instead of nested	MarkKoz	2020-02-12	-13/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The function will be easier to test if it's separate rather than nested.
\| \| * \| \| \| \| \| \|	Sync tests: create a separate test case for _send_prompt tests	MarkKoz	2020-02-12	-5/+12
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	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
\| \| \| \| \| \| \| \| \|