Process coverage hot spots: PurchaseProcesses 21%, PortfolioProcesses 35%, PlanProcesses 51% #18

Open
opened 2026-10-01 22:50:49 +00:00 by KofTwentyTwo · 0 comments
KofTwentyTwo commented 2026-10-01 22:50:49 +00:00 (Migrated from github.com)

Summary

The bundle floors (80% line, 60% branch, pom.xml:38) pass at 88.0% / 69.1% (local target/site/jacoco/jacoco.csv), but the native QQQ process classes that users actually click are the least covered: PurchaseProcesses 21% lines, PortfolioProcesses 35%, PlanProcesses 51%, CalendarWorkflows 68%, CarlMetadata 76%. The three largest classes (CarlClientWorkflows 1,077 lines, RentalRecords 1,051, CarlService 1,006) sit at 81% to 91%. No evidence file publishes per-class numbers, so the docs only ever quote the floors.

Suggested resolution

Add native-process HTTP tests for the purchase, portfolio and plan processes (the existing Carl*NativeTest pattern), and publish the per-class JaCoCo CSV in the evidence.

Priority rationale

P2: the user-facing process layer is the least tested layer.


Filed from the 2026-10-01 real-versus-fake audit of Carl (branch feature/cicd-liquibase-gitops, HEAD 5414d7a, main 1b334d1). Four read-only reviews covered domain and services, QQQ metadata and data model, tests/CI/evidence, and docs/integrations. A local offline build of a Carl copy against the installed foundation 0.5.0-SNAPSHOT is in progress; its result will be added as a comment. Existing issues #3, #4 and #5 (real Monarch accounts, Authentik with real data, CI/CD with Liquibase and ArgoCD) are referenced rather than duplicated.

## Summary The bundle floors (80% line, 60% branch, `pom.xml:38`) pass at 88.0% / 69.1% (local `target/site/jacoco/jacoco.csv`), but the native QQQ process classes that users actually click are the least covered: `PurchaseProcesses` 21% lines, `PortfolioProcesses` 35%, `PlanProcesses` 51%, `CalendarWorkflows` 68%, `CarlMetadata` 76%. The three largest classes (`CarlClientWorkflows` 1,077 lines, `RentalRecords` 1,051, `CarlService` 1,006) sit at 81% to 91%. No evidence file publishes per-class numbers, so the docs only ever quote the floors. ## Suggested resolution Add native-process HTTP tests for the purchase, portfolio and plan processes (the existing `Carl*NativeTest` pattern), and publish the per-class JaCoCo CSV in the evidence. ## Priority rationale P2: the user-facing process layer is the least tested layer. --- Filed from the 2026-10-01 real-versus-fake audit of Carl (branch `feature/cicd-liquibase-gitops`, HEAD `5414d7a`, main `1b334d1`). Four read-only reviews covered domain and services, QQQ metadata and data model, tests/CI/evidence, and docs/integrations. A local offline build of a Carl copy against the installed foundation 0.5.0-SNAPSHOT is in progress; its result will be added as a comment. Existing issues #3, #4 and #5 (real Monarch accounts, Authentik with real data, CI/CD with Liquibase and ArgoCD) are referenced rather than duplicated.
This repo is archived. You cannot comment on issues.
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/carl#18