Skip to content

Commit aa3f36f

Browse files
aledlieclaude
andcommitted
test(sentry): update tests for Sentry SDK 2.x API
Update test mocks to match the Sentry 2.x migration: - Replace Hub.current.client with get_client().is_active() - Replace push_scope with isolation_scope 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent c413851 commit aa3f36f

1 file changed

Lines changed: 17 additions & 17 deletions

File tree

‎tests/unit/test_logging_config.py‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -496,9 +496,9 @@ def test_logs_without_context(self):
496496
@patch('src.utils.logging_config.sentry_sdk')
497497
def test_sends_to_sentry_when_available(self, mock_sentry_sdk):
498498
"""Test that log_exception sends to Sentry when available."""
499-
mock_hub = MagicMock()
500-
mock_hub.client = MagicMock()
501-
mock_sentry_sdk.Hub.current = mock_hub
499+
mock_client = MagicMock()
500+
mock_client.is_active.return_value = True
501+
mock_sentry_sdk.get_client.return_value = mock_client
502502

503503
error = ValueError("Test error")
504504
log_exception(self.logger, error)
@@ -509,12 +509,12 @@ def test_sends_to_sentry_when_available(self, mock_sentry_sdk):
509509
@patch('src.utils.logging_config.sentry_sdk')
510510
def test_adds_context_to_sentry_scope(self, mock_sentry_sdk):
511511
"""Test that log_exception adds context to Sentry scope."""
512-
mock_hub = MagicMock()
513-
mock_hub.client = MagicMock()
514-
mock_sentry_sdk.Hub.current = mock_hub
512+
mock_client = MagicMock()
513+
mock_client.is_active.return_value = True
514+
mock_sentry_sdk.get_client.return_value = mock_client
515515
mock_scope = MagicMock()
516-
mock_sentry_sdk.push_scope.return_value.__enter__ = MagicMock(return_value=mock_scope)
517-
mock_sentry_sdk.push_scope.return_value.__exit__ = MagicMock(return_value=False)
516+
mock_sentry_sdk.isolation_scope.return_value.__enter__ = MagicMock(return_value=mock_scope)
517+
mock_sentry_sdk.isolation_scope.return_value.__exit__ = MagicMock(return_value=False)
518518

519519
error = ValueError("Test error")
520520
context = {'key': 'value'}
@@ -583,9 +583,9 @@ def test_works_without_metadata(self):
583583
@patch('src.utils.logging_config.sentry_sdk')
584584
def test_creates_sentry_transaction_when_available(self, mock_sentry_sdk):
585585
"""Test that log_performance_metric creates Sentry transaction."""
586-
mock_hub = MagicMock()
587-
mock_hub.client = MagicMock()
588-
mock_sentry_sdk.Hub.current = mock_hub
586+
mock_client = MagicMock()
587+
mock_client.is_active.return_value = True
588+
mock_sentry_sdk.get_client.return_value = mock_client
589589
mock_transaction = MagicMock()
590590
mock_sentry_sdk.start_transaction.return_value.__enter__ = MagicMock(return_value=mock_transaction)
591591
mock_sentry_sdk.start_transaction.return_value.__exit__ = MagicMock(return_value=False)
@@ -602,9 +602,9 @@ def test_creates_sentry_transaction_when_available(self, mock_sentry_sdk):
602602
@patch('src.utils.logging_config.sentry_sdk')
603603
def test_sets_measurement_on_sentry_transaction(self, mock_sentry_sdk):
604604
"""Test that log_performance_metric sets measurement on Sentry transaction."""
605-
mock_hub = MagicMock()
606-
mock_hub.client = MagicMock()
607-
mock_sentry_sdk.Hub.current = mock_hub
605+
mock_client = MagicMock()
606+
mock_client.is_active.return_value = True
607+
mock_sentry_sdk.get_client.return_value = mock_client
608608
mock_transaction = MagicMock()
609609
mock_sentry_sdk.start_transaction.return_value.__enter__ = MagicMock(return_value=mock_transaction)
610610
mock_sentry_sdk.start_transaction.return_value.__exit__ = MagicMock(return_value=False)
@@ -625,9 +625,9 @@ def test_sets_measurement_on_sentry_transaction(self, mock_sentry_sdk):
625625
@patch('src.utils.logging_config.sentry_sdk')
626626
def test_sets_tags_from_metadata(self, mock_sentry_sdk):
627627
"""Test that log_performance_metric sets tags from metadata."""
628-
mock_hub = MagicMock()
629-
mock_hub.client = MagicMock()
630-
mock_sentry_sdk.Hub.current = mock_hub
628+
mock_client = MagicMock()
629+
mock_client.is_active.return_value = True
630+
mock_sentry_sdk.get_client.return_value = mock_client
631631
mock_transaction = MagicMock()
632632
mock_sentry_sdk.start_transaction.return_value.__enter__ = MagicMock(return_value=mock_transaction)
633633
mock_sentry_sdk.start_transaction.return_value.__exit__ = MagicMock(return_value=False)

0 commit comments

Comments
 (0)