aboutsummaryrefslogtreecommitdiffstats
path: root/tests/helpers.py (unfollow)
Commit message (Collapse)AuthorLines
2020-03-15Document returns values of private methods.Gravatar Numerlor-0/+2
2020-03-15Block commands until all instance vars are loaded.Gravatar Numerlor-0/+4
2020-03-15Assign created task to a var.Gravatar Numerlor-1/+1
2020-03-15Pass kwargs directly instead of a PermissionOverwrite.Gravatar Numerlor-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-15Change BadArgument error wording.Gravatar Numerlor-1/+1
Co-authored-by: MarkKoz <[email protected]>
2020-03-15Add Silence cog load to docstring.Gravatar Numerlor-1/+1
2020-03-15Move coro execution outside of if condition.Gravatar Numerlor-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-15Adjust docstring styling.Gravatar Numerlor-2/+2
Co-authored-by: MarkKoz <[email protected]>
2020-03-15Use dict instead of a set and custom class.Gravatar Numerlor-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-11Add tests for `_notifier`.Gravatar Numerlor-0/+19
2020-03-11Improve commands help.Gravatar Numerlor-3/+5
2020-03-11Add logging to commands.Gravatar Numerlor-0/+3
2020-03-11Change various logging levels.Gravatar Numerlor-7/+7
2020-03-11Add tests for `remove_channel`.Gravatar Numerlor-0/+18
2020-03-11Remove `_current_loop` from setup.Gravatar Numerlor-2/+1
2020-03-11Add tests for `add_channel`.Gravatar Numerlor-0/+21
2020-03-11Create test case for `SilenceNotifier`Gravatar Numerlor-1/+7
2020-03-11Add tests for `_get_instance_vars`.Gravatar Numerlor-1/+28
2020-03-11Pass created channel instead of new object.Gravatar Numerlor-1/+1
Creating a new object caused the assert to fail because different objects were used.
2020-03-11Reset `self.ctx` call history after every subtest.Gravatar Numerlor-0/+1
2020-03-11Fix erroneous `assert_called_once_with` calls.Gravatar Numerlor-8/+7
`assert_called_once_with` was being tested on call_args which always reported success.st.
2020-03-11Add test for `cog_check`.Gravatar Numerlor-0/+7
2020-03-11Add test for `cog_unload`.Gravatar Numerlor-1/+17
2020-03-11Use patch decorators and assign names from `with` patches.Gravatar Numerlor-11/+11
2020-03-11Separate tests for permissions and `muted_channels.add` on `_silence`.Gravatar Numerlor-5/+8
2020-03-11Add tests for `_unsilence`Gravatar Numerlor-1/+33
2020-03-11Use async test case.Gravatar Numerlor-14/+12
This allows us to use coroutines with await directly instead of asyncio.run
2020-03-11Add test for `_silence` method.Gravatar Numerlor-1/+34
2020-03-11Move adding of channel to `muted_channels` up.Gravatar Numerlor-1/+1
Before the channel was not added if `persistent` was `True`.
2020-03-11Remove channel mentions from output discord messages.Gravatar Numerlor-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-11Remove unnecessary args from test cases.Gravatar Numerlor-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-10Add alert with silenced channels on `cog_unload`.Gravatar Numerlor-0/+11
2020-03-10Remove `channel` arg from commands.Gravatar Numerlor-20/+10
2020-03-10Retain previous channel overwrites.Gravatar Numerlor-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-09Use `patch.object` instead of patch with direct `return_value`.Gravatar Numerlor-9/+2
2020-03-08Move notifier to separate class.Gravatar Numerlor-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-08Remove "Channel" from output string for consistency.Gravatar Numerlor-1/+1
2020-03-08Use `Context.invoke` instead of calling `unsilence` directly.Gravatar Numerlor-1/+1
Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
2020-03-08Add test for `unsilence` discord output.Gravatar Numerlor-0/+9
2020-03-08Add test for `silence` discord output.Gravatar Numerlor-0/+24
2020-03-08Add test case for Silence cog.Gravatar Numerlor-1/+10
2020-03-08Add tests for FirstHash class.Gravatar Numerlor-0/+25
2020-03-08Add handling for shh/unshh for `CommandNotFound`.Gravatar Numerlor-4/+32
2020-03-08Restrict cog to moderators.Gravatar Numerlor-1/+7
2020-03-02Add logging to loop start and loop end.Gravatar Numerlor-0/+8
2020-03-02Add Silence cog.Gravatar Numerlor-0/+143
FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements.
2020-03-02Add HushDurationConverter.Gravatar Numerlor-0/+28
2020-03-01Add grabify (IP logger) domains to banned domainsGravatar Joseph-0/+24
2020-03-01Modify log test regex to be non-os-specificGravatar S. Co1-1/+1
Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\`
2020-03-01Move syncer confirmation reaction check out of finally clauseGravatar S. Co1-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