Skip to content

Rename or distribute miner-cli-json-error-coverage.test.ts #8579

Description

@JSONbored

Part of the '-coverage' bolt-on consolidation epic (#8574 — non-closing reference).

test/unit/miner-cli-json-error-coverage.test.ts (223 lines) is coverage-named but structurally different from the other bolt-ons: it spans the JSON-error paths of MANY miner CLI entrypoints (claim-ledger-cli, event-ledger-cli, governor-ledger-cli, loop-cli, manage-poll, plan-store-cli, queue CLIs, …) rather than duplicating one sibling suite. miner-cli-error.test.ts covers only the shared cli-error module, so a straight merge there is wrong.

Task — pick ONE:

  • Option A (cheap): rename to a behavior-describing name (e.g. miner-cli-json-error-paths.test.ts) so it stops reading as a coverage-filler file, and retitle generic case names.
  • Option B (thorough): distribute each case into the per-CLI suite that owns that entrypoint (miner-attempt-cli.test.ts-style siblings), then delete the file.

Either way, verify zero coverage regression on the touched packages/loopover-miner/lib/*-cli.ts files with an unsharded npm run test:coverage before/after comparison.

Metadata

Metadata

Assignees

Labels

maintainer-onlyOwner-only work — yields no Gittensor points.

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions