Re: [streams] Byte stream update, including reference implementation (#361)

> +  }).catch(e => {
> +    t.fail(e);
> +    t.end();
> +  });
> +});
> +
> +test('ReadableByteStream: read(), then respond()', t => {
> +  let controller;
> +
> +  const rbs = new ReadableByteStream({
> +    start(c) {
> +      controller = c;
> +    },
> +    pull() {
> +      try {
> +        controller.respond(1);

This is throwing because it's respond()ing but not to a pullInto(), right? I think the test title should say that, and also you should respond with a Uint8Array or something, because otherwise I'd expect a TypeError anyway when respond()ing with a number.

---
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/361/files#r32268425