[Coverage Report] Test Coverage Report — 2026-06-01 #4165
Replies: 5 comments
-
|
🔮 The ancient spirits stir: the smoke test agent passed through and left this 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 stirred, and the smoke test agent crossed the mists to mark this discussion with a passing omen. 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 was here. May the ledger remember this passing omen. 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 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-08T23:18:55.924Z.
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
📊 Overall Coverage
Overall coverage is excellent. No files fall below 50% statement coverage.
🔴 Critical Gaps (< 50% statement coverage)
None — all files are above 50%.
🟡 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.tsAll four primary security-critical files (
host-iptables.ts,squid-config.ts,docker-manager.ts,domain-patterns.ts) have excellent coverage.cli.tshas a 50% branch coverage gap.📋 Files with Notable Branch/Function Gaps
src/commands/validators/network-options.tssrc/test-helpers/docker-test-fixtures.test-utils.tssrc/cli.tssrc/cli-options.tssrc/logs/log-parser.tssrc/services/agent-volumes/etc-mounts.tssrc/services/agent-environment/environment-builder.ts🔍 Notable Findings
src/commands/validators/network-options.ts(66.7% stmt, 50% branch) — The lowest-coverage file. As a validator for network options, this is somewhat security-relevant. Missing branches likely cover error/validation paths for malformed inputs. Tests should cover invalid domain formats, conflicting options, and edge cases in network validation.src/cli-options.ts(25% function coverage) — Only 1 of 4 exported functions are called in tests. The 3 uncovered functions may handle CLI option parsing edge cases (help text, version flags, unusual option combinations). Low risk since this is glue code, but worth covering.src/cli.ts(50% branch coverage) — The main entry-point has an uncovered branch. Likely therequire.main === moduleguard or an early-exit error path. Should be verified since errors here propagate to the entire program.src/logs/log-parser.ts(67.1% branch coverage) — Log parsing has significant branch gaps. Edge cases like malformed log lines, unexpected timestamps, or unusual Squid output formats may not be tested. These are lower security priority but affect observability.📈 Recommendations
High: Add tests for
src/commands/validators/network-options.ts— cover invalid domain inputs, empty domain list, conflicting--allow-domains/--deny-domainscombinations, and malformed CIDR ranges. This validator gates network isolation configuration.Medium: Cover the missing functions in
src/cli-options.ts— likely option-parsing helpers that may silently accept invalid configurations without tests.Medium: Improve branch coverage in
src/logs/log-parser.ts— test malformed Squid log lines, lines with missing fields, and edge cases in timestamp parsing.Low: Close the branch gap in
src/cli.ts— the uncovered branch is likely a trivialrequire.mainguard but worth confirming.Generated by test-coverage-reporter workflow. Trigger:
push· Run: 26787802336 · 95 test files covering 130 source filesBeta Was this translation helpful? Give feedback.
All reactions