|
1 | 1 | 'use strict'; |
2 | | -const { mustNotCall, expectsError } = require('../common'); |
| 2 | +const common = require('../common'); |
| 3 | +const assert = require('assert'); |
3 | 4 | const { Readable } = require('stream'); |
| 5 | +const { finished } = require('stream/promises'); |
4 | 6 |
|
5 | | -async function* generate() { |
6 | | - yield null; |
7 | | -} |
8 | | - |
9 | | -const stream = Readable.from(generate()); |
10 | | - |
11 | | -stream.on('error', expectsError({ |
| 7 | +const expectedError = { |
12 | 8 | code: 'ERR_STREAM_NULL_VALUES', |
13 | 9 | name: 'TypeError', |
14 | 10 | message: 'May not write null values to stream' |
15 | | -})); |
| 11 | +}; |
| 12 | + |
| 13 | +async function rejectsNull(iterable, expectedChunks = []) { |
| 14 | + const stream = Readable.from(iterable); |
| 15 | + const chunks = []; |
| 16 | + const completion = finished(stream); |
| 17 | + |
| 18 | + stream.on('data', (chunk) => chunks.push(chunk)); |
| 19 | + |
| 20 | + await assert.rejects(completion, expectedError); |
| 21 | + assert.deepStrictEqual(chunks, expectedChunks); |
| 22 | + assert.strictEqual(stream.destroyed, true); |
| 23 | +} |
| 24 | + |
| 25 | +async function asyncIteratorYieldsNull() { |
| 26 | + const cleanup = common.mustCall(); |
| 27 | + |
| 28 | + async function* generate() { |
| 29 | + try { |
| 30 | + yield null; |
| 31 | + } finally { |
| 32 | + cleanup(); |
| 33 | + } |
| 34 | + } |
| 35 | + |
| 36 | + await rejectsNull(generate()); |
| 37 | +} |
16 | 38 |
|
17 | | -stream.on('data', mustNotCall()); |
| 39 | +async function syncIteratorYieldsNull() { |
| 40 | + const cleanup = common.mustCall(); |
| 41 | + |
| 42 | + function* generate() { |
| 43 | + try { |
| 44 | + yield null; |
| 45 | + } finally { |
| 46 | + cleanup(); |
| 47 | + } |
| 48 | + } |
| 49 | + |
| 50 | + await rejectsNull(generate()); |
| 51 | +} |
| 52 | + |
| 53 | +async function firstSyncValueResolvesToNull() { |
| 54 | + const cleanup = common.mustCall(); |
| 55 | + |
| 56 | + function* generate() { |
| 57 | + try { |
| 58 | + yield Promise.resolve(null); |
| 59 | + } finally { |
| 60 | + cleanup(); |
| 61 | + } |
| 62 | + } |
| 63 | + |
| 64 | + await rejectsNull(generate()); |
| 65 | +} |
| 66 | + |
| 67 | +async function laterSyncValueResolvesToNull() { |
| 68 | + const cleanup = common.mustCall(); |
| 69 | + |
| 70 | + function* generate() { |
| 71 | + try { |
| 72 | + yield Promise.resolve('first'); |
| 73 | + yield Promise.resolve(null); |
| 74 | + } finally { |
| 75 | + cleanup(); |
| 76 | + } |
| 77 | + } |
| 78 | + |
| 79 | + await rejectsNull(generate(), ['first']); |
| 80 | +} |
18 | 81 |
|
19 | | -stream.on('end', mustNotCall()); |
| 82 | +Promise.all([ |
| 83 | + asyncIteratorYieldsNull(), |
| 84 | + syncIteratorYieldsNull(), |
| 85 | + firstSyncValueResolvesToNull(), |
| 86 | + laterSyncValueResolvesToNull(), |
| 87 | +]).then(common.mustCall()); |
0 commit comments