Skip to content

tee: add short-read regression test#11784

Merged
sylvestre merged 1 commit into
uutils:mainfrom
kevinburke:tee-fix-short-read
Apr 13, 2026
Merged

tee: add short-read regression test#11784
sylvestre merged 1 commit into
uutils:mainfrom
kevinburke:tee-fix-short-read

Conversation

@kevinburke
Copy link
Copy Markdown
Contributor

Upstream commit 9f50c8b ("tee: fix input with sleep") already contains the functional fix for short reads from a paused writer. Keep the regression coverage so we do not reintroduce the bug.

The test writes one small chunk, waits long enough for a buggy tee to exit, then writes a second chunk and asserts both stdout and the output file receive both writes.

Upstream commit 9f50c8b ("tee: fix input with sleep") already
contains the functional fix for short reads from a paused writer.
Keep the regression coverage so we do not reintroduce the bug.

The test writes one small chunk, waits long enough for a buggy tee to
exit, then writes a second chunk and asserts both stdout and the output
file receive both writes.
@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/tail-n0f (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/basenc/bounded-memory is now being skipped but was previously passing.
Congrats! The gnu test tests/dd/no-allocate is now passing!
Congrats! The gnu test tests/printf/printf-surprise is now passing!

@sylvestre sylvestre merged commit b17025b into uutils:main Apr 13, 2026
167 checks passed
@sylvestre
Copy link
Copy Markdown
Contributor

thanks

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