|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | |\ \ \ \ \ \ \ \ \ \  
| | |_|_|_|_|_|_|_|_|_|/ / / / / / / / / /  
| |/| | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | | |  | 
| | | | |_|_|_|_|_|_|_|/ / / / / / / / / /  
| | |/| | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | It's better to have a self-documenting name than a comment, which, by
the way, was using the old name for the decorator. | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | |\ \ \ \ \ \ \ \ \  
| | |_|_|_|_|_|_|_|_|_|/ / / / / / / / /  
| |/| | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | It's only used as an argument to `SilenceNotifier`, so it doesn't need
to be an instance attribute. | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | Accidentally swapped the logic in a previous commit during a refactor. | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | Because the Scheduler is mocked, it doesn't actually do anything with
the coroutines passed to the schedule() functions, hence the warnings. | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | * Rename to `_silence_overwrites`
* Reduce responsibilities to only setting permission overwrites
* Log in `silence` instead
* Add to notifier in `silence` instead | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | Co-authored-by: Numerlor <[email protected]> | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | It's not really necessary to set to True when mocking functions. | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | | Duplicating strings in assertions is redundant, closely coupled, and
less maintainable. | 
| | | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | | |  |