| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | * | | | | | Create an AsyncContextManagerMock mock for testing asynchronous context managers |  Akarys42 | 2020-02-09 | -0/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | It can be used to test aiohttp request functions, since they are async context managers | ||||
| * | | | | | | Merge branch 'master' into fuzzy_zero_div |  Numerlor | 2020-02-28 | -7/+7 | |
| |\ \ \ \ \ \ | | |_|/ / / | |/| | | | | | | | | | | | | | | | | # Conflicts: # bot/cogs/error_handler.py | ||||
| | * | | | | | Merge branch 'master' into feat/backend/b496/rename-constants |  Leon Sandøy | 2020-02-28 | -15/+0 | |
| | |\ \ \ \ \ | |||||
| | * | | | | | | Fix mismatched constant names in syncer tests |  S. Co1 | 2020-02-25 | -2/+2 | |
| | | | | | | | | |||||
| | * | | | | | | Merge branch 'master' into feat/backend/b496/rename-constants | S. Co1 | 2020-02-25 | -205/+1197 | |
| | |\ \ \ \ \ \ | | | |_|_|_|/ | | |/| | | | | |||||
| | * | | | | | | Tests: fix setting bot-commands ID in information tests |  MarkKoz | 2020-02-23 | -4/+4 | |
| | | | | | | | | |||||
| | * | | | | | | Constants: rename roles to match their names in the guild | MarkKoz | 2020-02-23 | -1/+1 | |
| | | |_|_|_|/ | |/| | | | | |||||
| * | | | | | | Merge branch 'master' into fuzzy_zero_div |  Sebastiaan Zeeff | 2020-02-27 | -15/+0 | |
| |\ \ \ \ \ \ | | |_|/ / / | |/| | | | | |||||
| | * | | | | | Merge branch 'master' into channel_constants_check |  Numerlor | 2020-02-25 | -205/+1197 | |
| | |\ \ \ \ \ | | | |/ / / | | |/| | | | |||||
| | * | | | | | Remove unused `chunks` function and its tests. | Numerlor | 2020-02-24 | -15/+0 | |
| | | |/ / / | |/| | | | | | | | | | | | | | The function was only used in the since removed `Events` cog. | ||||
| * | | | | | Adjust tests for new converter behavior. | Numerlor | 2020-02-26 | -1/+1 | |
| | |/ / / |/| | | | |||||
| * | | | | Merge remote-tracking branch 'origin/master' into ↵ | MarkKoz | 2020-02-23 | -246/+642 | |
| |\| | | | | | | | | | | | | | | | bug/backend/b704/ready-missing-cache | ||||
| | * | | | Merge branch 'master' into information-refactor |  S. Co1 | 2020-02-23 | -239/+635 | |
| | |\ \ \ | |||||
| | | * \ \ | Merge branch 'master' into unittest-antispam-rules | Sebastiaan Zeeff | 2020-02-21 | -62/+2 | |
| | | |\ \ \ | |||||
| | | | * | | | Remove tests for custom bot log |  Joseph Banks | 2020-02-21 | -62/+2 | |
| | | | | |/ | | | |/| | |||||
| | | * | | | Add unit test for newlines antispam rule |  kwzrd | 2020-02-04 | -0/+105 | |
| | | | | | | |||||
| | | * | | | Add unit test for duplicates antispam rule | kwzrd | 2020-02-04 | -0/+66 | |
| | | | | | | |||||
| | | * | | | Merge branch 'master' into unittest-antispam-rules | Joseph | 2020-02-02 | -1/+8 | |
| | | |\| | | |||||
| | | * | | | Make RuleTest use ABCMeta | kwzrd | 2020-02-02 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | This will prevent child classes to be instantiated unless they implement all abstract methods, leading to a more descriptive error message. | ||||
| | | * | | | Adjust existing tests to inherit from RuleTest ABC | kwzrd | 2020-02-02 | -294/+157 | |
| | | | | | | |||||
| | | * | | | Implement RuleTest ABC | kwzrd | 2020-02-02 | -0/+76 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will serve as an ABC for tests for individual rules. The base class provides runners for allowed and disallowed cases, and the children classes then only provide the cases and implementations of helper methods specific to each rule. | ||||
| | | * | | | Adjust multi-line docstrings to prevailing style | kwzrd | 2020-01-31 | -3/+6 | |
| | | | | | | |||||
| | | * | | | Refactor msg helper function name to make_msg | kwzrd | 2020-01-26 | -25/+25 | |
| | | | | | | | | | | | | | | | | | | | | | The name msg is less descriptive and creates a needless name conflict in local gen exp. | ||||
| | | * | | | Fix incorrect config key in attachments antispam rule | kwzrd | 2020-01-26 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The rule was incorrectly printing out the maximum amount of allowed attachments instead of the configured interval. This commit also adjusts the rule's unit test case. | ||||
| | | * | | | Add unit test case for role mentions antispam rule | kwzrd | 2020-01-26 | -0/+66 | |
| | | | | | | |||||
| | | * | | | Add unit test case for discord emojis antispam rule | kwzrd | 2020-01-26 | -0/+68 | |
| | | | | | | |||||
| | | * | | | Add unit test case for chars antispam rule | kwzrd | 2020-01-26 | -0/+75 | |
| | | | | | | |||||
| | | * | | | Add unit test case for burst shared antispam rule | kwzrd | 2020-01-26 | -0/+65 | |
| | | | | | | |||||
| | | * | | | Add unit test case for burst antispam rule | kwzrd | 2020-01-26 | -0/+69 | |
| | | | |/ | | |/| | |||||
| | * | | | Update the tests accordingly to reflect the new changes |  Deniz | 2020-02-13 | -3/+3 | |
| | | | | | |||||
| | * | | | Fix some of the testing for information.py; I think this should be it. ↵ | Deniz | 2020-02-06 | -2/+2 | |
| | | | | | | | | | | | | | | | | | (hopefully). | ||||
| | * | | | Update tests to reflect status changes | Deniz | 2020-02-06 | -4/+4 | |
| | | |/ | |/| | |||||
| * | | | Sync: add confirmation timeout and max diff to config | MarkKoz | 2020-02-19 | -2/+2 | |
| | | | | |||||
| * | | | Tests: fix unawaited error for MockAPIClient | MarkKoz | 2020-02-12 | -5/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This error is due to the use of an actual instance of APIClient as the spec for the mock. recreate() is called in __init__ which in turn creates a task for the _create_session coroutine. The approach to the solution is to use the type for the spec rather than and instance, thus avoiding any call of __init__. However, without an instance, instance attributes will not be included in the spec. Therefore, they are defined as class attributes on the actual APIClient class definition and given default values. Alternatively, a subclass of APIClient could have been made in the tests.helpers module to define those class attributes. However, it seems easier to maintain if the attributes are in the original class definition. | ||||
| * | | | Sync tests: use async_test decorator | MarkKoz | 2020-02-12 | -94/+135 | |
| | | | | |||||
| * | | | Sync tests: test on_member_join | MarkKoz | 2020-02-12 | -0/+52 | |
| | | | | | | | | | | | | | | | | | | | 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. | ||||
| * | | | Sync tests: add a third role with a lower ID to on_member_update test | MarkKoz | 2020-02-12 | -1/+2 | |
| | | | | | | | | | | | | | This better ensures that roles are being sorted when patching. | ||||
| * | | | Sync tests: fix on_member_remove listener test | MarkKoz | 2020-02-12 | -13/+6 | |
| | | | | | | | | | | | | | | | | The listener was changed earlier to simply set in_guild to False. This commit accounts for that in the test. | ||||
| * | | | Sync tests: assert that listeners are actually added as listeners | MarkKoz | 2020-02-12 | -0/+14 | |
| | | | | |||||
| * | | | Sync tests: test sync commands require the admin permission | MarkKoz | 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 | MarkKoz | 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 | MarkKoz | 2020-02-12 | -0/+7 | |
| | | | | | | | | | | | | | sync() should be called on the UserSyncer. | ||||
| * | | | Sync tests: test sync roles command | MarkKoz | 2020-02-12 | -0/+9 | |
| | | | | | | | | | | | | | sync() should be called on the RoleSyncer. | ||||
| * | | | Sync tests: test Sync cog's on_user_update | MarkKoz | 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 | MarkKoz | 2020-02-12 | -1/+1 | |
| | | | | |||||
| * | | | Sync tests: test Sync cog's on_member_update for other attributes | MarkKoz | 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 | MarkKoz | 2020-02-12 | -0/+11 | |
| | | | | | | | | | | | | | Members should be patched if their roles have changed. | ||||
| * | | | Sync tests: test Sync cog's on_member_remove | MarkKoz | 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 | MarkKoz | 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 | MarkKoz | 2020-02-12 | -19/+4 | |
| | | | | | | | | | | | | | | | | It is obsolete because MockRole now takes care of creating the Colour and Permissions objects. | ||||
 |
index : bot | |
| A mirrored copy of the python-discord/bot repository. |
| aboutsummaryrefslogtreecommitdiffstats |