test(cli): coverage wins (preflight/progress/errors) + smoke-test hardening

[LukasWodka]

Summary

Lifts unit coverage on the cheap, cluster-free 0%-functions and hardens the CI smoke test. Independent of the modality stack — branches off develop.

Coverage

• progress_test.go: NewProgress non-TTY → NoOp; isTTY false cases.
• coverage_test.go (cli): printPushPreflight renders the key facts; exitError.Error/Code; runClusterInfo → exit 3 on a bad kubeconfig.
• watcherr_test.go (submit): WatchError.Unwrap + IsWatchError (exit-9 vs exit-8 mapping).

Per-package after: cli 61% · cluster 84% · push 84% · schema 81% · submit 66%. Remaining gaps are the real-I/O seams (SPDYExecutor.Exec, PortForwardJobsManager, NewClientset) — covered by the integration-harness PR.

Smoke test

CI now runs ingest validate on committed fixtures (valid → 0, missing-images → 2) + dataset push --help against the built binary — catches schema/validation/wiring regressions without a cluster.

Test plan

go build/vet/test ./... green; fixtures verified locally (good→0, bad→2).

🤖 Generated with Claude Code

[LukasWodka]

👋 Heads-up — Code review queue is at 19 / 8

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

• averaging-service#98 — ci: add fr-gate caller workflow · author: @LukasWodka · no reviewer assigned
• client#171 — Installer: verify readiness & credentials before reporting success; RHEL-family support · author: @LukasWodka · no reviewer assigned
• client-runtime#45 — Staging: Enhance jobs-manager with HTTP proxy, ingestion endpoint, and fixes · author: @saadqbal · no reviewer assigned
• client-runtime#61 — docs: record MySQL credential threat-model decision · author: @saadqbal · no reviewer assigned
• data-ingestors#132 — ci: add fr-gate caller workflow · author: @LukasWodka · no reviewer assigned
• data-ingestors#133 — docs: fix declarative-ingest path/column drift (issue #131 A-series) · author: @divyasinghds · no reviewer assigned
• data-ingestors#138 — test(e2e): end-to-end ingestion equivalence suite (all modalities, real MySQL) · author: @LukasWodka · no reviewer assigned
• data-ingestors#139 — fix(ingest): MLM tokenizer + relax PascalVOC difficult + reserved-id guard (#137, #135) · author: @LukasWodka · no reviewer assigned
• design-system#19 — fix: un-track coverage/ and node_modules/ from git · author: @LukasWodka · no reviewer assigned
• design-system#22 — ci: add Vitest test workflow · author: @LukasWodka · reviewer: @aptracebloc

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

[aptracebloc]

Rebased onto develop — schema-drift now green; all checks pass (Build x5, Lint, Test). Coverage additions are substantive (preflight content, exitError Code/Error, runClusterInfo exit-3, progress NoOp/isTTY, WatchError classification), and the CI smoke step now exercises ingest validate on a valid + invalid spec against the real binary. LGTM.