quic: Bring blob test closer to old spirit

Author: martenrichter

test/parallel/test-blob.js

@@ -240,36 +240,33 @@ assert.throws(() => new Blob({}), {
 })().then(common.mustCall());
 
 (async () => {
-  const b = new Blob(Array(10).fill('hello'));
+  const b = new Blob(Array(10).fill('hello'.repeat(3000)));
   const reader = b.stream().getReader();
   const chunks = [];
-  let byteLength = 0;
   while (true) {
     const res = await reader.read();
     if (res.done) break;
-    byteLength += res.value.byteLength;
-    assert.ok(res.value.byteLength >= 5);
+    assert.strictEqual(res.value.byteLength, 30000);
     chunks.push(res.value);
   }
-  assert.strictEqual(byteLength, 50);
+  assert.strictEqual(chunks.length, 5);
 })().then(common.mustCall());
 
 (async () => {
-  const b = new Blob(Array(10).fill('hello'));
+  const b = new Blob(Array(10).fill('hello'.repeat(3000)));
   const reader = b.stream().getReader();
   const chunks = [];
-  let byteLength = 0;
   while (true) {
     const res = await reader.read();
-    byteLength += res.value.byteLength;
-    if (byteLength >= 50) {
+    if (chunks.length === 5) {
       reader.cancel('boom');
       break;
     }
     if (res.done) break;
-    assert.ok(res.value.byteLength >= 5);
+    assert.strictEqual(res.value.byteLength, 30000);
     chunks.push(res.value);
   }
+  assert.strictEqual(chunks.length, 5);
   reader.closed.then(common.mustCall());
 })().then(common.mustCall());
 
@@ -332,18 +329,18 @@ assert.throws(() => new Blob({}), {
 })().then(common.mustCall());
 
 (async () => {
-  const b = new Blob(Array(10).fill('hello'));
+  const b = new Blob(Array(10).fill('hello'.repeat(3000)));
   const stream = b.stream();
   const reader = stream.getReader();
   assert.strictEqual(stream[kState].controller.desiredSize, 0);
   const { value, done } = await reader.read();
-  assert.ok(value.byteLength >= 5);
-  assert.ok(value.byteLength <= 50);
-  assert(!done || value.byteLength === 50);
+  assert.strictEqual(value.byteLength, 30000);
+  assert(!done);
   setTimeout(common.mustCall(() => {
     // The blob stream is now a byte stream hence after the first read,
     // it should pull in the next 'hello' which is 5 bytes hence -5.
     // but recently, we coalesce if possible adjacent memory
+    // the test code is adapted to the current limit for coalescing
     assert.strictEqual(stream[kState].controller.desiredSize, 0);
   }), 0);
 })().then(common.mustCall());
@@ -363,30 +360,28 @@ assert.throws(() => new Blob({}), {
 })().then(common.mustCall());
 
 (async () => {
-  const b = new Blob(Array(10).fill('hello'));
+  const b = new Blob(Array(10).fill('hello'.repeat(3000)));
   const stream = b.stream();
   const reader = stream.getReader({ mode: 'byob' });
   assert.strictEqual(stream[kState].controller.desiredSize, 0);
-  const { value, done } = await reader.read(new Uint8Array(100));
-  assert.ok(value.byteLength >= 5);
-  assert.ok(value.byteLength <= 50);
+  const { value, done } = await reader.read(new Uint8Array(40000));
+  assert.strictEqual(value.byteLength, 30000);
   assert(!done);
   setTimeout(common.mustCall(() => {
     assert.strictEqual(stream[kState].controller.desiredSize, 0);
   }), 0);
 })().then(common.mustCall());
 
 (async () => {
-  const b = new Blob(Array(10).fill('hello'));
+  const b = new Blob(Array(10).fill('hello'.repeat(3000)));
   const stream = b.stream();
   const reader = stream.getReader({ mode: 'byob' });
   assert.strictEqual(stream[kState].controller.desiredSize, 0);
   const { value, done } = await reader.read(new Uint8Array(2));
   assert.strictEqual(value.byteLength, 2);
   assert(!done);
   setTimeout(common.mustCall(() => {
-    assert.ok(stream[kState].controller.desiredSize <= -3,
-              'desiredSize must be smaller or equal than -3');
+    assert.strictEqual(stream[kState].controller.desiredSize, -29998);
   }), 0);
 })().then(common.mustCall());