Skip to content

Automated coverage is below the 80% target #30

@nficano

Description

@nficano

The current test suite does not meet the requested 80% coverage threshold. Running dotnet test ARCP.slnx --collect:"XPlat Code Coverage" --results-directory TestResults/review-coverage on May 23, 2026 passed 106 tests, and merging the two Cobertura reports with ReportGenerator produced 72.2% line coverage and 58.3% branch coverage. The largest gaps are Arcp.Client.Transport.StdioTransport at 0%, Arcp.Client.Transport.WebSocketClientTransport at 0%, ARCP.Runtime.Store.EventLog at 44.8%, ARCP.Runtime.JobContext at 35.8%, and several identifier and trace helper modules below the threshold.

Fix prompt: raise measured coverage above 80% line and branch coverage with focused tests rather than broad snapshot tests. Add unit tests for stdio framing, malformed-line handling, stream ownership, WebSocket close/error/send behavior using a fake or loopback WebSocket, event-log replay and eviction edge cases, JobContext authority and event emission paths, and property-based FsCheck tests for identifier round-trips, glob matching, lease subset laws, and codec round-trips. Add a documented coverage command or CI threshold so future changes can reproduce the same line and branch metrics.

Metadata

Metadata

Assignees

No one assigned

    Labels

    severity:mediumMedium severitytestingAutomated test coverage and test quality

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions