[Coverage Report] Test Coverage Report — 2026-09-24 #8952
Replies: 1 comment
**** |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
📊 Test Coverage Report — 2026-09-24
Overall Coverage
Summary: The codebase maintains strong overall coverage across statements, lines, and functions (89–93%). However, branch coverage lags at 84.61%, indicating conditional paths are not fully exercised. This is particularly important for security-critical paths where edge cases must be tested.
🛡️ Security-Critical Path Status
The following files are essential for network isolation and firewall enforcement:
src/cli.tssrc/docker-manager.tssrc/squid-config.tssrc/domain-patterns.tssrc/host-iptables-rules.tssrc/host-iptables-shared.tsKey Concern:
src/cli.tsshows 50% branch coverage, meaning half of all conditional branches in the main entry point are untested. This is a critical gap for a security wrapper.📋 Coverage Table
Top Gap Areas (pre-identified from test artifacts):
src/nvx/cleanup-registry.tssrc/bounded-execution/finite-cardinality.tssrc/bounded-execution/finite-schema.tssrc/microvm/network-reservation.tssrc/bounded-execution/finite-disclosure.ts🔧 Function Audit
Notable Coverage Patterns:
100% Coverage Files (27 files):
src/api-proxy-config-domains.ts— Domain validation fully tested ✅src/api-proxy-config-warnings.ts— Warning logic fully tested ✅src/chroot-home-setup.ts— Home directory setup fully tested ✅src/cli-options.ts— CLI option parsing fully tested ✅>95% Coverage Files (good security baseline):
src/artifact-preservation.ts(95.03%)src/capability-filter.ts(94.82%)src/cli-workflow.ts(98.75%)<80% Coverage Files (requires attention):
src/cli.ts— 85.71% overall, 50% branchessrc/artifact-permissions.ts(82.08%)📅 Recent Source Changes (Last 7 Days)
Note: Git history access was restricted in this environment. Please review recent commits manually via:
git log --since="7 days ago" --oneline -- src/Key areas to re-test after recent changes:
src/bounded-execution/*files (currently < 50% coverage)src/nvx/cleanup-registry.ts(critical path)src/cli.ts🔎 Notable Findings
Branch Coverage Crisis in CLI Entry Point:
src/cli.tshas only 50% branch coverage. This is the main orchestration point for the firewall and must have comprehensive conditional testing. This gap affects error handling, signal handling, and edge cases in container startup.Bounded Execution Framework Under-Tested: Three files in
src/bounded-execution/have < 50% statement coverage (42–49%), andfinite-disclosure.tshas only 11.42% branch coverage. These implement disclosure filtering and cardinality limits — security-critical constraints that must be bulletproof.Registry Cleanup Missing Error Paths:
src/nvx/cleanup-registry.ts(42.8% coverage) lacks testing for error conditions in NVX container registry cleanup. Failures here could leak credentials or leave dangling resources.Strong Coverage on Proxy Configuration: Files like
api-proxy-config-domains.tsandapi-proxy-config-warnings.tsare at 100% coverage, indicating good test discipline for the API proxy layer.🎯 Recommendations
🔴 HIGH PRIORITY
Increase
src/cli.tsbranch coverage from 50% to ≥90%Complete bounded-execution test suite
finite-cardinality.ts,finite-schema.ts, andfinite-disclosure.tsto ≥80% coverage🟡 MEDIUM PRIORITY
src/nvx/cleanup-registry.tsfailure modes: network errors, permission denied, registry unavailable⚪ LOW PRIORITY
📌 Next Steps
npm run test:coverageto generate detailed coverage reports. Review files flagged in the Coverage Table above.cleanup-registry.ts,finite-cardinality.ts,finite-schema.ts) for untested error paths.cli.ts).Generated: 2026-09-24 (pre-computed coverage data, coverage summary from
coverage/coverage-summary.json)All reactions