|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | The listener was changed earlier to simply set in_guild to False.
This commit accounts for that in the test. | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | The sync commands should only run if the author has the administrator
permission.
* Add missing spaces after class docstrings
* Add missing docstring to SyncCogCommandTests | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | 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() should be called on the UserSyncer. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | sync() should be called on the RoleSyncer. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | A user should be patched only if the name, discriminator, or avatar
changes. | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | Members should not be patched if other attributes have changed. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | Members should be patched if their roles have changed. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | A PUT request should be sent to set in_guild as False and update other
fields. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | A PUT request should be sent if the colour, name, permissions, or
position changes. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | It is obsolete because MockRole now takes care of creating the Colour
and Permissions objects. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | Instances of discord.Colour and discord.Permissions will be created
by default or when ints are given as values for those attributes. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | A DELETE request should be sent. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | A POST request should be sent with the new role's data.
* Add a fixture to create a MockRole | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | Reduces redundancy in the tests by taking care of the mocks, calling of
the function, and the assertion. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | 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 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | Roles and users should be synced only if a guild is successfully
retrieved. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | While on 3.7, the CustomMockMixin needs to be leveraged so that
coroutine members are replace with AsyncMocks instead. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | This prevents persistence of changes to the cog instance; sync_guild
would otherwise remain as a mock object for any subsequent tests. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | * Move patches to setUp | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | Should instantiate syncers and run a sync for the guild. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | The Sync cog should be added. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | Should return True if confirmed and False if _send_prompt fails or
aborted. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | Should always return True and the given message if the diff size is too
small. | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | 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. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | Author should be the bot or the ctx author, if a ctx is given. | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | 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. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | A confirmation prompt should not be sent if the diff is too small. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | The message should always be edited and only return True if the emoji is
a check mark. | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | The message will be displayed by the test runner when a subtest fails. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | | Should return False for invalid reaction events. | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | 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 | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | | * 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 | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | |  | 
| | | | | | |  |