aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | 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
| | | | | | |
| * | | | | | Move adding of channel to `muted_channels` up.Gravatar Numerlor2020-03-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Before the channel was not added if `persistent` was `True`.
| * | | | | | Remove channel mentions from output discord messages.Gravatar Numerlor2020-03-11-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
| * | | | | | 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.
| * | | | | | Add alert with silenced channels on `cog_unload`.Gravatar Numerlor2020-03-10-0/+11
| | | | | | |
| * | | | | | Remove `channel` arg from commands.Gravatar Numerlor2020-03-10-20/+10
| | | | | | |
| * | | | | | Retain previous channel overwrites.Gravatar Numerlor2020-03-10-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.
| * | | | | | Use `patch.object` instead of patch with direct `return_value`.Gravatar Numerlor2020-03-09-9/+2
| | | | | | |
| * | | | | | Move notifier to separate class.Gravatar Numerlor2020-03-08-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.
| * | | | | | Remove "Channel" from output string for consistency.Gravatar Numerlor2020-03-08-1/+1
| | | | | | |
| * | | | | | Use `Context.invoke` instead of calling `unsilence` directly.Gravatar Numerlor2020-03-08-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
| * | | | | | Add test for `unsilence` discord output.Gravatar Numerlor2020-03-08-0/+9
| | | | | | |
| * | | | | | Add test for `silence` discord output.Gravatar Numerlor2020-03-08-0/+24
| | | | | | |
| * | | | | | Add test case for Silence cog.Gravatar Numerlor2020-03-08-1/+10
| | | | | | |
| * | | | | | Add tests for FirstHash class.Gravatar Numerlor2020-03-08-0/+25
| | | | | | |
| * | | | | | Add handling for shh/unshh for `CommandNotFound`.Gravatar Numerlor2020-03-08-4/+32
| | | | | | |
| * | | | | | Restrict cog to moderators.Gravatar Numerlor2020-03-08-1/+7
| | | | | | |
| * | | | | | Add logging to loop start and loop end.Gravatar Numerlor2020-03-02-0/+8
| | | | | | |
| * | | | | | Add Silence cog.Gravatar Numerlor2020-03-02-0/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements.
| * | | | | | Add HushDurationConverter.Gravatar Numerlor2020-03-02-0/+28
| | | | | | |
| | * | | | | Increase syncer logging levelGravatar S. Co12020-03-28-1/+1
| | | | | | |
| | * | | | | Use debug log level instead of warning in `post_user`Gravatar MarkKoz2020-03-28-1/+1
| | | | | | |
| | * | | | | Logging severity pass from reviewGravatar S. Co12020-03-28-4/+4
| | | | | | |
| | * | | | | Initial pass on log severity reductionGravatar S. Co12020-03-26-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the updated definition on logging levels, there are a few events that were issuing logs at too high of a level. This also includes some kaizening of existing log messages.
| | | * | | | Merge branch 'master' into zen-match-fixGravatar S. Co12020-03-26-284/+320
| | | |\ \ \ \
| | | * \ \ \ \ Merge remote-tracking branch 'origin/zen-match-fix' into zen-match-fixGravatar ks1232020-03-17-1/+84
| | | |\ \ \ \ \
| | | | * \ \ \ \ Merge branch 'master' into zen-match-fixGravatar Matteo Bertucci2020-03-17-1/+84
| | | | |\ \ \ \ \
| | | * | | | | | | (!zen Command): Under exact word match, change matching way from substring ↵Gravatar ks1232020-03-17-5/+6
| | | |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | to sentence split iterate and equality check.
| | | * | | | | | (!zen Command): Added exact word check before `difflib`'s matching, due ↵Gravatar ks1232020-03-17-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | matching may not count exact word as best choice.
| | | | | | * | | Filtering: merge the word and token watch filtersGravatar MarkKoz2020-03-27-41/+21
| | | | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only difference was the automatic addition of word boundaries. Otherwise, they shared a lot of code. The regex lists were kept separate in the config to retain the convenience of word boundaries automatically being added. * Rename filter to `watch_regex` * Expand spoilers for both words and tokens * Ignore URLs for both words and tokens
| | | | | | * | (Tags): Removed unnecessary `send_embed_with_trashcan` function due using ↵Gravatar ks1232020-03-31-23/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | existing function.
| | | | | | * | (Tags): Moved to existing `wait_for_deletion` function instead using ↵Gravatar ks1232020-03-31-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | custom/new one.
| | | | | | * | (Tags): Added blank line between check function and `try:` block on ↵Gravatar ks1232020-03-31-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `send_embed_with_trashcan` function.
| | | | | | * | (Tags): Fixed `TimeoutError` shadowing with `asyncio.TimeoutError`.Gravatar ks1232020-03-31-2/+2
| | | | | | | |
| | | | | | * | (Tags): Modified helper function `handle_trashcan_react` to ↵Gravatar ks1232020-03-31-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `send_embed_with_trashcan`, applied to docstring and to command.
| | | | | | * | (Tags): Fixed trashcan handling check.Gravatar ks1232020-03-30-2/+2
| | | | | | | |
| | | | | | * | (Tags): Added trashcan handling to `!tags get` command.Gravatar ks1232020-03-30-2/+4
| | | | | | | |
| | | | | | * | (Tags): Added helper function `handle_trashcan_react` for tag response ↵Gravatar ks1232020-03-30-1/+21
| |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | deletion handling.
| | | | | | * (Infraction Edit): Changed already expired and no reason provided sentence.Gravatar Karlis S2020-03-31-3/+1
| | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | * (Infraction Edit): Don't change infraction when user try modify duration of ↵Gravatar ks1232020-03-31-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | infraction that is already expired and reason not specified.