search

con / duct

A helper to run a command, capture stdout/stderr and details about running
MIT License
1 stars 1 forks source link

test for bug: TeeStream cannot handle high output (ie cat thousand_lines) #20

Closed asmacdo closed 1 month ago

asmacdo commented 1 month ago

First attempt at e2e testing of the TeeStream class.

Currently not passing when using cat with more than 10^3 lines

Theres a couple problems. [minor] First, it isn't deterministic on how much is written at a time, ie 250 lines one call, 300 the next, so asserting has_calls isn't working.

Bigger problem: somewhere between cat 100lines and cat 1000lines, TeeStream fails to print the whole thing before closing.

asmacdo commented 1 month ago

Closing in favor of https://github.com/con/duct/pull/29