Skip to content

Commit cc14236

Browse files
mifu67andrewshie-sentry
authored andcommitted
fix(aci milestone 3): fix failing data condition tests (#91823)
#90898 changes what is returned by `migrate_metric_data_conditions`. The return value is primarily used by tests, and a few more tests using the function were added after the last time we merged latest master into the branch.
1 parent 482c10a commit cc14236

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

tests/sentry/api/endpoints/test_project_alert_rule_task_details.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ def test_workflow_engine_serializer(self):
7676
alert_rule_trigger=self.critical_trigger
7777
)
7878
_, _, _, self.detector, _, _, _, _ = migrate_alert_rule(self.rule)
79-
self.critical_detector_trigger, _ = migrate_metric_data_conditions(self.critical_trigger)
79+
self.critical_detector_trigger, _, _ = migrate_metric_data_conditions(self.critical_trigger)
8080

8181
self.critical_action, _, _ = migrate_metric_action(self.critical_trigger_action)
8282
self.resolve_trigger_data_condition = migrate_resolve_threshold_data_condition(self.rule)

tests/sentry/incidents/endpoints/test_organization_alert_rule_details.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ def test_workflow_engine_serializer(self):
236236
alert_rule_trigger=critical_trigger
237237
)
238238
_, _, _, self.detector, _, _, _, _ = migrate_alert_rule(self.alert_rule)
239-
self.critical_detector_trigger, _ = migrate_metric_data_conditions(critical_trigger)
239+
self.critical_detector_trigger, _, _ = migrate_metric_data_conditions(critical_trigger)
240240

241241
self.critical_action, _, _ = migrate_metric_action(critical_trigger_action)
242242
self.resolve_trigger_data_condition = migrate_resolve_threshold_data_condition(
@@ -719,7 +719,7 @@ def test_workflow_engine_serializer(self):
719719
alert_rule_trigger=critical_trigger
720720
)
721721
_, _, _, self.detector, _, _, _, _ = migrate_alert_rule(self.alert_rule)
722-
self.critical_detector_trigger, _ = migrate_metric_data_conditions(critical_trigger)
722+
self.critical_detector_trigger, _, _ = migrate_metric_data_conditions(critical_trigger)
723723

724724
self.critical_action, _, _ = migrate_metric_action(critical_trigger_action)
725725
self.resolve_trigger_data_condition = migrate_resolve_threshold_data_condition(

0 commit comments

Comments
 (0)