[Coverage Report] Test Coverage Report — 2026-06-11 #4758
Replies: 6 comments
-
|
🔮 The ancient spirits stir: the smoke test agent was here, and the latest omens favor this discussion. 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 has walked this discussion. The omens are clear, the build held, and the firewall stood firm. 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: the smoke test agent passed through this discussion, and the run remains in alignment. 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; the smoke-test agent has passed through this discussion and left a brief oracle-mark upon the record. 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 stirred, and the smoke test omen reads true. The Codex agent passed through the fire and returned with a clean sign. 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-18T15:14:21.953Z.
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Overall Coverage
160 test files across 155 source files.
🔴 Critical Gaps (< 50% statement coverage)
None.
🟡 Low Coverage (50–79% statement coverage)
src/commands/validators/network-options.ts🛡️ Security-Critical Path Status
src/host-iptables.tssrc/squid-config.tssrc/docker-manager.tssrc/domain-patterns.tssrc/cli.tssrc/cli.tsbranch gap: therequire.main === moduleguard (line 11) is never entered in tests since tests import the module rather than execute it as a script. Low risk — not a logic gap.📋 Files with Notable Branch Gaps (< 70%)
src/commands/validators/network-options.tssrc/cli.tsrequire.main === moduleguard (line 11)src/services/api-proxy-service.tsthrowwhenproxyIpmissing (line 33)src/services/agent-environment/environment-builder.tssrc/dind-bootstrap.tssrc/logs/log-parser.tssrc/services/agent-volumes/etc-mounts.ts/etcbind-mount guards🔍 Notable Findings
1.
src/commands/validators/network-options.ts— external Docker host paths uncoveredLines 47–74: the
!dockerHostCheck.validbranch (triggered whenDOCKER_HOSTpoints to an external TCP daemon) is never tested. No test currently setsDOCKER_HOSTto a non-standard value when callingvalidateNetworkOptions.2.
src/logs/log-parser.ts— IPv6 bracketed destination parsing uncoveredLines 181–189: the
rawDest.startsWith('[')branch for entries like[2001:db8::1]:443is entirely unexercised. Risk: incorrect metrics inawf logs statsoutput when agents connect to IPv6 upstreams.3.
src/dind-bootstrap.ts— DinD bootstrap failure paths uncoveredLines 23, 28, 33, 70, 78:
assertAbsolutePaththrows and non-standard Docker socket detection branches. These cover misconfigured ARC/DinD failure modes.4. Recent commit well-covered
fix(api-proxy): use 'token' auth prefix for GHES enterprise Copilot API (#4755)—src/api-proxy-config.tsnow has 100% statement / 98.2% branch coverage. ✅📈 Recommendations
Medium —
network-options.ts: Add a test settingDOCKER_HOST=(somehost/redacted) before callingvalidateNetworkOptions` to cover the external host warning branches (lines 47–74). Only file below 80% statement coverage.Medium —
log-parser.ts: Add test cases for IPv6 bracketed log entries (e.g.[2001:db8::1]:443) to ensureawf logs stats/awf logs summarywork correctly for IPv6 upstreams.Low —
dind-bootstrap.ts: TestassertAbsolutePatherror throws and non-standard Docker socket detection. Improves confidence in ARC/DinD setups.Low —
etc-mounts.ts: Unit-test conditional/etcbind-mount logic for absent files or differing feature flags.Generated by test-coverage-reporter workflow · Trigger:
push· Run: 27356509720Beta Was this translation helpful? Give feedback.
All reactions