| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | 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: ignore coverage for abstract methods | 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 | 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: create a separate function to get the confirmation result | 2020-02-12 | -9/+32 | |
| | | ||||
| * | 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: fix precedence of conditional expression in _reaction_check | 2020-02-12 | -1/+1 | |
| | | ||||
| * | 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: make the reaction check an instance method instead of nested | 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 | 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 | |
| | | ||||
| * | Sync tests: test _send_prompt fetches channel on a cache miss | 2020-02-12 | -0/+11 | |
| | | ||||
| * | Sync tests: test that _send_prompt gets channel from cache | 2020-02-12 | -0/+11 | |
| | | ||||
| * | Sync tests: test that _send_prompt edits message contents | 2020-02-12 | -0/+8 | |
| | | ||||
| * | Sync tests: create a Syncer subclass for testing | 2020-02-12 | -0/+9 | |
| | | ||||
| * | Sync tests: test instantiation fails without abstract methods | 2020-02-12 | -0/+17 | |
| | | ||||
| * | Sync tests: fix wait_until_ready in duck pond tests | 2020-02-12 | -2/+2 | |
| | | ||||
| * | Sync tests: fix docstring for UserSyncerSyncTests | 2020-02-12 | -1/+1 | |
| | | ||||
| * | Sync tests: create and use a fake_role fixture | 2020-02-12 | -33/+31 | |
| | | ||||