aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-10-08 13:22:54 -0700
committerGravatar MarkKoz <[email protected]>2020-10-08 13:22:54 -0700
commit5b87a272ff21df9fa4fb59fdf9ec92c6b57193c6 (patch)
treea451f179f09fbf21f9b88a1116d2fea1982ca560
parentSilence: add to notifier when indefinite rather than temporary (diff)
Silence: remove _mod_log_channel attribute
It's only used as an argument to `SilenceNotifier`, so it doesn't need to be an instance attribute.
-rw-r--r--bot/cogs/moderation/silence.py3
-rw-r--r--tests/bot/cogs/moderation/test_silence.py2
2 files changed, 2 insertions, 3 deletions
diff --git a/bot/cogs/moderation/silence.py b/bot/cogs/moderation/silence.py
index 95706392a..80c4e6a25 100644
--- a/bot/cogs/moderation/silence.py
+++ b/bot/cogs/moderation/silence.py
@@ -91,8 +91,7 @@ class Silence(commands.Cog):
guild = self.bot.get_guild(Guild.id)
self._verified_role = guild.get_role(Roles.verified)
self._mod_alerts_channel = self.bot.get_channel(Channels.mod_alerts)
- self._mod_log_channel = self.bot.get_channel(Channels.mod_log)
- self.notifier = SilenceNotifier(self._mod_log_channel)
+ self.notifier = SilenceNotifier(self.bot.get_channel(Channels.mod_log))
await self._reschedule()
@commands.command(aliases=("hush",))
diff --git a/tests/bot/cogs/moderation/test_silence.py b/tests/bot/cogs/moderation/test_silence.py
index 50d8419ac..6f8f4386b 100644
--- a/tests/bot/cogs/moderation/test_silence.py
+++ b/tests/bot/cogs/moderation/test_silence.py
@@ -119,7 +119,7 @@ class SilenceCogTests(unittest.IsolatedAsyncioTestCase):
mod_log = MockTextChannel()
self.bot.get_channel.side_effect = (None, mod_log)
await self.cog._async_init()
- notifier.assert_called_once_with(self.cog._mod_log_channel)
+ notifier.assert_called_once_with(mod_log)
@autospec(silence, "SilenceNotifier", pass_mocks=False)
async def test_async_init_rescheduled(self):