diff options
| author |  wookie184 <[email protected]> | 2024-01-28 23:39:48 +0000 |
|---|---|---|
| committer |  GitHub <[email protected]> | 2024-01-28 23:39:48 +0000 |
| commit | 88f020a11009cb2af9f3f67786205352013a5f18 (patch) | |
| tree | 7c61f5bf56542281a8f8f67a481526b574c2718f | |
| parent | Update URL regex in embed filter (#2896) (diff) | |
| parent | Merge branch 'main' into fix-incorrect-test (diff) | |
Merge pull request #2871 from python-discord/fix-incorrect-test
Fix test code that was doing nothing
| -rw-r--r-- | tests/bot/exts/backend/test_error_handler.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/tests/bot/exts/backend/test_error_handler.py b/tests/bot/exts/backend/test_error_handler.py index e28d5cfd0..9670d42a0 100644 --- a/tests/bot/exts/backend/test_error_handler.py +++ b/tests/bot/exts/backend/test_error_handler.py @@ -1,5 +1,5 @@ import unittest -from unittest.mock import AsyncMock, MagicMock, call, patch +from unittest.mock import AsyncMock, MagicMock, Mock, call, patch from discord.ext.commands import errors from pydis_core.site_api import ResponseCodeError @@ -503,6 +503,10 @@ class IndividualErrorHandlerTests(unittest.IsolatedAsyncioTestCase): self.ctx.reset_mock() log_mock.reset_mock() push_scope_mock.reset_mock() + scope_mock = Mock() + + # Mock `with push_scope_mock() as scope:` + push_scope_mock.return_value.__enter__.return_value = scope_mock self.ctx.guild = case await self.cog.handle_unexpected_error(self.ctx, errors.CommandError()) @@ -526,8 +530,8 @@ class IndividualErrorHandlerTests(unittest.IsolatedAsyncioTestCase): ) set_extra_calls.append(call("jump_to", url)) - push_scope_mock.set_tag.has_calls(set_tag_calls) - push_scope_mock.set_extra.has_calls(set_extra_calls) + scope_mock.set_tag.assert_has_calls(set_tag_calls) + scope_mock.set_extra.assert_has_calls(set_extra_calls) class ErrorHandlerSetupTests(unittest.IsolatedAsyncioTestCase): |