Repository navigation
[Coverage Report] Test Coverage Report - 2026-10-02 #9366
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-10-09T04:45:25.616Z.
|
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-10-02
Overall Coverage
The firewall achieves strong coverage across the codebase, with all metrics well above typical security project targets.
🛡️ Security-Critical Path Status
src/squid-config.tssrc/domain-patterns.tssrc/docker-manager.tssrc/host-iptables.tssrc/cli.tsKey Finding: Core L7 firewall logic (
squid-config.ts,domain-patterns.ts), container management (docker-manager.ts), and kernel-level isolation (host-iptables.ts) all have complete statement and function coverage. The only minor gap is incli.tsbranch coverage (50%), affecting rare error paths.📋 Coverage Table
✅ Excellent Coverage (>90%)
🔴 Critical Gaps (<50%)
🟡 Minor Gaps (50-70%)
🔧 Function Audit
Total Functions: 3,163 defined; 2,827 covered (89.37%)
By Category:
Network Isolation & Proxying (100% coverage)
Container Lifecycle (100% coverage)
Bounded Execution / Schema Validation (35-49% coverage)
Cloud Hypervisor Support (>95% coverage)
API Proxy Credential Injection (>95% coverage)
📅 Recent Source Changes (last 7 days)
Recent commits show continuous improvement in security and API handling:
fix: report and export the api-proxy token-usage log path for arc-dind runs— API proxy improvementsfeat(api-proxy): ordered fallback models on 5xx, timeout, or model_not_supported— API resilienceFail closed on invalid API proxy model allowlists— Security hardeningmessage_deltausage #9359fix(api-proxy): read input, cache, and thinking tokens from Anthropic message_delta usage— Token accounting accuracytest: make cloud hypervisor tests portable— Infrastructure test improvementstest: refresh model routing end-to-end smoke— Route validationAdd role-specific Cloud Hypervisor enclave export layouts— Enclave isolationTest-Related Commits:
[Test Coverage] src/docker-manager.ts— Docker manager test expansion🔎 Notable Findings
🟢 Security-Critical Paths Fully Covered
docker-manager.tsreached full coverage in commit [Test Coverage] src/docker-manager.ts #9308.🔴 Bounded Execution / Schema Validation Gap
nvx/cleanup-registry.ts,finite-cardinality.ts,finite-schema.ts) have <50% statement coverage.🟡 Branch Coverage Dips in Fallback Paths
cli.tshas 50% branch coverage (error conditions and signal handling edge cases).bounded-execution/finite-disclosure.tshas 11.42% branch coverage despite 51.78% statement coverage — error paths and edge case branches are untested.✅ Cloud Hypervisor & Enclave Support Strong
🎯 Recommendations
🔴 HIGH PRIORITY
src/nvx/cleanup-registry.ts,src/bounded-execution/finite-cardinality.ts,src/bounded-execution/finite-schema.ts🟡 MEDIUM PRIORITY
src/cli.tsbranch coverage (50%),src/bounded-execution/finite-disclosure.ts(11.42% branches)🟢 LOW PRIORITY
src/microvm/network-reservation.ts(51.5% coverage)Summary
gh-aw-firewall achieves 91.31% statement coverage with all security-critical L7 filtering, network isolation, and container management paths fully tested. The codebase is production-ready for the core firewall mission.
However, three CRITICAL modules controlling resource bounds and schema validation have <50% coverage and should be prioritized for test expansion. Addressing HIGH and MEDIUM recommendations will close the remaining gaps and ensure sandbox safety guarantees hold under stress conditions.
Report Generated: 2026-10-02
Test Run: 340 files, 3,163 functions, 135+ test cases
Status: ✅ Ready for merge with recommended follow-up improvements
All reactions