Skip to content

test: fix flaky timeout in test-pipe-file-to-http

Rodrigo Muino Tomonari requested to merge github/fork/jakecastelli/fix-52963 into main

Attempt to fix: https://github.com/nodejs/node/issues/52963

There are a few places could cause the test to timeout, first of all rely on process.exit was probably not a great idea.

Timeout was never cleared, 10mb might be a bit heavy on lower power machine.

Tried another 4000 runs, no failures, should be stable now?

Will actively monitor the issue once this PR lands to see if there is still any failure in CI.

Previous attempt PR was closed and cannot re-opened.

Merge request reports

Loading