diff --git a/lib/internal/errors.js b/lib/internal/errors.js index 7c87dd57d26..a8c4bfb3c33 100644 --- a/lib/internal/errors.js +++ b/lib/internal/errors.js @@ -936,7 +936,8 @@ function sysError(code, syscall, path, dest, messages.set('ERR_SYSTEM_ERROR', sysError); function invalidArgType(name, expected, actual) { - internalAssert(name, 'name is required'); + internalAssert(typeof name === 'string'); + internalAssert(arguments.length === 3); // determiner: 'must be' or 'must not be' let determiner; @@ -948,21 +949,16 @@ function invalidArgType(name, expected, actual) { } let msg; - if (Array.isArray(name)) { - var names = name.map((val) => `"${val}"`).join(', '); - msg = `The ${names} arguments ${determiner} ${oneOf(expected, 'type')}`; - } else if (name.endsWith(' argument')) { - // for the case like 'first argument' + if (name.endsWith(' argument')) { + // For cases like 'first argument' msg = `The ${name} ${determiner} ${oneOf(expected, 'type')}`; } else { const type = name.includes('.') ? 'property' : 'argument'; msg = `The "${name}" ${type} ${determiner} ${oneOf(expected, 'type')}`; } - // If actual value received, output it // TODO(BridgeAR): Improve the output by showing `null` and similar. - if (arguments.length === 3) - msg += `. Received type ${typeof actual}`; + msg += `. Received type ${typeof actual}`; return msg; }