forked from HAL9000/cleveragents-core
freemo
a808c395f9
Add 53 new .feature files and corresponding step definition files targeting uncovered lines identified in build/coverage.xml. Fix AmbiguousStep conflicts in 7 pre-existing step files by disambiguating step text. New tests cover: ACP clients/facade, actor CLI/config, application container, ACMS service/strategies, async worker, automation profile CLI, autonomy guardrail, bridge, change model, config CLI/service, context service, cross-plan correction, database models, decision service, decomposition clustering/service, discovery handler, langchain chat provider, langgraph nodes, materializers, multi-project service, plan apply/CLI/lifecycle/model/ preflight/resume/service, PostgreSQL analyzer, project CLI/context CLI, provider registry, reactive application/route, repositories, resolver handler, resource registry service, resume model, retry patterns, sandbox protocol, server CLI, skill CLI/service, skills registry, subplan execution/service, system CLI, UKO loader, UoW, and YAML template engine. Closes #645
68 lines
3.6 KiB
Gherkin
68 lines
3.6 KiB
Gherkin
Feature: ACMS advanced strategies coverage boost
|
|
Cover untested code paths in acms_advanced_strategies.py including
|
|
ArceStrategy.max_iterations property, ARCE non-convergence loop path,
|
|
_refine_scores empty-fragments guard, _refine_scores prefix-boost branch,
|
|
and the capabilities properties on TemporalArchaeologyStrategy and
|
|
PlanDecisionContextStrategy.
|
|
|
|
# -----------------------------------------------------------------------
|
|
# ArceStrategy — max_iterations property (line 97)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: ArceStrategy exposes max_iterations via property
|
|
Given an ArceStrategy with max_iterations set to 3
|
|
Then the max_iterations property returns 3
|
|
|
|
Scenario: ArceStrategy default max_iterations is 5
|
|
Given an ArceStrategy with default parameters
|
|
Then the max_iterations property returns 5
|
|
|
|
# -----------------------------------------------------------------------
|
|
# ArceStrategy — refinement loop does NOT converge early (line 158)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: ARCE refinement loop runs multiple iterations without early convergence
|
|
Given an ArceStrategy with max_iterations set to 2 and convergence_threshold 0.0
|
|
And a set of fragments with shared UKO prefixes that produce score changes each iteration
|
|
When the ArceStrategy assembles the fragments with a large budget
|
|
Then the result contains fragments ordered by refined score
|
|
|
|
# -----------------------------------------------------------------------
|
|
# ArceStrategy._refine_scores — empty fragments guard (line 208)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: ARCE _refine_scores returns original scores for empty fragment list
|
|
Given an ArceStrategy with default parameters
|
|
When _refine_scores is called with an empty fragment list and some scores
|
|
Then the original scores dict is returned unchanged
|
|
|
|
# -----------------------------------------------------------------------
|
|
# ArceStrategy._refine_scores — prefix-boost branch (lines 230-231)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: ARCE _refine_scores boosts non-anchor fragments that share an anchor prefix
|
|
Given an ArceStrategy with default parameters
|
|
And four fragments where two share UKO prefix with the anchor fragment
|
|
When _refine_scores is called with those fragments and initial scores
|
|
Then non-anchor fragments sharing the anchor prefix receive a 0.05 boost
|
|
|
|
# -----------------------------------------------------------------------
|
|
# TemporalArchaeologyStrategy.capabilities (lines 267-269)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: TemporalArchaeologyStrategy capabilities include temporal and graph support
|
|
Given a TemporalArchaeologyStrategy instance
|
|
Then its capabilities have supports_temporal_archaeology true
|
|
And its capabilities have supports_graph_navigation true
|
|
And its capabilities have supports_semantic_search false
|
|
|
|
# -----------------------------------------------------------------------
|
|
# PlanDecisionContextStrategy.capabilities (lines 364-365)
|
|
# -----------------------------------------------------------------------
|
|
|
|
Scenario: PlanDecisionContextStrategy capabilities include temporal support only
|
|
Given a PlanDecisionContextStrategy instance
|
|
Then its capabilities have supports_temporal_archaeology true
|
|
And its capabilities have supports_graph_navigation false
|
|
And its capabilities have supports_semantic_search false
|