Repository navigation
[Coverage Report] Test Coverage Report — 2026-10-07 #9535
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-10-14T05:27:13.502Z.
|
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
Total Tests: 135 passed | Execution Time: ~4.4 seconds
🛡️ Security-Critical Path Status
Coverage of Security-Sensitive Modules:
squid-config.tshost-iptables.tsdomain-patterns.tslogger.tsKey Finding: Squid domain filtering and iptables network isolation have strong coverage, but branch coverage in
host-iptables.ts(55.55%) and missing baseline fordomain-patterns.tsrequire attention.📋 Coverage Table
✅ Fully Covered (100%)
logger.tssquid-config.tscli-workflow.tshost-iptables.ts🔴 Critical Gaps (<50%)
docker-manager.tscli.tsnvx/cleanup-registry.tsbounded-execution/finite-cardinality.ts🔧 Function Audit
High-Impact Untested Functions:
cli.ts— 0 of 10 functions covereddocker-manager.ts— 1 of 25 functions covered (4%)host-iptables.ts(branch gaps) — 55.55% branch coveragenvx/cleanup-registry.ts— 42.8% statement coveragebounded-execution/finite-cardinality.ts— 46.03% statement coverage📅 Recent Source Changes (last 7 days)
Unable to retrieve recent git history due to environment constraints. Review the following likely changed areas:
src/nvx/cleanup-registry.ts) — new or recently modifiedsrc/bounded-execution/finite-*.ts) — new resource-control subsystemsrc/microvm/network-reservation.ts) — platform previewsrc/cli.ts,src/docker-manager.ts) — ongoing maintenance🔎 Notable Findings
Security-critical path coverage is asymmetric: Squid domain filtering (100% ✅) and iptables network rules (83.63% ✅) are well-tested, but the orchestration that invokes them (
cli.ts,docker-manager.ts) has 0–18% coverage. Any bug in container startup could bypass these controls entirely.Branching logic in
host-iptables.ts(55.55% branch coverage) is underdeveloped: Error handling during rule validation and fallback scenarios for incompatible host configurations are untested. This affects deployment on exotic network stacks (overlay networks, firewall managers, etc.).New subsystems lack baseline coverage:
nvx/cleanup-registry.ts(42.8%),bounded-execution/finite-cardinality.ts(46.03%), andmicrovm/network-reservation.ts(51.5%) are all below the global 38% threshold and likely represent new features without corresponding test suites. These affect reliability of optional features (NVX, gVisor sandboxing, microVM backend).Function coverage void in
docker-manager.ts: Only 1 of 25 functions are covered. The Docker Compose integration — container health checks, log streaming, graceful shutdown — is almost entirely untested, creating blind spots for infrastructure stability.🎯 Recommendations
🔴 HIGH — Must Address Before Merge
Add integration tests for
cli.tsentry pointExpand
docker-manager.tstest coverage to ≥50%🟡 MEDIUM — Backlog for Next Sprint
Add branch coverage for error paths in
host-iptables.tsEstablish baseline test suites for new subsystems
nvx/cleanup-registry.ts,bounded-execution/finite-*.ts,microvm/network-reservation.ts🔵 LOW — Nice to Have
domain-patterns.tsto baseline coverage trackingSummary
Overall Assessment: Coverage thresholds are met globally (38.39% statements), but this masks critical gaps in the orchestration layer (
cli.ts,docker-manager.ts). Security-sensitive filtering (squid-config.ts) and isolation (host-iptables.tsstatements) are well-covered, but the infrastructure plumbing that invokes them has minimal test safety. Recommended priority: fix orchestration coverage before expanding into new subsystems.Trend: New NVX, bounded-execution, and microVM subsystems represent feature growth, but lack corresponding test investment. These should reach baseline (≥50%) before being considered production-ready.
All reactions