[Coverage Report] Test Coverage Report — 2026-06-06 #4452
Replies: 3 comments
-
|
🔮 The ancient spirits stir; the smoke test agent has passed through this discussion and left a brief omen of success. 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.
-
|
🔮 The ancient spirits stir, and the smoke test agent was here. 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.
-
|
This discussion was automatically closed because it expired on 2026-06-13T16:58:15.594Z.
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
📊 Overall Coverage
Test suite: 105 test files, 139 source files. Coverage thresholds pass (≥30% branch, ≥35% fn, ≥38% stmt/lines).
🛡️ Security-Critical Path Status
src/host-iptables.tssrc/squid-config.tssrc/docker-manager.tssrc/domain-patterns.tssrc/cli.tsAll core firewall modules (iptables rules, Squid ACL generation, Docker lifecycle, domain matching) have excellent coverage. The only security-adjacent concern is
cli.tsentry-point branch coverage at 50%.🔴 Critical Gaps (< 80% statement coverage)
src/commands/validators/network-options.ts🟡 Low Coverage (80–89% statement coverage)
src/services/agent-volumes/etc-mounts.tssrc/logs/audit-enricher.tssrc/artifact-preservation.tssrc/cli.tssrc/logs/log-parser.tssrc/squid/policy-manifest.tssrc/services/agent-volumes/docker-host-staging.tssrc/commands/logs-command-helpers.tssrc/dind-bootstrap.tsAdditional branch-coverage concerns (< 70% branch, stmt ≥ 90%)
src/services/api-proxy-service.tssrc/services/agent-environment/environment-builder.ts🔍 Notable Findings
1.
network-options.tsvalidator has half its branches untested (50% branch, 66.7% stmt)The
validateNetworkOptions()function contains conditional logic for--docker-host,--docker-host-path-prefix, and DinD hints. The uncovered branches are the invalid-argument error paths — exactly the kind of security-relevant input validation that should be tested.2.
cli.tsentry point has 50% branch coveragesrc/cli.tsis only 13 lines but has 2 branches with only 1 covered. The uncovered branch is likely therequire.mainguard. As the orchestration entry point, all execution paths here should be verified.3.
squid/policy-manifest.tshas 3 uncovered functions (70% fn)generatePolicyManifest()drives the Squid ACL rules. With 30% of its functions uncovered, there are ACL policy code paths — particularly around blocked-domain lists, DLP, and pattern-based rules — that have no test coverage. Given that this module directly controls what traffic is allowed or denied, this is the most actionable security finding.4. Large active codebase — 258 TypeScript files changed in the last 7 days
The codebase is under heavy active development. The 96.46% statement baseline is strong, but the rate of change means coverage gaps can appear quickly. The 7 files with < 70% branch coverage warrant attention before they accumulate further.
📈 Recommendations
High —
src/commands/validators/network-options.ts: Add tests covering the invalid---docker-hostand conflicting---docker-host-path-prefixerror branches. These are ~4 untested conditional paths in a validation function.High —
src/squid/policy-manifest.ts: Add tests for the 3 uncovered functions. Focus on ACL generation with blocked-domain lists and DLP-enabled configs — the branches controlling which domains are allowed or denied.Medium —
src/cli.ts: Cover the second branch (likely therequire.main === moduleguard or an error-propagation path). This is a small change in the test but closes a gap at the entry point.Medium —
src/services/api-proxy-service.ts: Branch coverage at 50% for the API credential-injection sidecar warrants investigation. Likely missing error-path tests (credential missing, upstream unreachable).Low —
src/services/agent-volumes/etc-mounts.ts: Cover the UID/GID mismatch branches inbuildEtcMounts()— these handle staged passwd/group file edge cases during DinD operation.📋 Full per-file coverage table (production files only)
src/commands/validators/network-options.tssrc/services/agent-volumes/etc-mounts.tssrc/logs/audit-enricher.tssrc/artifact-preservation.tssrc/cli.tssrc/logs/log-parser.tssrc/squid/policy-manifest.tssrc/services/agent-volumes/docker-host-staging.tssrc/commands/logs-command-helpers.tssrc/dind-bootstrap.tssrc/services/doh-proxy-service.tssrc/commands/validators/log-and-limits.tssrc/services/host-path-prefix.tssrc/config-writer.tssrc/services/api-proxy-service.tssrc/services/agent-volumes/docker-socket.tssrc/logs/log-streamer.tssrc/diagnostic-collector.tssrc/commands/build-config.tssrc/commands/validators/agent-options.tssrc/services/agent-volumes/hosts-file.tssrc/services/agent-environment/environment-builder.tssrc/squid/ssl-bump.tssrc/ssl-bump.tssrc/host-env.tssrc/logs/log-aggregator.tssrc/upstream-proxy.tssrc/services/cli-proxy-service.tssrc/commands/main-action.tssrc/parsers/volume-parsers.tssrc/container-lifecycle.tssrc/services/agent-volumes/workspace-mounts.tssrc/container-cleanup.tssrc/services/agent-environment/env-passthrough.tssrc/commands/validators/config-assembly.tssrc/compose-sanitizer.tssrc/logs/log-formatter.tssrc/domain-patterns.tssrc/services/agent-service.tssrc/services/agent-volumes/home-strategy.tssrc/config-file.tssrc/rules.tssrc/compose-generator.tssrc/pid-tracker.tssrc/option-parsers.tssrc/api-proxy-config.tssrc/cli-options.tssrc/cli-workflow.tssrc/docker-manager.tssrc/domain-patterns.ts(above)src/host-iptables.tssrc/host-iptables-cleanup.tssrc/host-iptables-network.tssrc/host-iptables-rules.tssrc/host-iptables-shared.tssrc/squid-config.tssrc/squid/access-rules.tssrc/squid/acl-generator.tssrc/squid/config-generator.tssrc/squid/config-sections.tssrc/squid/domain-acl.tssrc/squid/policy-manifest.tssrc/squid/upstream-proxy.tssrc/squid/validation.tsGenerated by test-coverage-reporter workflow. Trigger:
push— 2026-06-06Beta Was this translation helpful? Give feedback.
All reactions