Skip to content

feat: report test progress under --debug - #124

Open
BKPepe wants to merge 1 commit into
librespeed:masterfrom
BKPepe:feat/debug-progress-output
Open

feat: report test progress under --debug#124
BKPepe wants to merge 1 commit into
librespeed:masterfrom
BKPepe:feat/debug-progress-output

Conversation

@BKPepe

@BKPepe BKPepe commented Aug 7, 2026

Copy link
Copy Markdown

--json, --csv and --simple suppress the spinner, so a run prints
nothing between launch and the final result. --debug did not help — every
existing debug call sits on an error path, so a run that succeeds stays
quiet.

This reports each phase with timings and byte counts:

Ping test starting: 10 pings, ICMP: true
Ping test finished in 9.117s: ping 14.00 ms, jitter 4.07 ms
Download test starting: 3 stream(s), 100 chunk(s), up to 3s
Download test finished in 3.606s: 169.48 Mbps, 55329144 byte(s) received

Debug goes to stderr, so stdout stays clean for piping. Rates follow the
run's own mode — Mbps normally, MB/s under --bytes — so a debug line
cannot contradict the JSON beside it.

Nothing changes without --debug. Tested against a live backend in
--json, --simple --bytes and default modes.

--json, --csv and --simple suppress the spinner, so a run prints nothing
between launch and the final result. --debug did not help: every existing
debug call sits on an error path, so a run that succeeds stays quiet.

Report each phase instead — server, IP info, ping, download, upload — with
the timings, byte counts and stream settings the spinner cannot carry.

Rates follow the run's own reporting mode, Mbps normally and MB/s under
--bytes, so a debug line cannot contradict the result printed beside it.

Nothing changes unless --debug is given.
Copilot AI lite review requested due to automatic review settings August 7, 2026 22:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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