Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(290)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/streams/writable-streams/bad-strategies.js

Issue 2772293002: Update WritableStream to new standard version (Closed)
Patch Set: Changes from domenic@ review Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/streams/writable-streams/bad-strategies.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/streams/writable-streams/bad-strategies.js b/third_party/WebKit/LayoutTests/http/tests/streams/writable-streams/bad-strategies.js
deleted file mode 100644
index eb14cf22e12bf0d69a9e2ae7189137e4d50be3d1..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/http/tests/streams/writable-streams/bad-strategies.js
+++ /dev/null
@@ -1,93 +0,0 @@
-'use strict';
-
-if (self.importScripts) {
- self.importScripts('/resources/testharness.js');
-}
-
-const error1 = new Error('a unique string');
-error1.name = 'error1';
-
-test(() => {
- assert_throws(error1, () => {
- new WritableStream({}, {
- get size() {
- throw error1;
- },
- highWaterMark: 5
- });
- }, 'construction should re-throw the error');
-}, 'Writable stream: throwing strategy.size getter');
-
-test(() => {
- assert_throws(new TypeError(), () => {
- new WritableStream({}, { size: 'a string' });
- });
-}, 'reject any non-function value for strategy.size');
-
-test(() => {
- assert_throws(error1, () => {
- new WritableStream({}, {
- size() {
- return 1;
- },
- get highWaterMark() {
- throw error1;
- }
- });
- }, 'construction should re-throw the error');
-}, 'Writable stream: throwing strategy.highWaterMark getter');
-
-test(() => {
-
- for (const highWaterMark of [-1, -Infinity, NaN, 'foo', {}]) {
- assert_throws(new RangeError(), () => {
- new WritableStream({}, {
- size() {
- return 1;
- },
- highWaterMark
- });
- }, `construction should throw a RangeError for ${highWaterMark}`);
- }
-}, 'Writable stream: invalid strategy.highWaterMark');
-
-promise_test(t => {
- const ws = new WritableStream({}, {
- size() {
- throw error1;
- },
- highWaterMark: 5
- });
-
- const writer = ws.getWriter();
-
- const p1 = promise_rejects(t, error1, writer.write('a'), 'write should reject with the thrown error');
-
- const p2 = promise_rejects(t, error1, writer.closed, 'closed should reject with the thrown error');
-
- return Promise.all([p1, p2]);
-}, 'Writable stream: throwing strategy.size method');
-
-promise_test(() => {
- const sizes = [NaN, -Infinity, Infinity, -1];
- return Promise.all(sizes.map(size => {
- const ws = new WritableStream({}, {
- size() {
- return size;
- },
- highWaterMark: 5
- });
-
- const writer = ws.getWriter();
-
- return writer.write('a').then(() => assert_unreached('write must reject'), writeE => {
- assert_equals(writeE.name, 'RangeError', `write must reject with a RangeError for ${size}`);
-
- return writer.closed.then(() => assert_unreached('write must reject'), closedE => {
- assert_equals(closedE, writeE, `closed should reject with the same error as write`);
- });
- });
- }));
-}, 'Writable stream: invalid strategy.size return value');
-
-done();

Powered by Google App Engine
This is Rietveld 408576698