From 2313424abc6f88b72630f79a483721dd0cd10c27 Mon Sep 17 00:00:00 2001 From: Luigi Pinca Date: Thu, 18 Jan 2018 18:51:09 +0100 Subject: [PATCH] test: use correct size in test-stream-buffer-list The `n` argument of `BufferList.prototype.concat()` is not the number of `Buffer` instances in the list, but their total length when concatenated. PR-URL: https://github.com/nodejs/node/pull/18239 Reviewed-By: Matteo Collina --- test/parallel/test-stream-buffer-list.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-stream-buffer-list.js b/test/parallel/test-stream-buffer-list.js index 87c9d2a2bb50ce..c151154afa3d01 100644 --- a/test/parallel/test-stream-buffer-list.js +++ b/test/parallel/test-stream-buffer-list.js @@ -15,16 +15,21 @@ assert.strictEqual(emptyList.join(','), ''); assert.deepStrictEqual(emptyList.concat(0), Buffer.alloc(0)); +const buf = Buffer.from('foo'); + // Test buffer list with one element. const list = new BufferList(); -list.push('foo'); +list.push(buf); + +const copy = list.concat(3); -assert.strictEqual(list.concat(1), 'foo'); +assert.notStrictEqual(copy, buf); +assert.deepStrictEqual(copy, buf); assert.strictEqual(list.join(','), 'foo'); const shifted = list.shift(); -assert.strictEqual(shifted, 'foo'); +assert.strictEqual(shifted, buf); assert.deepStrictEqual(list, new BufferList()); const tmp = util.inspect.defaultOptions.colors;