Re: [whatwg/streams] Port constructor tests to web-platform-tests (#524)

tyoshino commented on this pull request.

lgtm

> +  });
+}, 'controller argument should be passed to start method');
+
+promise_test(() => {
+  const ws = new WritableStream({}, {
+    highWaterMark: 1000,
+    size() { return 1; }
+  });
+
+  const writer = ws.getWriter();
+
+  assert_equals(writer.desiredSize, 1000, 'desiredSize should be 1000');
+  return writer.ready.then(v => {
+    assert_equals(v, undefined, 'ready promise should fulfill with undefined');
+  });
+}, 'highWaterMark should be copied to desiredSize');

how about s/copied/reflected/?

-- 
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/524#pullrequestreview-3042826

Received on Thursday, 6 October 2016 04:02:52 UTC