| Commit message (Collapse) | Author | Age | Lines |
| |
|
|
|
|
|
|
| |
Should PUT user's data or POST it if the user doesn't exist.
ResponseCodeError should be re-raised if status code isn't a 404.
A helper method was added to reduce code redundancy between the 2 tests.
|
|
|
|
| |
This better ensures that roles are being sorted when patching.
|
|
|
|
|
| |
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
|
| |
|