Skip to content

Commit a68628d

Browse files
committed
test(analytics): fix tests in test_organization_group_index
1 parent e8b551f commit a68628d

File tree

1 file changed

+16
-10
lines changed

1 file changed

+16
-10
lines changed

tests/sentry/issues/endpoints/test_organization_group_index.py

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
from django.utils import timezone
1313

1414
from sentry import options
15+
from sentry.analytics.events.advanced_search_feature_gated import AdvancedSearchFeatureGateEvent
1516
from sentry.feedback.usecases.create_feedback import FeedbackCreationSource, create_feedback_issue
1617
from sentry.integrations.models.external_issue import ExternalIssue
1718
from sentry.integrations.models.organization_integration import OrganizationIntegration
@@ -63,6 +64,7 @@
6364
from sentry.silo.base import SiloMode
6465
from sentry.testutils.cases import APITestCase, SnubaTestCase
6566
from sentry.testutils.helpers import parse_link_header
67+
from sentry.testutils.helpers.analytics import assert_last_analytics_event
6668
from sentry.testutils.helpers.datetime import before_now
6769
from sentry.testutils.helpers.features import Feature, apply_feature_flag_on_cls, with_feature
6870
from sentry.testutils.helpers.options import override_options
@@ -909,11 +911,13 @@ def test_advanced_search_errors(self, mock_record: MagicMock, _: MagicMock) -> N
909911
"search" == response.data["detail"]
910912
)
911913

912-
mock_record.assert_called_with(
913-
"advanced_search.feature_gated",
914-
user_id=self.user.id,
915-
default_user_id=self.user.id,
916-
organization_id=self.organization.id,
914+
assert_last_analytics_event(
915+
mock_record,
916+
AdvancedSearchFeatureGateEvent(
917+
user_id=self.user.id,
918+
default_user_id=self.user.id,
919+
organization_id=self.organization.id,
920+
),
917921
)
918922

919923
# This seems like a random override, but this test needed a way to override
@@ -3825,11 +3829,13 @@ def test_snuba_heavy_advanced_search_errors(self, mock_record: MagicMock, _: Mag
38253829
"search" == response.data["detail"]
38263830
)
38273831

3828-
mock_record.assert_called_with(
3829-
"advanced_search.feature_gated",
3830-
user_id=self.user.id,
3831-
default_user_id=self.user.id,
3832-
organization_id=self.organization.id,
3832+
assert_last_analytics_event(
3833+
mock_record,
3834+
AdvancedSearchFeatureGateEvent(
3835+
user_id=self.user.id,
3836+
default_user_id=self.user.id,
3837+
organization_id=self.organization.id,
3838+
),
38333839
)
38343840

38353841
def test_snuba_heavy_filter_not_unresolved(self, _: MagicMock) -> None:

0 commit comments

Comments
 (0)