aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | | | | | | * | | | | | | | | | Create test for case when posting infraction failsGravatar ks1292020-10-11-1/+12
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Revert recent walrus operator changeGravatar ks1292020-10-11-1/+2
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create test for case when user already have active Voice BanGravatar ks1292020-10-11-2/+11
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Simplify post infraction calling and None checkGravatar ks1292020-10-11-2/+1
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create test for case when trying to voice ban user who haven't passed gateGravatar ks1292020-10-11-2/+11
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create test for voice unbanGravatar ks1292020-10-11-0/+6
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create test for temporary voice banGravatar ks1292020-10-11-1/+7
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create test for permanent voice banGravatar ks1292020-10-11-0/+17
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Improve voice gate messages deletionGravatar ks1292020-10-11-2/+8
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Suppress Voice Gate cog InWhiteListCheckFailureGravatar ks1292020-10-11-0/+6
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Implement Voice Verifying command and delete message in voice gateGravatar ks1292020-10-11-1/+111
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Remove not used importsGravatar ks1292020-10-11-1/+0
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Implement voice bans (temporary and permanent)Gravatar ks1292020-10-11-0/+85
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Add voice ban icons and show appeal footer for voice banGravatar ks1292020-10-11-1/+2
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Merge remote-tracking branch 'up/master' into voice-gateGravatar ks1292020-10-11-695/+807
| | | | | | | | | | | |\ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Add voice gate configuration to config-default.ymlGravatar ks1292020-10-10-0/+6
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Define Voice Gate channel, role and requirement in constants.pyGravatar ks1292020-10-10-0/+10
| | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | | Create base Voice Gate cogGravatar ks1292020-10-10-0/+15
| | | |_|_|_|_|_|_|_|/ / / / / / / / / / | | |/| | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: update docstrings in notifier testsGravatar MarkKoz2020-10-21-2/+2
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Reminders: rename namespace constantGravatar MarkKoz2020-10-18-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's better to have a self-documenting name than a comment, which, by the way, was using the old name for the decorator.
| | | | | | | | | | | * | | | | | | | | Silence: add locks to commandsGravatar MarkKoz2020-10-18-1/+9
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: rename function to reduce ambiguityGravatar MarkKoz2020-10-18-10/+10
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: amend the manual unsilence messageGravatar MarkKoz2020-10-09-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify that this situation could also result from the cache being cleared prematurely. There's no way to distinguish the two scenarios, so a manual unsilence is required for both.
| | | | | | | | | | | * | | | | | | | | Silence: require only 1 permission to be False for a manual unsilenceGravatar MarkKoz2020-10-09-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, both sending messages and adding reactions had to be false in order for the manual unsilence failure message to be sent. Because staff may only set one of these manually, the message should be sent if at least one of the permissions is set.
| | | | | | | | | | | * | | | | | | | | Merge master, fix conflicts, and create redis session in silence testsGravatar MarkKoz2020-10-09-4469/+3950
| | | | | | | | | | | |\ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: fix scheduled tasks not being cancelled on unloadGravatar MarkKoz2020-10-09-1/+1
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: make _async_init attribute tests more robustGravatar MarkKoz2020-10-08-7/+11
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: remove _mod_log_channel attributeGravatar MarkKoz2020-10-08-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's only used as an argument to `SilenceNotifier`, so it doesn't need to be an instance attribute.
| | | | | | | | | | | * | | | | | | | | Silence: add to notifier when indefinite rather than temporaryGravatar MarkKoz2020-10-08-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Accidentally swapped the logic in a previous commit during a refactor.
| | | | | | | | | | | * | | | | | | | | Silence tests: fix unawaited coro warningsGravatar MarkKoz2020-10-07-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because the Scheduler is mocked, it doesn't actually do anything with the coroutines passed to the schedule() functions, hence the warnings.
| | | | | | | | | | | * | | | | | | | | Silence: rename _init_cog to _async_initGravatar MarkKoz2020-10-06-15/+15
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: refactor _silenceGravatar MarkKoz2020-10-06-33/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rename to `_silence_overwrites` * Reduce responsibilities to only setting permission overwrites * Log in `silence` instead * Add to notifier in `silence` instead
| | | | | | | | | | | * | | | | | | | | Silence: move unsilence scheduling to a separate functionGravatar MarkKoz2020-10-06-8/+13
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Add license & copyright for autospec's _decoration_helperGravatar MarkKoz2020-09-25-0/+52
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: use f-string for messageGravatar Mark2020-09-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Numerlor <[email protected]>
| | | | | | | | | | | * | | | | | | | | Silence: cancel init task when cog unloadsGravatar MarkKoz2020-08-18-4/+12
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence: rename cachesGravatar MarkKoz2020-08-18-29/+29
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: remove unnecessary spec_set argsGravatar MarkKoz2020-08-18-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not really necessary to set to True when mocking functions.
| | | | | | | | | | | * | | | | | | | | Silence tests: more accurately assert the silence cmd schedule a taskGravatar MarkKoz2020-08-18-3/+7
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: add a test for rescheduling active silencesGravatar MarkKoz2020-08-18-0/+29
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: test the rescheduler unsilences expired silencesGravatar MarkKoz2020-08-18-0/+14
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: test the rescheduler adds permanent silence to notifierGravatar MarkKoz2020-08-18-0/+14
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: test the scheduler skips missing channelsGravatar MarkKoz2020-08-18-0/+25
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: add a test for _init_cog rescheduling unsilencesGravatar MarkKoz2020-08-18-0/+7
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: remove _reschedule patch for cog testsGravatar MarkKoz2020-08-18-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They don't do anything because they patch the class rather than the instance. It's too late for patching the instance to work since the `setUp` fixture, which instantiates the cog, executes before the patches do. Patching `setUp` would work (and its done in the other test cases), but some tests in this case will need the unpatched function too. Patching it doesn't serve much benefit to most tests anyway, so it's not worth the effort trying to make them work where they aren't needed.
| | | | | | | | | | | * | | | | | | | | Silence tests: access everything via the silence moduleGravatar MarkKoz2020-08-18-24/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The module is imported anyway to keep imports short and clean. Using it in patch targets is shorter and allows for the two imports from the module to be removed.
| | | | | | | | | | | * | | | | | | | | Silence tests: add a subtest for the manual unsilence messageGravatar MarkKoz2020-08-17-4/+7
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: assert against message constantsGravatar MarkKoz2020-08-17-15/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Duplicating strings in assertions is redundant, closely coupled, and less maintainable.
| | | | | | | | | | | * | | | | | | | | Silence tests: also test that cache misses preserve other overwritesGravatar MarkKoz2020-08-17-12/+16
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | * | | | | | | | | Silence tests: add a test for a mod alert on cache missGravatar MarkKoz2020-08-17-0/+7
| | | | | | | | | | | | | | | | | | | |