tools,test: enable no-prototype-builtins

PR-URL: #41801 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Mestery <[email protected]>
nodejs · Feb 8, 2022 · 34dfc60 · 34dfc60
1 parent 87902a7
commit 34dfc60
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
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(),