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

Add a util function flushAsyncEvents() which spins the event loop to
ensure that the implementation doesn't do something it shouldn't
asynchronously.

Use this function to replace trivial uses of delay(100) and similar  to
improve test determinism.

This change relates to #548.

Tests which use multiple delays to express ordering requirements have
not been changed. Those tests require individual analysis.
You can view, comment on, or merge this pull request online at:

  https://github.com/whatwg/streams/pull/605

-- Commit Summary --

  * Remove trivial non-zero delays in wpts
  * Fix two comments that talk about 2 event loops.

-- File Changes --

    M reference-implementation/to-upstream-wpts/.eslintrc.json (1)
    M reference-implementation/to-upstream-wpts/piping/close-propagation-forward.js (4)
    M reference-implementation/to-upstream-wpts/piping/error-propagation-backward.js (6)
    M reference-implementation/to-upstream-wpts/piping/error-propagation-forward.js (4)
    M reference-implementation/to-upstream-wpts/piping/flow-control.js (6)
    M reference-implementation/to-upstream-wpts/resources/test-utils.js (8)
    M reference-implementation/to-upstream-wpts/transform-stream/flush.js (2)
    M reference-implementation/to-upstream-wpts/writable-streams/aborting.js (2)
    M reference-implementation/to-upstream-wpts/writable-streams/bad-underlying-sinks.js (2)
    M reference-implementation/to-upstream-wpts/writable-streams/close.js (4)
    M reference-implementation/to-upstream-wpts/writable-streams/start.js (6)
    M reference-implementation/to-upstream-wpts/writable-streams/write.js (4)

-- Patch Links --

https://github.com/whatwg/streams/pull/605.patch
https://github.com/whatwg/streams/pull/605.diff

-- 
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 Wednesday, 9 November 2016 09:10:28 UTC