tools,test: enable 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>
nodejs · Mar 14, 2022 · 6612056 · 6612056
1 parent 673c1fd
commit 6612056
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(),