[Coverage Report] Test Coverage Report — 2026-09-15 #8583
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-09-22T01:24:31.257Z.
|
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-15
Overall Coverage
Summary: Overall test coverage is strong across all metrics, with >92% line and statement coverage. Branch coverage is slightly lower at 86.61%, indicating some conditional paths need better coverage.
🛡️ Security-Critical Path Status
The following files are essential for network security and deserve special attention:
src/host-iptables-rules.tssrc/host-iptables-shared.tssrc/domain-patterns.tssrc/domain-validation.tssrc/domain-matchers.tssrc/docker-manager.tssrc/cli.tsKey Finding: Critical network isolation files (
host-iptables-*,domain-*) have excellent coverage. However,src/cli.ts(main entry point) has gaps at 85.71% lines and only 50% branch coverage that should be addressed.📋 Coverage Table
High-Risk Files (Statements <80%):
src/bounded-execution/finite-disclosure.tssrc/microvm/network-reservation.tssrc/enclave/runtime-preflight.tssrc/microvm/rootfs.tssrc/commands/validators/config-assembly.tssrc/artifact-permissions.tssrc/cli.tssrc/config-writer.tssrc/host-env.ts🔧 Function Audit
Functions with Incomplete Coverage:
finite-disclosure.ts(~6 functions, 38% coverage): Contains bounded execution timeout logic; low branch coverage (25.89%) suggests conditional path testing is insufficientnetwork-reservation.ts(~8 functions, 50% coverage): Network namespace and resource setup code; critical for isolation layerruntime-preflight.ts(~3 functions, 66% coverage): Pre-flight validation for sandboxed runtimesconfig-assembly.ts(~5 functions, 73% coverage): Config validation and assembly; 25% branch coverage is concerning📅 Recent Source Changes (last 7 days)
To identify if recent changes introduced coverage regressions:
finite-disclosure.ts,network-reservation.ts, orconfig-assembly.tswere recently modified🔎 Notable Findings
Critical Gap in Bounded Execution (
finite-disclosure.ts): At 38.12% statement coverage and 25.89% branch coverage, this module is significantly under-tested. This is especially concerning given its role in timeout enforcement and execution control.Strong Coverage of Core Network Isolation: The foundational network security modules (
host-iptables-*.ts,domain-*.ts) achieve 98-100% coverage, indicating excellent quality for the core firewall logic.Branch Coverage Lags Behind Line Coverage: While line coverage is 94.13%, branch coverage is 86.61% (7.52% gap). This suggests conditional logic and error paths are not consistently tested across the codebase.
CLI Entry Point Needs Refinement:
src/cli.tshas only 50% branch coverage despite 85.71% line coverage, indicating edge cases and error handling paths need additional test scenarios.🎯 Recommendations
HIGH PRIORITY
Increase
finite-disclosure.tstest coverage to ≥80%: This file has the lowest coverage in the codebase at 38.12%. Focus on:Improve branch coverage in
config-assembly.ts: Statements are at 73.68% but branches are critically low at 25%. Add tests for:MEDIUM PRIORITY
3. Expand
cli.tsbranch coverage (50% → ≥80%): The main entry point needs better error path and edge case coverage:LOW PRIORITY
4. Incrementally address remaining gaps:
network-reservation.ts(50%),runtime-preflight.ts(66%), androotfs.ts(71%) should be brought to ≥85% over the next sprint, focusing on gap areas identified in coverage reports.Baseline maintained: All critical security-path files remain at or above desired thresholds. The overall 94.13% line coverage reflects a mature, well-tested codebase. Focus areas should be the identified high-risk modules with external dependencies or complex conditional logic.
All reactions