After the recent changes in NIO where we introduced the executeThenClose method on the NIOAsyncChannel our tests here became flaky since we were waiting for something to happen but potentially closed the client channels before we wrote out the data.
Modification
This PR makes sure we are awaiting for the event in the executeThenClose scope; hence, making sure we are not closing the client channels too early.
Motivation
After the recent changes in NIO where we introduced the
executeThenClose
method on theNIOAsyncChannel
our tests here became flaky since we were waiting for something to happen but potentially closed the client channels before we wrote out the data.Modification
This PR makes sure we are awaiting for the event in the
executeThenClose
scope; hence, making sure we are not closing the client channels too early.Result
Less flakey tests.