| Commit message (Collapse) | Author | Lines | ||
|---|---|---|---|---|
| 2020-03-15 | Document returns values of private methods. | -0/+2 | ||
| 2020-03-15 | Block commands until all instance vars are loaded. | -0/+4 | ||
| 2020-03-15 | Assign created task to a var. | -1/+1 | ||
| 2020-03-15 | Pass kwargs directly instead of a PermissionOverwrite. | -9/+3 | ||
| The `set_permissions` method creates a `PermissionOverwrite` from kwargs internally, so we can skip creating it ourselves and unpack the dict directly into kwargs. | ||||
| 2020-03-15 | Change BadArgument error wording. | -1/+1 | ||
| Co-authored-by: MarkKoz <[email protected]> | ||||
| 2020-03-15 | Add Silence cog load to docstring. | -1/+1 | ||
| 2020-03-15 | Move coro execution outside of if condition. | -8/+6 | ||
| This gives us a clearer look at the general flow control and what's getting executed. Comment was also moved to its relevant line. Co-authored-by: MarkKoz <[email protected]> | ||||
| 2020-03-15 | Adjust docstring styling. | -2/+2 | ||
| Co-authored-by: MarkKoz <[email protected]> | ||||
| 2020-03-15 | Use dict instead of a set and custom class. | -44/+8 | ||
| 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. | ||||
| 2020-03-11 | Add tests for `_notifier`. | -0/+19 | ||
| 2020-03-11 | Improve commands help. | -3/+5 | ||
| 2020-03-11 | Add logging to commands. | -0/+3 | ||
| 2020-03-11 | Change various logging levels. | -7/+7 | ||
| 2020-03-11 | Add tests for `remove_channel`. | -0/+18 | ||
| 2020-03-11 | Remove `_current_loop` from setup. | -2/+1 | ||
| 2020-03-11 | Add tests for `add_channel`. | -0/+21 | ||
| 2020-03-11 | Create test case for `SilenceNotifier` | -1/+7 | ||
| 2020-03-11 | Add tests for `_get_instance_vars`. | -1/+28 | ||
| 2020-03-11 | Pass created channel instead of new object. | -1/+1 | ||
| Creating a new object caused the assert to fail because different objects were used. | ||||
| 2020-03-11 | Reset `self.ctx` call history after every subtest. | -0/+1 | ||
| 2020-03-11 | Fix erroneous `assert_called_once_with` calls. | -8/+7 | ||
| `assert_called_once_with` was being tested on call_args which always reported success.st. | ||||
| 2020-03-11 | Add test for `cog_check`. | -0/+7 | ||
| 2020-03-11 | Add test for `cog_unload`. | -1/+17 | ||
| 2020-03-11 | Use patch decorators and assign names from `with` patches. | -11/+11 | ||
| 2020-03-11 | Separate tests for permissions and `muted_channels.add` on `_silence`. | -5/+8 | ||
| 2020-03-11 | Add tests for `_unsilence` | -1/+33 | ||
| 2020-03-11 | Use async test case. | -14/+12 | ||
| This allows us to use coroutines with await directly instead of asyncio.run | ||||
| 2020-03-11 | Add test for `_silence` method. | -1/+34 | ||
| 2020-03-11 | Move adding of channel to `muted_channels` up. | -1/+1 | ||
| Before the channel was not added if `persistent` was `True`. | ||||
| 2020-03-11 | Remove channel mentions from output discord messages. | -8/+8 | ||
| With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly | ||||
| 2020-03-11 | Remove unnecessary args from test cases. | -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. | ||||
| 2020-03-10 | Add alert with silenced channels on `cog_unload`. | -0/+11 | ||
| 2020-03-10 | Remove `channel` arg from commands. | -20/+10 | ||
| 2020-03-10 | Retain previous channel overwrites. | -4/+12 | ||
| Previously silencing a channel reset all overwrites excluding `send_messages` and unsilencing them removed all overwrites. This is prevented by getting the current overwrite and applying it with only send_messages changed. | ||||
| 2020-03-09 | Use `patch.object` instead of patch with direct `return_value`. | -9/+2 | ||
| 2020-03-08 | Move notifier to separate class. | -40/+46 | ||
| Separating the notifier allows us to keep the Silence class and its methods to be more focused on the class' purpose, handling the logic of adding/removing channels and the loop itself behind `SilenceNotifier`'s interface. | ||||
| 2020-03-08 | Remove "Channel" from output string for consistency. | -1/+1 | ||
| 2020-03-08 | Use `Context.invoke` instead of calling `unsilence` directly. | -1/+1 | ||
| Calling the command coro directly did unnecessary checks and made tests for the method harder to realize. | ||||
| 2020-03-08 | Add test for `unsilence` discord output. | -0/+9 | ||
| 2020-03-08 | Add test for `silence` discord output. | -0/+24 | ||
| 2020-03-08 | Add test case for Silence cog. | -1/+10 | ||
| 2020-03-08 | Add tests for FirstHash class. | -0/+25 | ||
| 2020-03-08 | Add handling for shh/unshh for `CommandNotFound`. | -4/+32 | ||
| 2020-03-08 | Restrict cog to moderators. | -1/+7 | ||
| 2020-03-02 | Add logging to loop start and loop end. | -0/+8 | ||
| 2020-03-02 | Add Silence cog. | -0/+143 | ||
| FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements. | ||||
| 2020-03-02 | Add HushDurationConverter. | -0/+28 | ||
| 2020-03-01 | Add grabify (IP logger) domains to banned domains | -0/+24 | ||
| 2020-03-01 | Modify log test regex to be non-os-specific | -1/+1 | ||
| Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\` | ||||
| 2020-03-01 | Move syncer confirmation reaction check out of finally clause | -11/+11 | ||
| Returning directly out of a `finally` clause can cause any exceptions raised in the clause to be discarded, so we can remove the finally clause entirely and shift the control statements into the body of the function | ||||