Skip to content

Add regression test for concurrent dub output reads - #335

Merged
atilaneves merged 1 commit into
masterfrom
test/pr334-pipe-deadlock
Sep 22, 2026
Merged

atilaneves merged 1 commit into
masterfrom
test/pr334-pipe-deadlock

Conversation

@atilaneves

Copy link
Copy Markdown
Owner

Summary

Protects the deadlock fix in #334 with an integration test that writes 1 MiB to stderr before completing stdout. A ten-second timeout makes sequential pipe reads fail instead of hanging CI. Runs in both the POSIX and Windows CI scripts.

fake dub describe → fill stderr → write stdout JSON
getDubInfo → drain both streams → parse JSON before timeout

Evidence

Merge Danger

Door: two-way; test-only changes can be reverted.

Blast Radius: CI.

@atilaneves
atilaneves merged commit ae1cbb7 into master Sep 22, 2026
22 checks passed
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.

1 participant