[Coverage Report] Test Coverage Report — 2026-06-09 #4650
Replies: 1 comment
-
|
🔮 The ancient spirits stir above the coverage veil. Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
Beta Was this translation helpful? Give feedback.
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
108 test files · 155 source files · ~29,558 test lines
🔴 Critical Gaps (< 50% statement coverage)
None. All 155 source files are above 50% statement coverage.
🟡 Low Coverage (50–79% statement coverage)
src/commands/validators/network-options.tsOnly 1 file falls in this range. It covers Docker host detection and domain resolution validation — worth improving.
🛡️ Security-Critical Path Status
src/host-iptables.tssrc/squid-config.tssrc/docker-manager.tssrc/domain-patterns.tssrc/cli.tssrc/cli.ts's 50% branch coverage (1/2 branches) is therequire.main === moduleguard — a module-boundary check that unit tests inherently skip. Acceptable.📋 Files with Branch Coverage < 70%
src/cli.tssrc/commands/validators/network-options.tssrc/services/api-proxy-service.tssrc/dind-bootstrap.tssrc/services/agent-environment/environment-builder.tssrc/services/agent-volumes/etc-mounts.tssrc/logs/log-parser.ts🔍 Notable Findings
1.
src/commands/validators/network-options.ts— 50% branch, 66.7% stmtThe validator for Docker host resolution has 3 conditional warning paths (lines 47, 59, 64) around
dockerHostCheck.validand DinD hint detection. The warning branches for "external Docker host detected" and "DinD hint without path prefix" are not covered. These code paths are part of the DinD support feature added in recent commits.2.
src/logs/log-parser.ts— 68.6% branch (22 uncovered out of 70)The log parser has extensive branching for Squid access log format variations, null-field handling, and partial-line edge cases. With 22 uncovered branches, there are likely edge cases in malformed-log recovery and timestamp parsing that lack test coverage. This is medium-risk since it affects audit integrity.
3.
src/services/agent-volumes/etc-mounts.ts— 67.8% branch (9 uncovered out of 28)The
/etcfile mounting logic determines which host files are selectively exposed to the agent container. Uncovered branches likely include conditional paths for files that may or may not exist on the host (e.g.,/etc/alternatives,/etc/ld.so.cache). These missing-file fallback paths are security-sensitive.4.
src/dind-bootstrap.ts— 66.7% branch (11 uncovered out of 33)The Docker-in-Docker bootstrap has 11 uncovered branches. Given that DinD support is an advanced configuration path, error handling and early-exit branches may lack test scenarios.
📈 Recommendations
High —
src/commands/validators/network-options.ts: Add tests for the 3 warning branches triggered by external Docker host (dockerHostCheck.valid === false) and DinD hint with missing--docker-host-path-prefix. These are validation paths users hit when misconfiguring DinD.Medium —
src/logs/log-parser.ts: Add tests for malformed Squid log lines (missing fields, truncated entries, non-standard timestamps). The 22 uncovered branches suggest the "happy path" is well-tested but edge cases in the audit log pipeline are not.Medium —
src/services/agent-volumes/etc-mounts.ts: Add tests simulating missing host/etcfiles (e.g., when/etc/ld.so.cachedoes not exist). These code paths determine which files are mounted into the agent sandbox — missed branches risk incorrect security posture.Low —
src/dind-bootstrap.ts: Cover error handling branches for failed Docker socket detection and probe timeouts. Lower priority since DinD is an optional advanced feature.Generated by test-coverage-reporter workflow. Trigger:
push· Run: 27241154861Beta Was this translation helpful? Give feedback.
All reactions