Merge pull request #274 from uktrade/LTD-5327-fix-sentry-sdk-lookup-e…

…rror

[LTD-5327] Update sentry scope pushing call after major version bump

mail/libraries/helpers.py

@@ -338,7 +338,7 @@ def sort_dtos_by_date(input_dtos):
 
 def log_to_sentry(message, extra=None, level="info"):
     extra = extra or {}
-    with sentry_sdk.push_scope() as scope:
+    with sentry_sdk.new_scope() as scope:
         for key, value in extra.items():
             scope.set_extra(key, value)
         sentry_sdk.capture_message(message, level=level)

mail/tests/libraries/test_helpers.py

@@ -0,0 +1,15 @@
+import unittest
+from unittest import mock
+
+from mail.libraries.helpers import log_to_sentry
+
+
+class TestLogToSentry(unittest.TestCase):
+
+    @mock.patch("sentry_sdk.scope.Scope.capture_event")
+    def test_log_to_sentry(self, mock_capture_event):
+        log_to_sentry("some message", {"extra": "context"}, level="debug")
+        mock_capture_event.assert_called_with(
+            {"message": "some message", "level": "debug"},
+            scope=None,
+        )