aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Sync tests: test Sync cog's on_guild_role_delete listenerGravatar MarkKoz2020-02-12-0/+7
| | | | A DELETE request should be sent.
* Sync tests: test Sync cog's on_guild_role_create listenerGravatar MarkKoz2020-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 testsGravatar MarkKoz2020-02-12-0/+7
|
* Sync tests: create a base TestCase class for Sync cog testsGravatar MarkKoz2020-02-12-2/+6
|
* Sync tests: add helper function for testing patch_userGravatar MarkKoz2020-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_userGravatar MarkKoz2020-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_guildGravatar MarkKoz2020-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 methodsGravatar MarkKoz2020-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_guildGravatar MarkKoz2020-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 setUpGravatar MarkKoz2020-02-12-5/+19
| | | | * Move patches to setUp
* Sync tests: test Sync cog __init__Gravatar MarkKoz2020-02-12-0/+21
| | | | Should instantiate syncers and run a sync for the guild.
* Sync tests: test the extension setupGravatar MarkKoz2020-02-12-0/+15
| | | | The Sync cog should be added.
* Sync tests: ignore coverage for abstract methodsGravatar MarkKoz2020-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 diffsGravatar MarkKoz2020-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 diffsGravatar MarkKoz2020-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 errorGravatar MarkKoz2020-02-12-4/+12
|
* Sync tests: test sync redirects confirmation message to given contextGravatar MarkKoz2020-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 confirmationGravatar MarkKoz2020-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 sentGravatar MarkKoz2020-02-12-0/+15
|
* Sync tests: test diff size calculationGravatar MarkKoz2020-02-12-0/+22
|
* Sync tests: adjust sync test to account for _get_confirmation_resultGravatar MarkKoz2020-02-12-23/+13
|
* Sync: create a separate function to get the confirmation resultGravatar MarkKoz2020-02-12-9/+32
|
* Sync tests: test sync sends a confirmation promptGravatar MarkKoz2020-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 diffGravatar MarkKoz2020-02-12-1/+25
| | | | A confirmation prompt should not be sent if the diff is too small.
* Sync tests: test _wait_for_confirmationGravatar MarkKoz2020-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__Gravatar MarkKoz2020-02-12-0/+1
|
* Sync tests: add messages to _reaction_check subtestsGravatar MarkKoz2020-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_checkGravatar MarkKoz2020-02-12-1/+1
|
* Sync tests: test _reaction_check for invalid reactionsGravatar MarkKoz2020-02-12-0/+43
| | | | Should return False for invalid reaction events.
* Sync tests: test _reaction_check for valid emoji and authorsGravatar MarkKoz2020-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 testsGravatar MarkKoz2020-02-12-1/+9
|
* Sync: make the reaction check an instance method instead of nestedGravatar MarkKoz2020-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 testsGravatar MarkKoz2020-02-12-5/+12
|
* Sync tests: test the return value of _send_promptGravatar MarkKoz2020-02-12-7/+9
|
* Sync tests: use channel fixtures with subtests in add reaction testGravatar MarkKoz2020-02-12-4/+15
|
* Sync tests: use channel fixtures with subtestsGravatar MarkKoz2020-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 fixturesGravatar MarkKoz2020-02-12-2/+2
|
* Sync tests: reset mocks in channel fixturesGravatar MarkKoz2020-02-12-0/+4
|
* Sync tests: add fixtures to mock dev core channel get and fetchGravatar MarkKoz2020-02-12-0/+21
|
* Sync tests: test _send_prompt adds reactionsGravatar MarkKoz2020-02-12-0/+8
|
* Sync tests: test _send_prompt sends a new message if one isn't givenGravatar MarkKoz2020-02-12-0/+11
|
* Sync tests: test _send_prompt returns None if channel fetch failsGravatar MarkKoz2020-02-12-0/+12
|
* 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
|