stream: improve buffer list inspection

This makes sure the indentation level is correct, no matter if the inspected buffer list is inspected on a deeper level and custom inspect options are now passed through to the actual inspection. PR-URL: #23109 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: BridgeAR

lib/internal/streams/buffer_list.js

@@ -158,8 +158,14 @@ module.exports = class BufferList{
 returnret;
 }
 
-[inspect.custom](){
-constobj=inspect({length: this.length});
-return`${this.constructor.name}${obj}`;
+// Make sure the linked list only shows the minimal necessary information.
+[inspect.custom](_,options){
+returninspect(this,{
+ ...options,
+// Only inspect one level.
+depth: 0,
+// It should not recurse.
+customInspect: false
+});
 }
 };

test/parallel/test-stream2-readable-from-list.js

@@ -25,6 +25,7 @@ require('../common');
 constassert=require('assert');
 constfromList=require('_stream_readable')._fromList;
 constBufferList=require('internal/streams/buffer_list');
+constutil=require('util');
 
 functionbufferListFromArray(arr){
 constbl=newBufferList();
@@ -41,6 +42,16 @@ function bufferListFromArray(arr){
 Buffer.from('kuel')];
 list=bufferListFromArray(list);
 
+assert.strictEqual(
+util.inspect([list],{compact: false}),
+`[
+ BufferList{
+ head: [Object],
+ tail: [Object],
+ length: 4
+ }
+]`);
+
 // read more than the first element.
 letret=fromList(6,{buffer: list,length: 16});
 assert.strictEqual(ret.toString(),'foogba');