Skip to content

Add direct unit tests for app/output.py #871

@VaibhavUpreti

Description

@VaibhavUpreti

Why This Matters

app/output.py controls progress spinners, output-format selection, and humanized messages, but it has no direct test module.

Read First

  • app/output.py

Scope

  1. Add a dedicated test module for pure helpers and tracker behavior.
  2. Prefer testing helper outputs over brittle terminal snapshots.
  3. Keep the tests deterministic by patching time and TTY checks.

Tests / Coverage

  • tests/test_output.py
  • cover get_output_format()
  • cover _humanise_message()
  • cover _fmt_timing()
  • cover ProgressTracker.start() and complete() in text mode

Acceptance Criteria

  • the module has direct unit coverage
  • tests do not require a real TTY
  • progress text remains stable across refactors

Metadata

Metadata

Assignees

Labels

cliCommand-line interface worktestsTest coverage and test organizationuxDeveloper or user experience improvements

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions