Re: [whatwg/streams] test non-function strategy.size (#594)

ricea commented on this pull request.



> @@ -0,0 +1,10 @@
+'use strict';
+
+if (self.importScripts) {
+  self.importScripts('/resources/testharness.js');
+}
+
+test(() => assert_throws(new TypeError(), () => new WritableStream({}, { size: 'a string' })),

While it's impressive that you could fit this test on one line, I found it difficult to understand.

I'm sorry to be boring, but if you could split the test body into two statements I would appreciate it.

-- 
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/594#pullrequestreview-6994837

Received on Thursday, 3 November 2016 12:32:26 UTC