[Coverage Report] Test Coverage Report — 2026-06-14

[github-actions[bot]]

📊 Test Coverage Report — 2026-06-14

Generated from pre-computed coverage data (Jest + Istanbul). Repository: github/gh-aw-firewall.

---

Overall Coverage

Metric	Coverage	Covered / Total
Statements	96.51% ✅	4,880 / 5,056
Branches	91.21% ✅	2,722 / 2,984
Functions	98.80% ✅	577 / 584
Lines	96.64% ✅	4,723 / 4,887

Overall health is excellent — all four metrics are above the 80% threshold. No files fall below 50% statement coverage.

---

🔴 Critical Gaps (< 50% statement coverage)

None. All source files are above 50% statement coverage.

---

🟡 Low Coverage (50–79% statement coverage)

File	Stmts	Branches	Notes
src/commands/validators/network-options.ts	66.7%	50.0% ⚠️	Validates docker-host + network opts

Only one file falls in this range. Notably its branch coverage (50%) is the lowest in the entire codebase — relevant because this module validates Docker host configuration and network options before firewall rules are applied.

---

🛡️ Security-Critical Path Status

src/host-iptables.ts (re-export facade)     ✅ 100% stmt | 100% branch | 100% fn
  └─ host-iptables-rules.ts                 ✅ 100% stmt | 100% branch | 100% fn
  └─ host-iptables-shared.ts                ✅ 100% stmt | 100% branch | 100% fn
  └─ host-iptables-cleanup.ts               ✅ 100% stmt | 100% branch | 100% fn
  └─ host-iptables-network.ts               ✅ 100% stmt | 100% branch | 100% fn

src/squid-config.ts (re-export facade)      ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/acl-generator.ts                 ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/access-rules.ts                  ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/config-generator.ts              ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/domain-acl.ts                    ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/validation.ts                    ✅ 100% stmt | 100% branch | 100% fn
  └─ squid/config-sections.ts               ✅ 100% stmt |  82.8% branch
  └─ squid/policy-manifest.ts               ⚠️  87.5% stmt |  88.2% branch | 70% fn

src/domain-patterns.ts                      ✅  97.7% stmt |  95.4% branch | 100% fn
src/docker-manager.ts (re-export facade)    ✅ 100% stmt | 100% branch | 100% fn
src/cli.ts                                  ⚠️  85.7% stmt |  50.0% branch

BRANCH COVERAGE < 70% (security-relevant):
  src/commands/validators/network-options.ts   50.0% ⚠️
  src/services/agent-volumes/etc-mounts.ts     67.9% ⚠️
  src/logs/log-parser.ts                       68.6% ⚠️
  src/services/agent-environment/environment-builder.ts  66.7% ⚠️

---

📋 Full Coverage Table (files with ≥ 10 statements, sorted by branch coverage asc)

File	Stmts	Branch	Fns	Lines
commands/validators/network-options.ts	66.7%	50.0%	100%	66.7%
services/agent-volumes/etc-mounts.ts	82.5%	67.9%	100%	82.5%
services/agent-environment/environment-builder.ts	93.5%	66.7%	100%	93.5%
logs/log-parser.ts	86.9%	68.6%	100%	87.8%
services/agent-volumes/docker-host-staging.ts	87.8%	72.4%	100%	87.2%
services/agent-volumes/workspace-mounts.ts	96.3%	75.0%	100%	96.3%
services/agent-volumes/system-mounts.ts	92.3%	75.0%	100%	92.3%
services/doh-proxy-service.ts	90.0%	75.0%	100%	90.0%
workdir-setup.ts	94.4%	79.6%	100%	94.4%
logs/log-streamer.ts	92.2%	77.8%	88.9%	92.1%
commands/validators/log-and-limits.ts	90.3%	77.6%	100%	90.3%
squid/config-sections.ts	100%	82.8%	100%	100%
pid-tracker.ts	98.8%	80.8%	100%	98.7%
commands/preflight.ts	100%	83.8%	100%	100%
logs/audit-enricher.ts	83.6%	74.1%	100%	89.4%
squid-log-reader.ts	82.2%	80.0%	100%	82.2%
config-writer.ts	85.3%	80.6%	100%	85.3%
cli.ts	85.7%	50.0%	100%	85.7%
squid/policy-manifest.ts	87.5%	88.2%	70.0%	87.2%
artifact-preservation.ts	85.4%	95.3%	100%	85.4%
ssl-bump.ts	94.0%	84.6%	100%	94.7%
host-env.ts	94.1%	80.0%	100%	95.9%
All other files	≥ 90%	≥ 86%	≥ 87%	≥ 90%

