tools,test: enable no-prototype-builtins

Trott · danielleadams · commit 66120564b2c4 · 2022-03-14T09:32:14.000-04:00
PR-URL: #41801 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com>
diff --git a/.eslintrc.js b/.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',
diff --git a/test/js-native-api/test_object/test_null.js b/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(),
