diff --git a/src/sentry/seer/autofix/autofix.py b/src/sentry/seer/autofix/autofix.py index 8584f35f0f0d1d..b4641828a30764 100644 --- a/src/sentry/seer/autofix/autofix.py +++ b/src/sentry/seer/autofix/autofix.py @@ -653,12 +653,10 @@ def trigger_autofix( group.update(seer_autofix_last_triggered=timezone.now()) - # seer runs are free for web vitals issues during testing phase - if group.issue_type != WebVitalsGroup: - # log billing event for seer autofix - quotas.backend.record_seer_run( - group.organization.id, group.project.id, DataCategory.SEER_AUTOFIX - ) + # log billing event for seer autofix + quotas.backend.record_seer_run( + group.organization.id, group.project.id, DataCategory.SEER_AUTOFIX + ) return Response( { diff --git a/src/sentry/seer/autofix/issue_summary.py b/src/sentry/seer/autofix/issue_summary.py index c7e5c2b3442c10..3ee6a884ef7b1e 100644 --- a/src/sentry/seer/autofix/issue_summary.py +++ b/src/sentry/seer/autofix/issue_summary.py @@ -14,7 +14,6 @@ from sentry.api.serializers import EventSerializer, serialize from sentry.api.serializers.rest_framework.base import convert_dict_key_case, snake_to_camel_case from sentry.constants import DataCategory -from sentry.issues.grouptype import WebVitalsGroup from sentry.locks import locks from sentry.models.group import Group from sentry.net.http import connection_from_url @@ -420,9 +419,6 @@ def _generate_summary( def _log_seer_scanner_billing_event(group: Group, source: SeerAutomationSource): if source == SeerAutomationSource.ISSUE_DETAILS: return - # seer runs are free for web vitals issues during testing phase - if group.issue_type == WebVitalsGroup: - return quotas.backend.record_seer_run( group.organization.id, group.project.id, DataCategory.SEER_SCANNER diff --git a/tests/sentry/seer/autofix/test_autofix.py b/tests/sentry/seer/autofix/test_autofix.py index 663f348702a1e0..9fd2e7047b4e0f 100644 --- a/tests/sentry/seer/autofix/test_autofix.py +++ b/tests/sentry/seer/autofix/test_autofix.py @@ -941,7 +941,7 @@ def test_trigger_autofix_with_web_vitals_issue( response = trigger_autofix(group=group, user=user, instruction="Test instruction") assert response.status_code == 202 - mock_record_seer_run.assert_not_called() + mock_record_seer_run.assert_called_once() @requires_snuba diff --git a/tests/sentry/seer/autofix/test_issue_summary.py b/tests/sentry/seer/autofix/test_issue_summary.py index a7b72b7eeb0363..f6fa1b9fd00e05 100644 --- a/tests/sentry/seer/autofix/test_issue_summary.py +++ b/tests/sentry/seer/autofix/test_issue_summary.py @@ -607,7 +607,7 @@ def test_get_issue_summary_with_web_vitals_issue( ) assert status_code == 200 - mock_record_seer_run.assert_not_called() + mock_record_seer_run.assert_called_once() mock_trigger_autofix_task.assert_called_once() @patch("sentry.seer.autofix.issue_summary.get_seer_org_acknowledgement")