diff --git a/benefits/oauth/analytics.py b/benefits/oauth/analytics.py index 23e956415..54e4aea4d 100644 --- a/benefits/oauth/analytics.py +++ b/benefits/oauth/analytics.py @@ -10,9 +10,9 @@ class OAuthEvent(core.Event): def __init__(self, request, event_type): super().__init__(request, event_type) - verifier = session.flow(request) - if verifier and verifier.uses_claims_verification: - self.update_event_properties(auth_provider=verifier.claims_provider.client_name) + flow = session.flow(request) + if flow and flow.uses_claims_verification: + self.update_event_properties(claims_provider=flow.claims_provider.client_name) class OAuthErrorEvent(OAuthEvent): diff --git a/tests/pytest/oauth/test_analytics.py b/tests/pytest/oauth/test_analytics.py index d571b92f6..e6e094fd9 100644 --- a/tests/pytest/oauth/test_analytics.py +++ b/tests/pytest/oauth/test_analytics.py @@ -10,8 +10,8 @@ def test_OAuthEvent_flow_client_name_when_uses_claims_verification(app_request, event = OAuthEvent(app_request, "event type") - assert "auth_provider" in event.event_properties - assert event.event_properties["auth_provider"] == mocked_flow.claims_provider.client_name + assert "claims_provider" in event.event_properties + assert event.event_properties["claims_provider"] == mocked_flow.claims_provider.client_name @pytest.mark.django_db @@ -19,7 +19,7 @@ def test_OAuthEvent_flow_client_name_when_uses_claims_verification(app_request, def test_OAuthEvent_flow_no_client_name_when_does_not_use_claims_verification(app_request): event = OAuthEvent(app_request, "event type") - assert "auth_provider" not in event.event_properties + assert "claims_provider" not in event.event_properties @pytest.mark.django_db