freemo
a808c395f9
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / build (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 17s
CI / typecheck (pull_request) Successful in 35s
CI / security (pull_request) Successful in 50s
CI / unit_tests (pull_request) Successful in 2m46s
CI / integration_tests (pull_request) Successful in 3m16s
CI / docker (pull_request) Successful in 40s
CI / coverage (pull_request) Successful in 5m6s
CI / lint (push) Successful in 13s
CI / quality (push) Successful in 16s
CI / build (push) Successful in 18s
CI / security (push) Successful in 32s
CI / typecheck (push) Successful in 35s
CI / benchmark-regression (push) Has been skipped
CI / unit_tests (push) Successful in 2m52s
CI / integration_tests (push) Successful in 3m8s
CI / docker (push) Successful in 39s
CI / coverage (push) Successful in 5m53s
CI / benchmark-publish (push) Successful in 16m55s
CI / benchmark-regression (pull_request) Successful in 33m0s
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
44 lines
2.2 KiB
Gherkin
44 lines
2.2 KiB
Gherkin
Feature: Change model coverage boost
|
|
Target uncovered lines in src/cleveragents/domain/models/core/change.py:
|
|
- Line 268: DELETE branch of ChangeEntry.has_integrity_hashes
|
|
- Lines 487, 491, 499, 503: InvocationTracker Protocol method bodies
|
|
- Lines 560, 566, 570, 574, 578: ChangeSetStore Protocol method bodies
|
|
|
|
# ---- ChangeEntry.has_integrity_hashes DELETE branch (line 268) ----
|
|
|
|
Scenario: DELETE entry with before_hash reports full integrity
|
|
Given a DELETE ChangeEntry with before_hash "sha256:abc123"
|
|
Then the entry has_integrity_hashes should be true
|
|
|
|
Scenario: DELETE entry without before_hash reports missing integrity
|
|
Given a DELETE ChangeEntry without before_hash
|
|
Then the entry has_integrity_hashes should be false
|
|
|
|
# ---- InvocationTracker Protocol default method bodies (lines 487-503) ----
|
|
|
|
Scenario: InvocationTracker protocol methods are callable with default bodies
|
|
Given a bare InvocationTracker subclass instance
|
|
When I call track on the protocol with a ToolInvocation
|
|
Then track should return None
|
|
When I call get_invocations on the protocol with plan_id "plan-proto"
|
|
Then get_invocations should return None
|
|
When I call get_invocations_for_skill on the protocol with plan_id "plan-proto" and skill "s1"
|
|
Then get_invocations_for_skill should return None
|
|
When I call get_changes on the protocol with plan_id "plan-proto"
|
|
Then get_changes should return None
|
|
|
|
# ---- ChangeSetStore Protocol default method bodies (lines 560-578) ----
|
|
|
|
Scenario: ChangeSetStore protocol methods are callable with default bodies
|
|
Given a bare ChangeSetStore subclass instance
|
|
When I call start on the store protocol with plan_id "plan-store"
|
|
Then start should return None
|
|
When I call record on the store protocol with changeset_id "cs1" and a ChangeEntry
|
|
Then record should return None
|
|
When I call get on the store protocol with changeset_id "cs1"
|
|
Then protocol get should return None
|
|
When I call get_for_plan on the store protocol with plan_id "plan-store"
|
|
Then get_for_plan should return None
|
|
When I call summarize on the store protocol with changeset_id "cs1"
|
|
Then summarize should return None
|