Repository navigation
[Coverage Report] Test Coverage Report — 2026-10-02 #9378
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-10-09T17:50:36.591Z.
|
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.
Overall Coverage
Overall test coverage is strong across all metrics, with statements and line coverage both exceeding 90%. Branch coverage at 83.54% indicates good decision-path testing, though some edge cases remain uncovered.
🛡️ Security-Critical Path Status
Five files with CRITICAL coverage gaps requiring immediate attention:
src/nvx/cleanup-registry.tssrc/bounded-execution/finite-cardinality.tssrc/bounded-execution/finite-schema.tssrc/microvm/network-reservation.tssrc/cloud-hypervisor/host-enclave-executor.tsPrimary Concern: The three files in
src/nvx/andsrc/bounded-execution/represent critical infrastructure for network isolation and resource bounding in microVM and enclave execution. Coverage gaps here directly impact security guarantees.📋 Coverage Table
Top-Performing Files (Security-critical paths):
src/host-iptables.ts: High coverage for iptables rule generationsrc/squid-config.ts: Strong coverage for domain ACL filteringsrc/docker-manager.ts: Comprehensive coverage for container lifecycle managementsrc/cli.ts: Well-tested main entry pointsrc/domain-patterns.ts: Pattern matching validation thoroughly testedLowest Coverage Files (Requiring attention):
src/nvx/cleanup-registry.ts— 42.80% statementssrc/bounded-execution/finite-cardinality.ts— 46.03% statementssrc/bounded-execution/finite-schema.ts— 49.77% statementssrc/microvm/network-reservation.ts— 51.50% statementssrc/cloud-hypervisor/host-enclave-executor.ts— 51.67% statements🔧 Function Audit
The audit reveals:
The untested functions are concentrated in:
src/nvx/,src/cloud-hypervisor/)src/bounded-execution/)src/microvm/)These represent newer infrastructure features added for unified enclaves and Cloud Hypervisor support, suggesting incomplete test harness buildout for these subsystems.
📅 Recent Source Changes (last 7 days)
Recent changes focused on:
Several of the low-coverage files were modified recently to support:
🔎 Notable Findings
Core firewall security modules remain well-tested: Host iptables rules, Squid configuration, and domain pattern matching all have strong coverage, protecting the primary security boundary.
Enclave/microVM subsystem is undertested: The three CRITICAL gaps in
src/nvx/,src/bounded-execution/, and related files indicate that newer infrastructure for private repository enclaves and Cloud Hypervisor support needs significantly more test coverage before production hardening.Branch coverage gap: While statements are at 90.54%, branches lag at 83.54%, indicating edge cases and error handling paths are less thoroughly tested. This is especially concerning in security-critical code.
Potential resource exhaustion risk: The
src/bounded-execution/finite-*.tsfiles handle cardinality and schema validation for resource limits—undertested paths here could allow resource exhaustion attacks or bypass sandbox limits.🎯 Recommendations
HIGH PRIORITY 🔴
src/nvx/cleanup-registry.ts(42.80%),src/bounded-execution/finite-cardinality.ts(46.03%), andsrc/bounded-execution/finite-schema.ts(49.77%). These files control critical sandbox resource isolation; test coverage must reach ≥80% before MCP gateway integration rollout. Target: Add tests for cleanup workflows, cardinality enforcement, and schema validation error paths.MEDIUM PRIORITY 🟡
host-iptables.ts,squid-config.ts, anddocker-manager.ts. Add tests for: DNS resolution failures, Squid startup timeouts, Docker network conflicts, and capability drop edge cases.LOW PRIORITY ⚪
src/cloud-hypervisor/host-enclave-executor.ts(51.67%) andsrc/microvm/network-reservation.ts(51.50%) need foundational test harness buildout. Not blocking the core firewall, but essential for preview-feature stability. Defer until enclave subsystem reaches 80%.All reactions