[Coverage Report] Test Coverage Report — 2026-08-28 #7833
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-09-04T18:44:12.860Z.
|
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.
The project maintains strong test coverage across core infrastructure:
Summary: 253 files tracked; 92% statement coverage across 12,073 total statements.
🛡️ Security-Critical Path Status
Key Findings:
finite-disclosure.tshas critical coverage gaps (9% statements, 0% branches) — this component controls disclosure of sensitive execution data📋 Coverage Details
Fully Covered (100% statements)
src/host-iptables-rules.ts— All iptables rule generation paths testedsrc/host-iptables-cleanup.ts— Cleanup procedures fully exercisedsrc/host-iptables-network.ts— Network topology setup fully coveredsrc/host-iptables-shared.ts— Shared utilities (routing, helpers) 100% testedsrc/host-iptables-validation.ts— Rule validation logic fully coveredsrc/host-iptables.ts— Main iptables module fully testedsrc/domain-patterns.ts— Domain pattern matching 100% coveredsrc/squid-config.ts— Proxy configuration generation fully testedHigh Coverage (90–99%)
src/cli-workflow.ts— 98.63% (main CLI workflow orchestration)src/cli-options.ts— 100% (option parsing)src/cloud-hypervisor/api-client.ts— 97.33% (REST API client)src/microvm/vsock-client.ts— 97.32% (vsock communication)src/services/cli-proxy-service.ts— 98.11% (CLI proxy)🔧 Function Audit
Host iptables subsystem: All core modules (rules, cleanup, network, shared, validation) have 100% statement coverage. 9 functions tested across 5 modules, ensuring network isolation rules are properly validated and applied.
Domain pattern matching: All 3 domain-matching functions fully tested (100%). Pattern validation and subdomain resolution both have comprehensive test coverage.
CLI entry point: Main CLI module has only 7 statements (architectural constraint). 85.71% statement coverage (1 line) and only 50% branch coverage (1 untested branch) suggests one conditional path remains untested.
Bounded Execution (finite-disclosure): 44 total functions, only 2 tested (4.54%). This module controls disclosure of sensitive execution state. Coverage is critically low and requires urgent expansion.
📅 Recent Source Changes (last 7 days)
All monitored source files were updated 2026-08-28 18:37–18:38. No file-specific regressions detected from previous week.
🔎 Notable Findings
Security-critical coverage is strong: Core firewall components (iptables rules, domain patterns, squid config) all achieve 100% statement coverage, providing high confidence in network isolation behavior and domain whitelisting enforcement.
Critical coverage regression in bounded-execution/finite-disclosure.ts: Only 9% of statements and 0.26% of branches are covered. This module is responsible for controlling disclosure of sensitive agent state to monitoring systems. Of 44 functions, only 2 are exercised. This is a security control that warrants urgent test expansion.
Branch coverage gap in cli.ts: Statement coverage is 85.71%, but only 50% of branches are covered (1 of 2). This suggests an untested error path or conditional in the main entry point. Should be reviewed for error handling coverage.
Enclave and microVM features have moderate gaps:
runtime-preflight.ts(66% statements, 41% branches) androotfs.ts(71% statements, 54% branches) are newer subsystems with room for improvement.config-assembly.tshas only 25% branch coverage, indicating many conditional paths remain untested.🎯 Recommendations
runtime-preflight.tsand 70% forrootfs.ts. Focus on error handling and edge cases in initialization and configuration.Generated: 2026-08-28
Recommendation Priority: High — Address finite-disclosure.ts before next release
All reactions