test: avoid using Object.prototype methods directly on objects

This prepares us to enable the no-prototype-builtins ESLint rule.

Refs: https://eslint.org/docs/rules/no-prototype-builtins

PR-URL: #41801
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>

Author: Trott

test/parallel/test-console-group.js

@@ -151,7 +151,7 @@ function teardown() {
 // Check that the kGroupIndent symbol property is not enumerable
 {
   const keys = Reflect.ownKeys(console)
-                      .filter((val) => console.propertyIsEnumerable(val))
+                      .filter((val) => Object.prototype.propertyIsEnumerable.call(console, val))
                       .map((val) => val.toString());
   assert(!keys.includes('Symbol(groupIndent)'),
          'groupIndent should not be enumerable');

test/parallel/test-stream-base-prototype-accessors-enumerability.js

@@ -14,8 +14,8 @@ const { internalBinding } = require('internal/test/binding');
 const TTY = internalBinding('tty_wrap').TTY;
 
 {
-  assert.strictEqual(TTY.prototype.propertyIsEnumerable('bytesRead'), false);
-  assert.strictEqual(TTY.prototype.propertyIsEnumerable('fd'), false);
-  assert.strictEqual(
-    TTY.prototype.propertyIsEnumerable('_externalStream'), false);
+  const ttyIsEnumerable = Object.prototype.propertyIsEnumerable.bind(TTY);
+  assert.strictEqual(ttyIsEnumerable('bytesRead'), false);
+  assert.strictEqual(ttyIsEnumerable('fd'), false);
+  assert.strictEqual(ttyIsEnumerable('_externalStream'), false);
 }