diff --git a/src/sentry/testutils/fixtures.py b/src/sentry/testutils/fixtures.py index 04c761549b6c4b..b3bbcaa709b1b3 100644 --- a/src/sentry/testutils/fixtures.py +++ b/src/sentry/testutils/fixtures.py @@ -485,7 +485,6 @@ def create_alert_rule_trigger_action( self, alert_rule_trigger=None, target_identifier=None, - triggered_for_incident=None, *args, **kwargs, ): diff --git a/tests/sentry/incidents/test_charts.py b/tests/sentry/incidents/test_charts.py index 43ebfcdda6ee0c..7d8262c4089612 100644 --- a/tests/sentry/incidents/test_charts.py +++ b/tests/sentry/incidents/test_charts.py @@ -109,9 +109,7 @@ def test_eap_alert(self, mock_client_get: MagicMock, mock_generate_chart: MagicM date_started=timezone.now() - datetime.timedelta(minutes=2), ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) alert_rule_serialized_response: AlertRuleSerializerResponse = serialize( alert_rule, None, AlertRuleSerializer() @@ -157,9 +155,7 @@ def test_eap_log_alert( date_started=timezone.now() - datetime.timedelta(minutes=2), ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) alert_rule_serialized_response: AlertRuleSerializerResponse = serialize( alert_rule, None, AlertRuleSerializer() @@ -204,9 +200,7 @@ def test_eap_trace_metric_alert( date_started=timezone.now() - datetime.timedelta(minutes=2), ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) alert_rule_serialized_response: AlertRuleSerializerResponse = serialize( alert_rule, None, AlertRuleSerializer() diff --git a/tests/sentry/integrations/slack/test_unfurl.py b/tests/sentry/integrations/slack/test_unfurl.py index bcc27b33234a57..31fcbe96acc6f9 100644 --- a/tests/sentry/integrations/slack/test_unfurl.py +++ b/tests/sentry/integrations/slack/test_unfurl.py @@ -438,9 +438,7 @@ def test_unfurl_metric_alert(self) -> None: ) incident.update(identifier=123) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) links = [ UnfurlableUrl( @@ -477,9 +475,7 @@ def test_unfurl_metric_alerts_chart(self, mock_generate_chart: MagicMock) -> Non ) incident.update(identifier=123) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) self._wire_workflow_engine_for_incident(alert_rule, incident) url = f"https://sentry.io/organizations/{self.organization.slug}/issues/alerts/rules/details/{alert_rule.id}/?alert={incident.identifier}" @@ -587,9 +583,7 @@ def test_unfurl_metric_alerts_chart_eap_spans(self, mock_generate_chart: MagicMo date_started=timezone.now() - timedelta(minutes=2), ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) self._wire_workflow_engine_for_incident(alert_rule, incident) url = f"https://sentry.io/organizations/{self.organization.slug}/issues/alerts/rules/details/{alert_rule.id}/?alert={incident.identifier}" diff --git a/tests/sentry/integrations/test_metric_alerts.py b/tests/sentry/integrations/test_metric_alerts.py index a1a5d9df6a5e4a..78e8f42f4490eb 100644 --- a/tests/sentry/integrations/test_metric_alerts.py +++ b/tests/sentry/integrations/test_metric_alerts.py @@ -45,9 +45,7 @@ def test_returns_correct_info_with_workflow_engine_dual_write(self) -> None: date_started=date_started, ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) metric_value = 123 referrer = "metric_alert_custom" notification_uuid = str(uuid.uuid4()) @@ -115,9 +113,7 @@ def test_returns_correct_info_placeholder_incident(self) -> None: date_started=date_started, ) trigger = self.create_alert_rule_trigger(alert_rule, CRITICAL_TRIGGER_LABEL, 100) - self.create_alert_rule_trigger_action( - alert_rule_trigger=trigger, triggered_for_incident=incident - ) + self.create_alert_rule_trigger_action(alert_rule_trigger=trigger) metric_value = 123 referrer = "metric_alert_custom" notification_uuid = str(uuid.uuid4())