diff options
| author |  ks123 <[email protected]> | 2020-03-08 08:41:23 +0200 |
|---|---|---|
| committer | ks123 <[email protected]> | 2020-03-08 08:41:23 +0200 |
| commit | f4fff7139ddffa08b12973d69c8f4bd7c47c0224 (patch) | |
| tree | 8082e7c7906a834d67fdc5b26b7a077dc8d201e4 | |
| parent | (Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` and re... (diff) | |
(Moderation Utils Tests): Removed unnecessary `ctx.send` mock resetting, moved `send_private_embed` mock reset to beginning of subtest.
| -rw-r--r-- | tests/bot/cogs/moderation/test_utils.py | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/tests/bot/cogs/moderation/test_utils.py b/tests/bot/cogs/moderation/test_utils.py index 5ab279391..5637ff508 100644 --- a/tests/bot/cogs/moderation/test_utils.py +++ b/tests/bot/cogs/moderation/test_utils.py @@ -135,9 +135,9 @@ class ModerationUtilsTests(unittest.IsolatedAsyncioTestCase): send = case["send_result"] with self.subTest(args=args, expected=expected, send=send): + send_private_embed_mock.reset_mock() send_private_embed_mock.return_value = send - result = await utils.notify_infraction(*args) self.assertEqual(send, result) @@ -155,9 +155,6 @@ class ModerationUtilsTests(unittest.IsolatedAsyncioTestCase): send_private_embed_mock.assert_awaited_once_with(args[0], embed) - self.ctx.send.reset_mock(side_effect=True) - send_private_embed_mock.reset_mock() - @patch("bot.cogs.moderation.utils.send_private_embed") async def test_notify_pardon(self, send_private_embed_mock): """Test does `notify_pardon` create correct embed and return correct bool.""" |