- From: Adam Rice <notifications@github.com>
- Date: Wed, 09 Nov 2016 01:09:53 -0800
- To: whatwg/streams <streams@noreply.github.com>
- Message-ID: <whatwg/streams/pull/605@github.com>
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