async-h1 icon indicating copy to clipboard operation
async-h1 copied to clipboard

Published 20 hours ago • verified publisher icon http-rs

Implement tests to expose sequencing bugs

Open Diggsey opened this issue 4 years ago • 1 comments

Regarding the addition of a sleep to accept_one:

This is used by the test_accept_partial_read_sequential_requests test to make it easier to reproduce the problem. Hopefully you agree that such a delay could naturally occur in a complex program given the surrounding async operations, and that if introducing a delay here causes a problem, then the code already had a bug. If not, then ignore that test.

Identical tests have been added to #179 to show that it fixes these issues.

Diggsey avatar Feb 02 '21 01:02 Diggsey

@yoshuawuyts did you have a chance to review these test cases?

Diggsey avatar Mar 04 '21 17:03 Diggsey