tools,test: enable no-prototype-builtins

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

.eslintrc.js

@@ -297,7 +297,6 @@ module.exports = {
     // ESLint recommended rules that we disable
     'no-empty': 'off',
     'no-inner-declarations': 'off',
-    'no-prototype-builtins': 'off',
 
     // JSDoc recommended rules that we disable
     'jsdoc/require-jsdoc': 'off',

test/js-native-api/test_object/test_null.js

@@ -14,6 +14,7 @@ const expectedForProperty = {
 assert.deepStrictEqual(testNull.setProperty(), expectedForProperty);
 assert.deepStrictEqual(testNull.getProperty(), expectedForProperty);
 assert.deepStrictEqual(testNull.hasProperty(), expectedForProperty);
+// eslint-disable-next-line no-prototype-builtins
 assert.deepStrictEqual(testNull.hasOwnProperty(), expectedForProperty);
 // It's OK not to want the result of a deletion.
 assert.deepStrictEqual(testNull.deleteProperty(),