| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Sync tests: test sync commands require the admin permission | 2020-02-12 | -1/+18 | |
| | | | | | | | | | The sync commands should only run if the author has the administrator permission. * Add missing spaces after class docstrings * Add missing docstring to SyncCogCommandTests | |||
| * | Tests: create TestCase subclass with a permissions check assertion | 2020-02-12 | -0/+34 | |
| | | | | | | | The subclass will contain assertions that are useful for testing Discord commands. The currently included assertion tests that a command will raise a MissingPermissions exception if the author lacks permissions. | |||
| * | Sync tests: test sync users command | 2020-02-12 | -0/+7 | |
| | | | | | sync() should be called on the UserSyncer. | |||
| * | Sync tests: test sync roles command | 2020-02-12 | -0/+9 | |
| | | | | | sync() should be called on the RoleSyncer. | |||
| * | Sync tests: test Sync cog's on_user_update | 2020-02-12 | -1/+42 | |
| | | | | | | A user should be patched only if the name, discriminator, or avatar changes. | |||
| * | Sync tests: fix ID in endpoint for test_sync_cog_on_member_remove | 2020-02-12 | -1/+1 | |
| | | ||||
| * | Sync tests: test Sync cog's on_member_update for other attributes | 2020-02-12 | -0/+21 | |
| | | | | | Members should not be patched if other attributes have changed. | |||
| * | Sync tests: test Sync cog's on_member_update for roles | 2020-02-12 | -0/+11 | |
| | | | | | Members should be patched if their roles have changed. | |||
| * | Sync tests: test Sync cog's on_member_remove | 2020-02-12 | -0/+17 | |
| | | | | | | A PUT request should be sent to set in_guild as False and update other fields. | |||
| * | Sync tests: test Sync cog's on_guild_role_update | 2020-02-12 | -0/+35 | |
| | | | | | | A PUT request should be sent if the colour, name, permissions, or position changes. | |||
| * | Sync tests: remove mock_role fixture | 2020-02-12 | -19/+4 | |
| | | | | | | It is obsolete because MockRole now takes care of creating the Colour and Permissions objects. | |||
| * | Tests: use objects for colour and permissions of MockRole | 2020-02-12 | -1/+13 | |
| | | | | | | Instances of discord.Colour and discord.Permissions will be created by default or when ints are given as values for those attributes. | |||
| * | Sync tests: test Sync cog's on_guild_role_delete listener | 2020-02-12 | -0/+7 | |
| | | | | | A DELETE request should be sent. | |||
| * | Sync tests: test Sync cog's on_guild_role_create listener | 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 | 2020-02-12 | -0/+7 | |
| | | ||||
| * | Sync tests: create a base TestCase class for Sync cog tests | 2020-02-12 | -2/+6 | |
| | | ||||
| * | Sync tests: add helper function for testing patch_user | 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 | 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 | 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 | 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 | 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 | 2020-02-12 | -5/+19 | |
| | | | | | * Move patches to setUp | |||
| * | Sync tests: test Sync cog __init__ | 2020-02-12 | -0/+21 | |
| | | | | | Should instantiate syncers and run a sync for the guild. | |||
| * | Sync tests: test the extension setup | 2020-02-12 | -0/+15 | |
| | | | | | The Sync cog should be added. | |||
| * | Sync tests: test _get_confirmation_result for large diffs | 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 | 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 | 2020-02-12 | -4/+12 | |
| | | ||||
| * | Sync tests: test sync redirects confirmation message to given context | 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 | 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 | 2020-02-12 | -0/+15 | |
| | | ||||
| * | Sync tests: test diff size calculation | 2020-02-12 | -0/+22 | |
| | | ||||
| * | Sync tests: adjust sync test to account for _get_confirmation_result | 2020-02-12 | -23/+13 | |
| | | ||||
| * | Sync tests: test sync sends a confirmation prompt | 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 | 2020-02-12 | -1/+25 | |
| | | | | | A confirmation prompt should not be sent if the diff is too small. | |||
| * | Sync tests: test _wait_for_confirmation | 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__ | 2020-02-12 | -0/+1 | |
| | | ||||
| * | Sync tests: add messages to _reaction_check subtests | 2020-02-12 | -4/+9 | |
| | | | | | The message will be displayed by the test runner when a subtest fails. | |||
| * | Sync tests: test _reaction_check for invalid reactions | 2020-02-12 | -0/+43 | |
| | | | | | Should return False for invalid reaction events. | |||
| * | Sync tests: test _reaction_check for valid emoji and authors | 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 | 2020-02-12 | -1/+9 | |
| | | ||||
| * | Sync tests: create a separate test case for _send_prompt tests | 2020-02-12 | -5/+12 | |
| | | ||||
| * | Sync tests: test the return value of _send_prompt | 2020-02-12 | -7/+9 | |
| | | ||||
| * | Sync tests: use channel fixtures with subtests in add reaction test | 2020-02-12 | -4/+15 | |
| | | ||||
| * | Sync tests: use channel fixtures with subtests | 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 | 2020-02-12 | -2/+2 | |
| | | ||||
| * | Sync tests: reset mocks in channel fixtures | 2020-02-12 | -0/+4 | |
| | | ||||
| * | Sync tests: add fixtures to mock dev core channel get and fetch | 2020-02-12 | -0/+21 | |
| | | ||||
| * | Sync tests: test _send_prompt adds reactions | 2020-02-12 | -0/+8 | |
| | | ||||
| * | Sync tests: test _send_prompt sends a new message if one isn't given | 2020-02-12 | -0/+11 | |
| | | ||||
| * | Sync tests: test _send_prompt returns None if channel fetch fails | 2020-02-12 | -0/+12 | |
| | | ||||