Re: [whatwg/streams] Remove trivial non-zero delays in wpts (#605)

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