stream: use validateString for consumer encoding

Replace the manual `options.encoding` type check in stream iterator
consumers with the shared validator. Add coverage for non-string values
in the async and sync text consumers.
Signed-off-by: sjungwon03 <sjungwon03@gmail.com>
PR-URL: #64754
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>

Author: sjungwon03

lib/internal/streams/iter/consumers.js

@@ -28,7 +28,6 @@ const {
 
 const{
 codes: {
-ERR_INVALID_ARG_TYPE,
 ERR_INVALID_ARG_VALUE,
 ERR_OUT_OF_RANGE,
 },
@@ -39,6 +38,7 @@ const {
  validateFunction,
  validateInteger,
  validateObject,
+ validateString,
 }=require('internal/validators');
 
 const{
@@ -196,10 +196,7 @@ function validateBaseConsumerOptions(options) {
 validateInteger(options.limit,'options.limit',0);
 }
 if(options.encoding!==undefined){
-if(typeofoptions.encoding!=='string'){
-thrownewERR_INVALID_ARG_TYPE('options.encoding','string',
-options.encoding);
-}
+validateString(options.encoding,'options.encoding');
 try{
 newTextDecoder(options.encoding);
 }catch{

test/parallel/test-stream-iter-consumers-text.js

@@ -143,6 +143,20 @@ function testTextSyncUnsupportedEncodingThrowsRangeError() {
 );
 }
 
+asyncfunctiontestTextNonStringEncodingThrowsTypeError(){
+awaitassert.rejects(
+()=>text(from('hello'),{encoding: 1}),
+{code: 'ERR_INVALID_ARG_TYPE'},
+);
+}
+
+functiontestTextSyncNonStringEncodingThrowsTypeError(){
+assert.throws(
+()=>textSync(fromSync('hello'),{encoding: 1}),
+{code: 'ERR_INVALID_ARG_TYPE'},
+);
+}
+
 Promise.all([
 testTextSyncBasic(),
 testTextAsync(),
@@ -159,4 +173,6 @@ Promise.all([
 testTextSyncBOMStripped(),
 testTextUnsupportedEncodingThrowsRangeError(),
 testTextSyncUnsupportedEncodingThrowsRangeError(),
+testTextNonStringEncodingThrowsTypeError(),
+testTextSyncNonStringEncodingThrowsTypeError(),
 ]).then(common.mustCall());