- From: Adam Rice <notifications@github.com>
- Date: Tue, 03 Oct 2017 02:40:29 -0700
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 3 October 2017 09:40:51 UTC
ricea commented on this pull request. > @@ -308,4 +308,19 @@ promise_test(t => { }); }, 'erroring during write with backpressure should result in the write failing'); +promise_test(t => { + const ts = new TransformStream({}, undefined, { highWaterMark: 0 }); + return delay(0).then(() => { + const writer = ts.writable.getWriter(); + // write should start synchronously + const writePromise = writer.write(0); + const abortPromise = writer.abort(); + return promise_rejects(t, new TypeError(), ts.readable.getReader().read(), 'read() should reject') I think I was trying to make it clearer why the read() was there. But it didn't work, because I couldn't remember why it was needed. I have added comments instead. -- 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/817#discussion_r142355486
Received on Tuesday, 3 October 2017 09:40:51 UTC