- From: Adam Rice <notifications@github.com>
- Date: Thu, 10 Nov 2016 01:27:38 -0800
- To: whatwg/streams <streams@noreply.github.com>
Received on Thursday, 10 November 2016 09:28:13 UTC
ricea commented on this pull request. > @@ -286,7 +286,7 @@ promise_test(t => { const ws = recordingWritableStream({ write() { if (ws.events.length > 2) { - return delay(10).then(() => { + return flushAsyncEvents().then(() => { Okay, where we are not waiting for something to not happen, but instead just testing async behaviour, I have changed the flushAsyncEvents() to delay(0). delay(0) is currently the usual way to say "do this asynchronously" in these tests. The net result of this is that a lot of the changes just end up changing delay(n) to delay(0), but that still satisfies the goal of reducing use of delay(0). -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/whatwg/streams/pull/605
Received on Thursday, 10 November 2016 09:28:13 UTC