Skip to content

Commit 4c024ba

Browse files
committed
tests
1 parent bf0cc37 commit 4c024ba

File tree

2 files changed

+11
-9
lines changed

2 files changed

+11
-9
lines changed

src/sentry/workflow_engine/migrations/0060_backfill_metric_alert_resolution_action_filters.py renamed to src/sentry/workflow_engine/migrations/0061_backfill_metric_alert_resolution_action_filters.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ class Migration(CheckedMigration):
7878
is_post_deployment = True
7979

8080
dependencies = [
81-
("workflow_engine", "0059_fix_high_priority_condition_triggers"),
81+
("workflow_engine", "0060_rename_azure_devops_action_to_vsts"),
8282
]
8383

8484
operations = [
Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@
55

66
class TestBackfillMetricAlertResolutionActionFilters(TestMigrations):
77
app = "workflow_engine"
8-
migrate_from = "0059_fix_high_priority_condition_triggers"
9-
migrate_to = "0060_backfill_metric_alert_resolution_action_filters"
8+
migrate_from = "0060_rename_azure_devops_action_to_vsts"
9+
migrate_to = "0061_backfill_metric_alert_resolution_action_filters"
1010

1111
def mock_aci_objects(self) -> tuple[DataConditionGroup, DataConditionGroup]:
1212
alert_rule = self.create_alert_rule(organization=self.organization)
1313
workflow = self.create_workflow(organization=self.organization)
14-
self.create_alert_rule_workflow(alert_rule=alert_rule, workflow=workflow)
14+
self.create_alert_rule_workflow(alert_rule_id=alert_rule.id, workflow=workflow)
1515

1616
critical_dcg = self.create_data_condition_group(organization=self.organization)
1717
self.create_workflow_data_condition_group(workflow=workflow, condition_group=critical_dcg)
@@ -33,9 +33,11 @@ def mock_aci_objects(self) -> tuple[DataConditionGroup, DataConditionGroup]:
3333

3434
return critical_dcg, warning_dcg
3535

36-
def create_resolve_action_filter(self, dcg: DataConditionGroup) -> None:
36+
def create_resolve_action_filter(
37+
self, dcg: DataConditionGroup, comparison: DetectorPriorityLevel
38+
) -> None:
3739
self.create_data_condition(
38-
comparison=DetectorPriorityLevel.HIGH,
40+
comparison=comparison,
3941
condition_result=True,
4042
type=Condition.ISSUE_PRIORITY_DEESCALATING,
4143
condition_group=dcg,
@@ -56,12 +58,12 @@ def setup_initial_state(self):
5658

5759
# both dcgs have a resolution action filter
5860
self.critical_dcg_2, self.warning_dcg_2 = self.mock_aci_objects()
59-
self.create_resolve_action_filter(self.critical_dcg_2)
60-
self.create_resolve_action_filter(self.warning_dcg_2)
61+
self.create_resolve_action_filter(self.critical_dcg_2, DetectorPriorityLevel.HIGH)
62+
self.create_resolve_action_filter(self.warning_dcg_2, DetectorPriorityLevel.MEDIUM)
6163

6264
# only one dcg has a resolution action filter
6365
self.critical_dcg_3, self.warning_dcg_3 = self.mock_aci_objects()
64-
self.create_resolve_action_filter(self.warning_dcg_3)
66+
self.create_resolve_action_filter(self.warning_dcg_3, DetectorPriorityLevel.MEDIUM)
6567

6668
def test_simple(self):
6769
self.assert_resolve_action_filter_exists(self.critical_dcg_1, DetectorPriorityLevel.HIGH)

0 commit comments

Comments
 (0)