aboutsummaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorGravatar Xithrius <[email protected]>2022-05-27 15:35:22 -0700
committerGravatar GitHub <[email protected]>2022-05-27 15:35:22 -0700
commit5f30adc70787f13a440ed7e53524b35dee791b79 (patch)
treefde3dbbe0b2ec7fffd171cc90b3cfbf017df7975 /tests
parentMerge branch 'main' into revert-markdownify-bump (diff)
parentCatch correct exception in clean cog (#2176) (diff)
Merge branch 'main' into revert-markdownify-bump
Diffstat (limited to 'tests')
-rw-r--r--tests/bot/exts/backend/test_error_handler.py6
1 files changed, 4 insertions, 2 deletions
diff --git a/tests/bot/exts/backend/test_error_handler.py b/tests/bot/exts/backend/test_error_handler.py
index 193f1d822..d02bd7c34 100644
--- a/tests/bot/exts/backend/test_error_handler.py
+++ b/tests/bot/exts/backend/test_error_handler.py
@@ -477,11 +477,11 @@ class IndividualErrorHandlerTests(unittest.IsolatedAsyncioTestCase):
@patch("bot.exts.backend.error_handler.log")
async def test_handle_api_error(self, log_mock):
- """Should `ctx.send` on HTTP error codes, `log.debug|warning` depends on code."""
+ """Should `ctx.send` on HTTP error codes, and log at correct level."""
test_cases = (
{
"error": ResponseCodeError(AsyncMock(status=400)),
- "log_level": "debug"
+ "log_level": "error"
},
{
"error": ResponseCodeError(AsyncMock(status=404)),
@@ -505,6 +505,8 @@ class IndividualErrorHandlerTests(unittest.IsolatedAsyncioTestCase):
self.ctx.send.assert_awaited_once()
if case["log_level"] == "warning":
log_mock.warning.assert_called_once()
+ elif case["log_level"] == "error":
+ log_mock.error.assert_called_once()
else:
log_mock.debug.assert_called_once()