aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar ks129 <[email protected]>2020-04-16 13:40:47 +0300
committerGravatar ks129 <[email protected]>2020-04-16 13:40:47 +0300
commit1a3fa6a395141c4fcdd1d388d6ce3e7bd89bcbf0 (patch)
tree92f67047f03c5df417a5031f9fccde2274995c76
parent(ModLog): Removed unused `textwrap` import. (diff)
(Infractions and ModLog Tests): Replaced `shortening` with `truncation`, removed unnecessary type hint and added comment to kick truncation test about awaiting `kick`.
-rw-r--r--tests/bot/cogs/moderation/test_infractions.py9
-rw-r--r--tests/bot/cogs/moderation/test_modlog.py2
2 files changed, 6 insertions, 5 deletions
diff --git a/tests/bot/cogs/moderation/test_infractions.py b/tests/bot/cogs/moderation/test_infractions.py
index 39ea93952..51a8cc645 100644
--- a/tests/bot/cogs/moderation/test_infractions.py
+++ b/tests/bot/cogs/moderation/test_infractions.py
@@ -6,8 +6,8 @@ from bot.cogs.moderation.infractions import Infractions
from tests.helpers import MockBot, MockContext, MockGuild, MockMember, MockRole
-class ShorteningTests(unittest.IsolatedAsyncioTestCase):
- """Tests for ban and kick command reason shortening."""
+class TruncationTests(unittest.IsolatedAsyncioTestCase):
+ """Tests for ban and kick command reason truncation."""
def setUp(self):
self.bot = MockBot()
@@ -19,7 +19,7 @@ class ShorteningTests(unittest.IsolatedAsyncioTestCase):
@patch("bot.cogs.moderation.utils.has_active_infraction")
@patch("bot.cogs.moderation.utils.post_infraction")
- async def test_apply_ban_reason_shortening(self, post_infraction_mock, has_active_mock):
+ async def test_apply_ban_reason_truncation(self, post_infraction_mock, has_active_mock):
"""Should truncate reason for `ctx.guild.ban`."""
has_active_mock.return_value = False
post_infraction_mock.return_value = {"foo": "bar"}
@@ -38,7 +38,7 @@ class ShorteningTests(unittest.IsolatedAsyncioTestCase):
await ban
@patch("bot.cogs.moderation.utils.post_infraction")
- async def test_apply_kick_reason_shortening(self, post_infraction_mock) -> None:
+ async def test_apply_kick_reason_truncation(self, post_infraction_mock):
"""Should truncate reason for `Member.kick`."""
post_infraction_mock.return_value = {"foo": "bar"}
@@ -51,4 +51,5 @@ class ShorteningTests(unittest.IsolatedAsyncioTestCase):
kick.cr_frame.f_locals["kwargs"]["reason"],
textwrap.shorten("foo bar" * 3000, 512, placeholder="...")
)
+ # Await kick to avoid warning
await kick
diff --git a/tests/bot/cogs/moderation/test_modlog.py b/tests/bot/cogs/moderation/test_modlog.py
index 46e01d2ea..d60836474 100644
--- a/tests/bot/cogs/moderation/test_modlog.py
+++ b/tests/bot/cogs/moderation/test_modlog.py
@@ -14,7 +14,7 @@ class ModLogTests(unittest.IsolatedAsyncioTestCase):
self.cog = ModLog(self.bot)
self.channel = MockTextChannel()
- async def test_log_entry_description_shortening(self):
+ async def test_log_entry_description_truncation(self):
"""Should truncate embed description for ModLog entry."""
self.bot.get_channel.return_value = self.channel
await self.cog.send_log_message(