aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
...
| * Cog tests: add a test for duplicate command names & aliasesGravatar MarkKoz2020-03-13-0/+17
| |
| * Cog tests: fix duplicate commands being yieldedGravatar MarkKoz2020-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 foundGravatar MarkKoz2020-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 yieldedGravatar MarkKoz2020-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`Gravatar MarkKoz2020-03-13-3/+3
| |
| * Cog tests: add a function to yield all commandsGravatar MarkKoz2020-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 cmdGravatar MarkKoz2020-03-13-0/+8
| |
| * Cog tests: add a function to get all cogsGravatar MarkKoz2020-03-13-0/+7
| |
| * Cog tests: add a function to get all extensionsGravatar MarkKoz2020-03-13-0/+11
| |
| * Cog tests: add a function to get all commandsGravatar MarkKoz2020-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 testsGravatar MarkKoz2020-03-09-0/+7
| |
| * Merge branch 'master' into roles-command-paginationGravatar scragly2020-03-05-48/+11
| |\
| | * Update code for pep8-naming complianceGravatar S. Co12020-03-03-6/+6
| | |
| | * Remove CaseInsensitiveDictGravatar S. Co12020-03-03-37/+0
| | | | | | | | | This was added by the now-removed Snake cog & is not used elsewhere on bot.
| | * Add more pre-commit hooksGravatar MarkKoz2020-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 ↵Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | | role(s)" due changes in command.
| * | (Information Cog, !roles command test): Applied empty parameter change.Gravatar ks1232020-03-05-1/+1
| | |
| * | Merge branch 'master' into roles-command-paginationGravatar S. Co12020-03-03-512/+196
| |\|
| * | !roles Command Test: Applied !roles command changesGravatar Karlis. S2020-03-01-3/+2
| | |
* | | Add tests for `HushDurationConverter`.Gravatar Numerlor2020-03-17-0/+30
| | |
* | | Add tests ensuring permissions get preserved.Gravatar Numerlor2020-03-17-0/+31
| | |
* | | Use set `discard` instead of `remove`.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | Discard ignores non present values, allowing us to skip the KeyError suppress.
* | | Add docstring to test.Gravatar Numerlor2020-03-17-0/+1
| | |
* | | Fix test name.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | `removed` was describing the opposite behaviour.
* | | Remove `alert_channel` mention from docstring.Gravatar Numerlor2020-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.Gravatar Numerlor2020-03-17-3/+3
| | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
* | | Rename `cog_unload` tests.Gravatar Numerlor2020-03-17-2/+2
| | | | | | | | | | | | Previous names were undescriptive from testing phases.
* | | Fix `test_cog_unload` passing tests with invalid values.Gravatar Numerlor2020-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`.Gravatar Numerlor2020-03-15-0/+2
| | |
* | | Adjust tests for new calling behaviour.Gravatar Numerlor2020-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.Gravatar Numerlor2020-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`.Gravatar Numerlor2020-03-11-0/+19
| | |
* | | Add tests for `remove_channel`.Gravatar Numerlor2020-03-11-0/+18
| | |
* | | Remove `_current_loop` from setup.Gravatar Numerlor2020-03-11-2/+1
| | |
* | | Add tests for `add_channel`.Gravatar Numerlor2020-03-11-0/+21
| | |
* | | Create test case for `SilenceNotifier`Gravatar Numerlor2020-03-11-1/+7
| | |
* | | Add tests for `_get_instance_vars`.Gravatar Numerlor2020-03-11-1/+28
| | |
* | | Pass created channel instead of new object.Gravatar Numerlor2020-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.Gravatar Numerlor2020-03-11-0/+1
| | |
* | | Fix erroneous `assert_called_once_with` calls.Gravatar Numerlor2020-03-11-8/+7
| | | | | | | | | | | | `assert_called_once_with` was being tested on call_args which always reported success.st.
* | | Add test for `cog_check`.Gravatar Numerlor2020-03-11-0/+7
| | |
* | | Add test for `cog_unload`.Gravatar Numerlor2020-03-11-1/+17
| | |
* | | Use patch decorators and assign names from `with` patches.Gravatar Numerlor2020-03-11-11/+11
| | |
* | | Separate tests for permissions and `muted_channels.add` on `_silence`.Gravatar Numerlor2020-03-11-5/+8
| | |
* | | Add tests for `_unsilence`Gravatar Numerlor2020-03-11-1/+33
| | |
* | | Use async test case.Gravatar Numerlor2020-03-11-14/+12
| | | | | | | | | | | | This allows us to use coroutines with await directly instead of asyncio.run
* | | Add test for `_silence` method.Gravatar Numerlor2020-03-11-1/+34
| | |
* | | Remove channel mentions from output discord messages.Gravatar Numerlor2020-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.Gravatar Numerlor2020-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`.Gravatar Numerlor2020-03-09-9/+2
| | |