Commit message (Collapse) | Author | Age | Lines | ||
---|---|---|---|---|---|
... | |||||
| * | Cog tests: add a test for duplicate command names & aliases | 2020-03-13 | -0/+17 | ||
| | | |||||
| * | Cog tests: fix duplicate commands being yielded | 2020-03-13 | -1/+2 | ||
| | | | | | | | | | | discord.py yields duplicate Command objects for each alias a command has, so the duplicates need to be removed on our end. | ||||
| * | Cog tests: fix nested modules not being found | 2020-03-13 | -9/+13 | ||
| | | | | | | | | | | * Rename `walk_extensions` to `walk_modules` because some extensions don't consist of a single module | ||||
| * | Cog tests: fix duplicate cogs being yielded | 2020-03-13 | -1/+2 | ||
| | | | | | | | | Have to check the modules are equal to prevent yielding imported cogs. | ||||
| * | Cog tests: fix Cog type check in `walk_cogs` | 2020-03-13 | -3/+3 | ||
| | | |||||
| * | Cog tests: add a function to yield all commands | 2020-03-13 | -0/+7 | ||
| | | | | | | | | This will help reduce nesting in the actual test. | ||||
| * | Cog tests: add a function to get all qualified names for a cmd | 2020-03-13 | -0/+8 | ||
| | | |||||
| * | Cog tests: add a function to get all cogs | 2020-03-13 | -0/+7 | ||
| | | |||||
| * | Cog tests: add a function to get all extensions | 2020-03-13 | -0/+11 | ||
| | | |||||
| * | Cog tests: add a function to get all commands | 2020-03-09 | -0/+12 | ||
| | | | | | | | | | | | | | | | | For tests, ideally creating instances of cogs should be avoided to avoid extra code execution. This function was copied over from discord.py because their function is not a static method, though it still works as one. It was probably just a design decision on their part to not make it static. | ||||
| * | Cog tests: create boilerplate for command name tests | 2020-03-09 | -0/+7 | ||
| | | |||||
| * | Merge branch 'master' into roles-command-pagination | 2020-03-05 | -48/+11 | ||
| |\ | |||||
| | * | Update code for pep8-naming compliance | 2020-03-03 | -6/+6 | ||
| | | | |||||
| | * | Remove CaseInsensitiveDict | 2020-03-03 | -37/+0 | ||
| | | | | | | | | | This was added by the now-removed Snake cog & is not used elsewhere on bot. | ||||
| | * | Add more pre-commit hooks | 2020-03-02 | -5/+5 | ||
| | | | | | | | | | | | | | | | * Remove trailing whitespaces * Specify error code for a noqa in the free command | ||||
| * | | (Information Cog, !roles command test): Removed 's' at end of "Total 1 ↵ | 2020-03-05 | -1/+1 | ||
| | | | | | | | | | | | | role(s)" due changes in command. | ||||
| * | | (Information Cog, !roles command test): Applied empty parameter change. | 2020-03-05 | -1/+1 | ||
| | | | |||||
| * | | Merge branch 'master' into roles-command-pagination | 2020-03-03 | -512/+196 | ||
| |\| | |||||
| * | | !roles Command Test: Applied !roles command changes | 2020-03-01 | -3/+2 | ||
| | | | |||||
* | | | Add tests for `HushDurationConverter`. | 2020-03-17 | -0/+30 | ||
| | | | |||||
* | | | Add tests ensuring permissions get preserved. | 2020-03-17 | -0/+31 | ||
| | | | |||||
* | | | Use set `discard` instead of `remove`. | 2020-03-17 | -1/+1 | ||
| | | | | | | | | | | | | | | | Discard ignores non present values, allowing us to skip the KeyError suppress. | ||||
* | | | Add docstring to test. | 2020-03-17 | -0/+1 | ||
| | | | |||||
* | | | Fix test name. | 2020-03-17 | -1/+1 | ||
| | | | | | | | | | | | | `removed` was describing the opposite behaviour. | ||||
* | | | Remove `alert_channel` mention from docstring. | 2020-03-17 | -1/+1 | ||
| | | | | | | | | | | | | | | | After removing the optional channel arg and changing output message channels we're only testing `ctx`'s `send`. | ||||
* | | | Remove one indentation level. | 2020-03-17 | -3/+3 | ||
| | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]> | ||||
* | | | Rename `cog_unload` tests. | 2020-03-17 | -2/+2 | ||
| | | | | | | | | | | | | Previous names were undescriptive from testing phases. | ||||
* | | | Fix `test_cog_unload` passing tests with invalid values. | 2020-03-17 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | The first assert - `asyncio_mock.create_task.assert_called_once_with` called `alert_channel`'s send resulting in an extra call. `send` on `alert_channel` was not tested properly because of a typo and a missing assert in the method call. | ||||
* | | | Set `_get_instance_vars_event` in test's `setUp`. | 2020-03-15 | -0/+2 | ||
| | | | |||||
* | | | Adjust tests for new calling behaviour. | 2020-03-15 | -2/+2 | ||
| | | | | | | | | | | | | | | | `.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests. | ||||
* | | | Use dict instead of a set and custom class. | 2020-03-15 | -25/+3 | ||
| | | | | | | | | | | | | | | | The FirstHash class is no longer necessary with only channels and the current loop in tuples. FirstHash was removed, along with its tests and tests were adjusted for new dict behaviour. | ||||
* | | | Add tests for `_notifier`. | 2020-03-11 | -0/+19 | ||
| | | | |||||
* | | | Add tests for `remove_channel`. | 2020-03-11 | -0/+18 | ||
| | | | |||||
* | | | Remove `_current_loop` from setup. | 2020-03-11 | -2/+1 | ||
| | | | |||||
* | | | Add tests for `add_channel`. | 2020-03-11 | -0/+21 | ||
| | | | |||||
* | | | Create test case for `SilenceNotifier` | 2020-03-11 | -1/+7 | ||
| | | | |||||
* | | | Add tests for `_get_instance_vars`. | 2020-03-11 | -1/+28 | ||
| | | | |||||
* | | | Pass created channel instead of new object. | 2020-03-11 | -1/+1 | ||
| | | | | | | | | | | | | Creating a new object caused the assert to fail because different objects were used. | ||||
* | | | Reset `self.ctx` call history after every subtest. | 2020-03-11 | -0/+1 | ||
| | | | |||||
* | | | Fix erroneous `assert_called_once_with` calls. | 2020-03-11 | -8/+7 | ||
| | | | | | | | | | | | | `assert_called_once_with` was being tested on call_args which always reported success.st. | ||||
* | | | Add test for `cog_check`. | 2020-03-11 | -0/+7 | ||
| | | | |||||
* | | | Add test for `cog_unload`. | 2020-03-11 | -1/+17 | ||
| | | | |||||
* | | | Use patch decorators and assign names from `with` patches. | 2020-03-11 | -11/+11 | ||
| | | | |||||
* | | | Separate tests for permissions and `muted_channels.add` on `_silence`. | 2020-03-11 | -5/+8 | ||
| | | | |||||
* | | | Add tests for `_unsilence` | 2020-03-11 | -1/+33 | ||
| | | | |||||
* | | | Use async test case. | 2020-03-11 | -14/+12 | ||
| | | | | | | | | | | | | This allows us to use coroutines with await directly instead of asyncio.run | ||||
* | | | Add test for `_silence` method. | 2020-03-11 | -1/+34 | ||
| | | | |||||
* | | | Remove channel mentions from output discord messages. | 2020-03-11 | -4/+4 | ||
| | | | | | | | | | | | | | | | | | | With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly | ||||
* | | | Remove unnecessary args from test cases. | 2020-03-11 | -6/+6 | ||
| | | | | | | | | | | | | | | | Needless call args which were constant were kept in the test cases, resulting in redundant code, the args were moved directly into the function call. | ||||
* | | | Use `patch.object` instead of patch with direct `return_value`. | 2020-03-09 | -9/+2 | ||
| | | |