Skip to content

Commit ae176eb

Browse files
committed
fix(seer): Fix autofix pr_iteration test mocks and feedback kwarg
Patch get_autofix_run_state (not get_autofix_agent_state) in pr_iteration tests, and add the feedback=None kwarg to trigger_autofix_agent call assertions.
1 parent 5bc7bf9 commit ae176eb

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

tests/sentry/seer/endpoints/test_group_ai_autofix.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,7 @@ def test_stopping_point(self, mock_trigger_explorer):
196196
run_id=None,
197197
user_context=None,
198198
insert_index=None,
199+
feedback=None,
199200
)
200201

201202
@patch("sentry.seer.endpoints.group_ai_autofix.trigger_autofix_agent")
@@ -220,6 +221,7 @@ def test_insert_index_passed_through(self, mock_trigger_explorer):
220221
run_id=42,
221222
user_context=None,
222223
insert_index=3,
224+
feedback=None,
223225
)
224226

225227
@patch("sentry.seer.endpoints.group_ai_autofix.publish_action", autospec=True)
@@ -277,7 +279,7 @@ def test_coding_agent_handoff_skips_action(self, mock_handoff, mock_publish):
277279
mock_publish.assert_not_called()
278280

279281
@with_feature("organizations:autofix-pr-iteration")
280-
@patch("sentry.seer.endpoints.group_ai_autofix.get_autofix_agent_state")
282+
@patch("sentry.seer.endpoints.group_ai_autofix.get_autofix_run_state")
281283
@patch("sentry.seer.endpoints.group_ai_autofix.trigger_autofix_agent")
282284
def test_pr_iteration(self, mock_trigger_explorer, mock_get_state):
283285
group = self.create_group()
@@ -310,6 +312,7 @@ def test_pr_iteration(self, mock_trigger_explorer, mock_get_state):
310312
run_id=123,
311313
user_context=None,
312314
insert_index=None,
315+
feedback=None,
313316
)
314317

315318
@patch("sentry.seer.endpoints.group_ai_autofix.trigger_autofix_agent")
@@ -342,7 +345,7 @@ def test_pr_iteration_requires_run_id(self, mock_trigger_explorer):
342345
mock_trigger_explorer.assert_not_called()
343346

344347
@with_feature("organizations:autofix-pr-iteration")
345-
@patch("sentry.seer.endpoints.group_ai_autofix.get_autofix_agent_state")
348+
@patch("sentry.seer.endpoints.group_ai_autofix.get_autofix_run_state")
346349
@patch("sentry.seer.endpoints.group_ai_autofix.trigger_autofix_agent")
347350
def test_pr_iteration_requires_existing_pr(self, mock_trigger_explorer, mock_get_state):
348351
group = self.create_group()

0 commit comments

Comments
 (0)