---

🔍 Notable Findings

1. src/commands/validators/network-options.ts — 50% branch coverage
Only 5 of 10 branches are exercised. The uncovered branches include the warning paths for !dockerHostCheck.valid && !dockerHostPathPrefixResolution.dockerHostPathPrefix and the dindHint warning path. These are executed when AWF detects a non-standard Docker host — an important code path for DinD/ARC runners. Tests should cover: (a) valid docker host, (b) invalid docker host with path prefix, (c) invalid docker host without path prefix, (d) dindHint with and without prefix.

2. src/services/agent-volumes/etc-mounts.ts — 67.9% branch coverage
This module controls which /etc identity files (passwd, group) are mounted into the agent container. Uncovered branches include: the resolveUniqueName counter loop (when both the base name and ${name}-${id} are taken), and the fallback || passwdPath / || groupPath branches when synthesizeIdentityFile returns undefined. These are error-recovery paths for minimal ARC containers.

3. src/logs/log-parser.ts — 68.6% branch coverage
The log parser has 22 uncovered branches out of 70. The coverage gap is in the field-extraction and protocol-parsing logic — specifically the handling of malformed log lines where individual fields are "-" or absent. Incomplete parsing coverage means that unusual Squid log entries (e.g., from blocked UDP or ICP traffic) may silently return null without being auditable.

4. src/squid/policy-manifest.ts — 70% function coverage
3 of 10 functions are not covered by tests. policy-manifest.ts generates the structured policy document that the audit enricher uses to attribute blocked connections to specific rules. Uncovered functions likely include edge cases for blockedDomains with regex patterns and allowHostPorts. Since the enricher depends on this manifest for correct attribution, gaps here reduce the accuracy of the audit log.

---

📈 Recommendations

1. High — network-options.ts branch coverage: Add tests for the four Docker-host warning scenarios. The 50% branch coverage on a security-gating validator is the highest-priority gap. Target: ≥ 90% branches. Relevant test file: src/commands/validators/config-assembly.test.ts or a new network-options.test.ts.

2. High — etc-mounts.ts DinD error paths: Add tests simulating: (a) synthesizeIdentityFile returning undefined (disk full / permission denied in staging), (b) duplicate username collision triggering the counter loop in resolveUniqueName. These are silent fallback paths that determine what identity files land in the container.

3. Medium — log-parser.ts malformed-line branches: Add test cases for partially-formed Squid log lines: timestamp only, missing user-agent field, "-" dest IP/port, and protocol version variants. Target: ≥ 80% branches.

4. Low — squid/policy-manifest.ts function gaps: Ensure generatePolicyManifest is called with: blockedDomains containing regex patterns, allowHostPorts set, and enableDlp: true. This would cover the 3 missing functions and improve audit attribution reliability.

---

Test file count: ~120 test files across src/ and tests/integration/. Source files covered: ~100 TypeScript modules.
Generated by test-coverage-reporter workflow. Trigger: schedule

Generated by Test Coverage Reporter · ◷

• expires on Jun 21, 2026, 6:11 AM UTC

[github-actions[bot]]

This discussion was automatically closed because it expired on 2026-06-21T06:11:49.114Z.

Closed by Workflow