errors: fix incorrect argument label for array elements

Author: kimtaejin3

lib/internal/errors.js

@@ -1404,7 +1404,8 @@ E('ERR_INVALID_ARG_TYPE',
       // For cases like 'first argument'
       msg += `${name} `;
     } else {
-      const type = StringPrototypeIncludes(name, '.') ? 'property' : 'argument';
+      const type = (StringPrototypeIncludes(name, '.') ||
+                    StringPrototypeIncludes(name, '[')) ? 'property' : 'argument';
       msg += `"${name}" ${type} `;
     }
     msg += 'must be ';
@@ -1468,7 +1469,8 @@ E('ERR_INVALID_ARG_VALUE', (name, value, reason = 'is invalid') => {
   if (inspected.length > 128) {
     inspected = `${StringPrototypeSlice(inspected, 0, 128)}...`;
   }
-  const type = StringPrototypeIncludes(name, '.') ? 'property' : 'argument';
+  const type = (StringPrototypeIncludes(name, '.') ||
+                StringPrototypeIncludes(name, '[')) ? 'property' : 'argument';
   return `The ${type} '${name}' ${reason}. Received ${inspected}`;
 }, TypeError, RangeError, HideStackFramesError);
 E('ERR_INVALID_ASYNC_ID', 'Invalid %s value: %s', RangeError);

test/parallel/test-buffer-concat.js

@@ -59,7 +59,7 @@ assert.strictEqual(flatLongLen.toString(), check);
     Buffer.concat(value);
   }, {
     code: 'ERR_INVALID_ARG_TYPE',
-    message: 'The "list[0]" argument must be an instance of Buffer ' +
+    message: 'The "list[0]" property must be an instance of Buffer ' +
              `or Uint8Array.${common.invalidArgTypeHelper(value[0])}`
   });
 });
@@ -68,7 +68,7 @@ assert.throws(() => {
   Buffer.concat([Buffer.from('hello'), 3]);
 }, {
   code: 'ERR_INVALID_ARG_TYPE',
-  message: 'The "list[1]" argument must be an instance of Buffer ' +
+  message: 'The "list[1]" property must be an instance of Buffer ' +
            'or Uint8Array. Received type number (3)'
 });
 

test/parallel/test-diff.js

@@ -31,7 +31,7 @@ describe('diff', () => {
     const expected = ['1', '2'];
 
     assert.throws(() => diff(actual, expected), {
-      message: 'The "actual[1]" argument must be of type string. Received an instance of Object'
+      message: 'The "actual[1]" property must be of type string. Received an instance of Object'
     });
   });
 

test/parallel/test-dns-setservers-type-check.js

@@ -60,7 +60,7 @@ const promiseResolver = new dns.promises.Resolver();
     const errObj = {
       code: 'ERR_INVALID_ARG_TYPE',
       name: 'TypeError',
-      message: 'The "servers[0]" argument must be of type string.' +
+      message: 'The "servers[0]" property must be of type string.' +
                common.invalidArgTypeHelper(val[0])
     };
     assert.throws(
@@ -110,7 +110,7 @@ const promiseResolver = new dns.promises.Resolver();
     const errObj = {
       code: 'ERR_INVALID_ARG_TYPE',
       name: 'TypeError',
-      message: 'The "servers[0]" argument must be of type string.' +
+      message: 'The "servers[0]" property must be of type string.' +
               common.invalidArgTypeHelper(val[0])
     };
     assert.throws(() => {

test/parallel/test-process-setgroups.js

@@ -42,7 +42,7 @@ assert.throws(
     {
       code: 'ERR_INVALID_ARG_TYPE',
       name: 'TypeError',
-      message: 'The "groups[0]" argument must be ' +
+      message: 'The "groups[0]" property must be ' +
                'one of type number or string.' +
                common.invalidArgTypeHelper(val)
     }

test/parallel/test-tls-set-default-ca-certificates-error.js

@@ -27,14 +27,14 @@ for (const invalid of [null, undefined, 42, {}, true]) {
   // Test input validation - should throw when passed an array with invalid elements
   assert.throws(() => tls.setDefaultCACertificates([invalid]), {
     code: 'ERR_INVALID_ARG_TYPE',
-    message: /The "certs\[0\]" argument must be of type string or an instance of ArrayBufferView/
+    message: /The "certs\[0\]" property must be of type string or an instance of ArrayBufferView/
   });
   // Verify that default certificates remain unchanged after error.
   assertEqualCerts(tls.getCACertificates('default'), defaultCerts);
 
   assert.throws(() => tls.setDefaultCACertificates([fixtureCert, invalid]), {
     code: 'ERR_INVALID_ARG_TYPE',
-    message: /The "certs\[1\]" argument must be of type string or an instance of ArrayBufferView/
+    message: /The "certs\[1\]" property must be of type string or an instance of ArrayBufferView/
   });
   // Verify that default certificates remain unchanged after error.
   assertEqualCerts(tls.getCACertificates('default'), defaultCerts);