Skip to content

Commit 45d7d56

Browse files
committed
refactor(error_handler): update _log_error method to accept mock source
- Modified the _log_error method to include a mock source parameter for improved error logging context. - Updated test cases to reflect the new method signature, ensuring proper testing of error handling with Sentry enabled and disabled.
1 parent 62b7a35 commit 45d7d56

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

tests/services/test_error_handler.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,11 @@ def test_log_error_with_sentry(
9191
"""Test _log_error with Sentry enabled."""
9292
error = ValueError("Test error")
9393
config = ErrorHandlerConfig(send_to_sentry=True, log_level="ERROR")
94+
mock_source = MagicMock()
95+
mock_source.command = None
96+
mock_source.guild = None
9497

95-
error_handler._log_error(error, config)
98+
error_handler._log_error(error, config, mock_source)
9699

97100
mock_logger.error.assert_called_once()
98101

@@ -105,8 +108,11 @@ def test_log_error_without_sentry(
105108
"""Test _log_error with Sentry disabled."""
106109
error = ValueError("Test error")
107110
config = ErrorHandlerConfig(send_to_sentry=False, log_level="INFO")
111+
mock_source = MagicMock()
112+
mock_source.command = None
113+
mock_source.guild = None
108114

109-
error_handler._log_error(error, config)
115+
error_handler._log_error(error, config, mock_source)
110116

111117
mock_logger.info.assert_called_once()
112118

0 commit comments

Comments
 (0)