Skip to content

test: cover human-render, init launch, and error-path gaps#23

Merged
alexkroman merged 3 commits into
mainfrom
claude/improve-test-coverage-aVvZB
Jun 6, 2026
Merged

test: cover human-render, init launch, and error-path gaps#23
alexkroman merged 3 commits into
mainfrom
claude/improve-test-coverage-aVvZB

Conversation

@alexkroman

Copy link
Copy Markdown
Collaborator

Raise branch coverage from 97% to 99% by filling the largest gaps:

  • init: interactive picker (TTY/Ctrl-C/missing questionary), install
    failure reporting, and the launch path (port, browser open, exit code)
  • stdio: silence_stdout, piped/binary stdin fallbacks
  • keys/login/transcribe: human-readable (non-JSON) render paths
  • llm --follow: missing-prompt and non-piped-stdin guards
  • auth flow/loopback: no-projects, wrong token type, 404 path, _capture
  • scaffold: registered-but-missing template
  • audio-transcription template: 502 on missing id / gateway error
  • login: env-override warning emitted by the root callback

claude and others added 3 commits June 6, 2026 03:14
Raise branch coverage from 97% to 99% by filling the largest gaps:

- init: interactive picker (TTY/Ctrl-C/missing questionary), install
  failure reporting, and the launch path (port, browser open, exit code)
- stdio: silence_stdout, piped/binary stdin fallbacks
- keys/login/transcribe: human-readable (non-JSON) render paths
- llm --follow: missing-prompt and non-piped-stdin guards
- auth flow/loopback: no-projects, wrong token type, 404 path, _capture
- scaffold: registered-but-missing template
- audio-transcription template: 502 on missing id / gateway error
- login: env-override warning emitted by the root callback
io.StringIO declares buffer as a property, so assigning a BytesIO class
attribute tripped reportIncompatibleMethodOverride under pyright (tests
standard). Use a plain stub object carrying a .buffer attribute instead —
read_binary_stdin only does getattr(sys.stdin, "buffer", None).
@alexkroman alexkroman merged commit 79506fb into main Jun 6, 2026
7 checks passed
@alexkroman alexkroman deleted the claude/improve-test-coverage-aVvZB branch June 6, 2026 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants