forked from cleveragents/cleveragents-core
freemo
2eb31a598c
Added targeted Behave BDD feature files and step definitions to improve unit test coverage for: - decision_service.py: Full coverage of all 7 service methods (18 scenarios) - plan_apply_service.py: Branch coverage for handle_merge_failure (2 scenarios) - plan_executor.py: Edge cases for rollback, checkpoint, and parse_steps (15 scenarios) - cli/commands/plan.py: Uncovered region lines 1950-2273 (23 scenarios) - repositories.py: Remaining missed branches and lines (14 scenarios) - sandbox/checkpoint.py: Full coverage of CheckpointManager (26 scenarios) - langgraph/bridge.py: Remaining uncovered lines and branches (10 scenarios) - cli/commands/config.py: Safety net to maintain 100% coverage (42 scenarios) Total: 150 new scenarios, 596 steps, all passing. Also fixed a step definition collision in plan_lifecycle_coverage by renaming "the delete result should be false" to "the plan delete result should be false". ISSUES CLOSED: #475
24 lines
1.3 KiB
Gherkin
24 lines
1.3 KiB
Gherkin
Feature: PlanApplyService branch coverage for handle_merge_failure logger path
|
|
As a developer
|
|
I want to cover the missed branch at line 426→432 in plan_apply_service.py
|
|
So that branch-rate reaches 1.0
|
|
|
|
# The missed branch is the exception-propagation path from the
|
|
# self._logger.error() call inside handle_merge_failure (line 426).
|
|
# When the logger raises, execution skips the ``return plan`` on line 431
|
|
# and exits the function via the exception (line 432).
|
|
|
|
Scenario: handle_merge_failure propagates exception when logger.error raises
|
|
Given pas_branch a service whose logger.error will raise RuntimeError
|
|
When pas_branch I call handle_merge_failure and capture any exception
|
|
Then pas_branch a RuntimeError should have been raised
|
|
And pas_branch the plan error_details should still contain merge_conflict
|
|
And pas_branch lifecycle _commit_plan should have been invoked before the error
|
|
And pas_branch lifecycle fail_apply should have been invoked before the error
|
|
|
|
Scenario: handle_merge_failure returns plan when logger.error succeeds
|
|
Given pas_branch a service whose logger.error will succeed normally
|
|
When pas_branch I call handle_merge_failure normally
|
|
Then pas_branch the returned plan should not be None
|
|
And pas_branch the service logger.error should have been called with merge failure details
|