Re: [whatwg/streams] Reject writer.ready promise on errors, instead of fulfilling (#526)

ricea commented on this pull request.



> +
+  const ws = recordingWritableStream({
+    close() {
+      throw error1;
+    }
+  });
+
+  const writer = ws.getWriter();
+
+  return promise_rejects(t, error1, writer.close(), 'close() promise must reject with the thrown error')
+  .then(() => promise_rejects(t, error1, writer.ready, 'ready promise must reject with the thrown error'))
+  .then(() => {
+    assert_array_equals(ws.events, ['close']);
+  });
+
+}, 'Underlying sink close: throwing method');

I found when I was porting tests that it was easy to see what a test was for when it was phrased in terms of requirements. I think this is important when looking for gaps in testing coverage.

-- 
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/526