fix(tests): add resolve_actor_options to mock lifecycle helpers

The PR added resolve_actor_options() to StrategyActor, LLMStrategizeActor,
and LLMExecuteActor, but the three mock lifecycle SimpleNamespace objects
used by tests did not include this method, causing AttributeError at
runtime:

- features/mocks/mock_strategy_llm.py: make_mock_lifecycle() used by
  robot/helper_strategy_actor.py (strategy_actor.robot llm-json test)
  and features/steps/strategy_actor_llm_steps.py
- features/steps/llm_actors_coverage_steps.py: _make_mock_lifecycle()
  used by llm_actors_coverage.feature scenarios
- robot/helper_m5_e2e_context.py: inline lifecycle SimpleNamespace
  used by m5_e2e_verification.robot Execute Phase LLM Uses ACMS Context

All three now include resolve_actor_options=MagicMock(return_value=None)
matching the None-return contract the actors use when no custom backend
is configured.

ISSUES CLOSED: #11256
This commit is contained in:
2026-05-28 16:09:00 -04:00
committed by Forgejo
parent d5fd6b1765
commit 2dd920078a
3 changed files with 3 additions and 0 deletions
+1
View File
@@ -321,6 +321,7 @@ def make_mock_lifecycle(
get_plan=MagicMock(return_value=plan),
get_action=MagicMock(return_value=action),
resolve_actor_provider_model=MagicMock(return_value=None),
resolve_actor_options=MagicMock(return_value=None),
)
@@ -93,6 +93,7 @@ def _make_mock_lifecycle(strategy_actor=None, execution_actor=None):
get_plan=MagicMock(return_value=plan),
get_action=MagicMock(return_value=action),
resolve_actor_provider_model=MagicMock(return_value=None),
resolve_actor_options=MagicMock(return_value=None),
)
return lifecycle
+1
View File
@@ -476,6 +476,7 @@ def llm_execute_acms_context() -> None:
get_plan=MagicMock(return_value=plan),
get_action=MagicMock(return_value=action),
resolve_actor_provider_model=MagicMock(return_value=None),
resolve_actor_options=MagicMock(return_value=None),
)
actor = LLMExecuteActor(