test(coverage): add Behave BDD tests to improve unit test coverage across 53 source modules #646

2026-03-09T04:37:51Z

freemo commented

2026-03-09 04:37:51 +00:00

Summary

Add 53 new Behave BDD feature files and corresponding step definition files targeting uncovered lines identified in build/coverage.xml. Also fix AmbiguousStep conflicts in 7 pre-existing step files.

Closes #645

What changed

New test files (53 features + 53 step definitions)

Coverage boost tests for: 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.

Modified pre-existing files (AmbiguousStep fixes)

features/steps/bridge_remaining_coverage_steps.py
features/steps/config_service_coverage_steps.py
features/steps/lsp_registry_steps.py
features/steps/plan_executor_new_coverage_steps.py
features/steps/sandbox_manager_coverage_steps.py
features/steps/uko_ontology_registry_steps.py
features/steps/uko_ontology_steps.py
features/consolidated_config.feature
features/sandbox_manager_coverage.feature

Test results

All 53 new features pass (337 features passed total). The 8 failing features in CI are pre-existing failures on master unrelated to this PR:

bridge_remaining_coverage.feature (2 scenarios)
consolidated_config.feature (2 scenarios)
consolidated_misc.feature (3 scenarios)
plan_executor_new_coverage.feature (15 scenarios)
retry_patterns.feature (1 scenario)
security_async.feature (3 scenarios)
uko_ontology.feature (2 scenarios)
uko_ontology_registry.feature (5 scenarios)

Approach

Analyzed build/coverage.xml to identify uncovered lines across all 53 target source modules
Used unique step text prefixes (e.g., coverage boost, r2cov, dscb-, ssc-) to avoid AmbiguousStep collisions with ~450 existing step files
Each feature file exercises specific uncovered branches: error paths, edge cases, protocol bodies, empty-string handling, log capture, etc.
All tests follow Behave BDD conventions per CONTRIBUTING.md

## Summary Add 53 new Behave BDD feature files and corresponding step definition files targeting uncovered lines identified in `build/coverage.xml`. Also fix AmbiguousStep conflicts in 7 pre-existing step files. Closes #645 ## What changed ### New test files (53 features + 53 step definitions) - Coverage boost tests for: 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. ### Modified pre-existing files (AmbiguousStep fixes) - `features/steps/bridge_remaining_coverage_steps.py` - `features/steps/config_service_coverage_steps.py` - `features/steps/lsp_registry_steps.py` - `features/steps/plan_executor_new_coverage_steps.py` - `features/steps/sandbox_manager_coverage_steps.py` - `features/steps/uko_ontology_registry_steps.py` - `features/steps/uko_ontology_steps.py` - `features/consolidated_config.feature` - `features/sandbox_manager_coverage.feature` ## Test results All 53 new features pass (337 features passed total). The 8 failing features in CI are **pre-existing failures on master** unrelated to this PR: - `bridge_remaining_coverage.feature` (2 scenarios) - `consolidated_config.feature` (2 scenarios) - `consolidated_misc.feature` (3 scenarios) - `plan_executor_new_coverage.feature` (15 scenarios) - `retry_patterns.feature` (1 scenario) - `security_async.feature` (3 scenarios) - `uko_ontology.feature` (2 scenarios) - `uko_ontology_registry.feature` (5 scenarios) ## Approach - Analyzed `build/coverage.xml` to identify uncovered lines across all 53 target source modules - Used unique step text prefixes (e.g., `coverage boost`, `r2cov`, `dscb-`, `ssc-`) to avoid AmbiguousStep collisions with ~450 existing step files - Each feature file exercises specific uncovered branches: error paths, edge cases, protocol bodies, empty-string handling, log capture, etc. - All tests follow Behave BDD conventions per CONTRIBUTING.md

freemo force-pushed task/unit-test-coverage-boost from fa068a9e57 to e557569ebf

2026-03-09 15:41:49 +00:00

Compare

freemo force-pushed task/unit-test-coverage-boost from e557569ebf to ba4507b2d9

2026-03-09 16:27:47 +00:00

Compare

freemo force-pushed task/unit-test-coverage-boost from ba4507b2d9 to 0b4b71ccc2

2026-03-09 16:56:44 +00:00

Compare

freemo force-pushed task/unit-test-coverage-boost from 0b4b71ccc2 to a808c395f9

2026-03-09 17:02:38 +00:00

Compare

freemo scheduled this pull request to auto merge when all checks succeed 2026-03-09 17:06:41 +00:00

freemo merged commit a808c395f9 into master

2026-03-09 17:08:25 +00:00

freemo deleted branch task/unit-test-coverage-boost

2026-03-09 17:08:25 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#646