Skip to content

Commit b474df1

Browse files
committed
fix a bunch of mocks
1 parent 7e26ba7 commit b474df1

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

tests/sentry/workflow_engine/receivers/test_project_detectors.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def test_creates_detector_with_all_components(self) -> None:
3434
project.add_team(team)
3535

3636
with mock.patch(
37-
"sentry.workflow_engine.processors.detector.send_new_detector_data"
37+
"sentry.workflow_engine.defaults.detectors.send_new_detector_data"
3838
) as mock_send:
3939
detector = ensure_default_anomaly_detector(
4040
project, owner_team_id=team.id, enabled=False
@@ -77,7 +77,7 @@ def test_creates_detector_without_team(self) -> None:
7777
"""Test that detector can be created without an owner team."""
7878
project = self.create_project()
7979

80-
with mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"):
80+
with mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"):
8181
detector = ensure_default_anomaly_detector(project, owner_team_id=None, enabled=True)
8282

8383
assert detector is not None
@@ -89,7 +89,7 @@ def test_send_new_detector_data_failure_blocks_creation(self) -> None:
8989
project = self.create_project()
9090

9191
with mock.patch(
92-
"sentry.workflow_engine.processors.detector.send_new_detector_data",
92+
"sentry.workflow_engine.defaults.detectors.send_new_detector_data",
9393
side_effect=Exception("Seer unavailable"),
9494
):
9595
with pytest.raises(Exception, match="Seer unavailable"):
@@ -102,7 +102,7 @@ def test_returns_existing_detector_without_creating_duplicates(self) -> None:
102102
"""Test that calling ensure_default_anomaly_detector twice returns the same detector."""
103103
project = self.create_project()
104104

105-
with mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"):
105+
with mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"):
106106
detector1 = ensure_default_anomaly_detector(project)
107107
detector2 = ensure_default_anomaly_detector(project)
108108

@@ -124,7 +124,7 @@ def test_creates_enabled_detector_when_both_features_enabled(self) -> None:
124124
team = project.teams.first()
125125
assert team is not None
126126

127-
with mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"):
127+
with mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"):
128128
create_default_anomaly_detector(project, user=self.user)
129129

130130
detector = Detector.objects.get(project=project, type=MetricIssue.slug)
@@ -137,7 +137,7 @@ def test_creates_disabled_detector_when_plan_feature_missing(self) -> None:
137137
"""Test that detector is created but disabled when anomaly-detection-alerts is off."""
138138
project = self.create_project()
139139

140-
with mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"):
140+
with mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"):
141141
create_default_anomaly_detector(project, user=self.user)
142142

143143
detector = Detector.objects.get(project=project, type=MetricIssue.slug)
@@ -159,7 +159,7 @@ def test_creates_detector_without_team(self) -> None:
159159
# Remove all teams
160160
project.teams.clear()
161161

162-
with mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"):
162+
with mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"):
163163
create_default_anomaly_detector(project, user=self.user)
164164

165165
detector = Detector.objects.get(project=project, type=MetricIssue.slug)
@@ -197,7 +197,7 @@ def test_context_manager_disables_metric_detector_signal(self) -> None:
197197
"""Test that disable_default_detector_creation also prevents metric detector creation."""
198198
with (
199199
disable_default_detector_creation(),
200-
mock.patch("sentry.workflow_engine.processors.detector.send_new_detector_data"),
200+
mock.patch("sentry.workflow_engine.defaults.detectors.send_new_detector_data"),
201201
):
202202
# fire_project_created=True ensures the project_created signal is sent
203203
project = self.create_project(fire_project_created=True)
@@ -248,15 +248,15 @@ def test_disable_default_detector_creation_prevents_performance_detectors(self)
248248

249249
@with_feature("projects:workflow-engine-performance-detectors")
250250
@mock.patch(
251-
"sentry.workflow_engine.processors.detector.DEFAULT_PROJECT_PERFORMANCE_DETECTION_SETTINGS",
251+
"sentry.workflow_engine.defaults.detectors.DEFAULT_PROJECT_PERFORMANCE_DETECTION_SETTINGS",
252252
{
253253
"slow_db_queries_detection_enabled": True,
254254
"large_http_payload_detection_enabled": True,
255255
"db_query_injection_detection_enabled": False,
256256
},
257257
)
258258
@mock.patch(
259-
"sentry.workflow_engine.processors.detector.get_disabled_platforms_by_detector_type",
259+
"sentry.workflow_engine.defaults.detectors.get_disabled_platforms_by_detector_type",
260260
return_value={
261261
"performance_slow_db_query": frozenset({"ruby", "php"}),
262262
},

0 commit comments

Comments
 (0)