tools: remove custom buffer-constructor lint rule

This rule is no longer needed. ESLint's built in
no-buffer-constructor rule is enabled instead.

PR-URL: https://github.com/nodejs/node/pull/25261
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

lib/.eslintrc.yaml

@@ -27,7 +27,6 @@ rules:
       message: "Use an error exported by the internal/errors module."
   # Custom rules in tools/eslint-rules
   node-core/require-globals: error
-  node-core/buffer-constructor: error
   node-core/no-let-in-for-declaration: error
   node-core/lowercase-name-for-primitive: error
   node-core/non-ascii-character: error

test/parallel/test-eslint-buffer-constructor.js

@@ -1,27 +0,0 @@
-'use strict';
-
-const common = require('../common');
-common.skipIfEslintMissing();
-
-const RuleTester = require('../../tools/node_modules/eslint').RuleTester;
-const rule = require('../../tools/eslint-rules/buffer-constructor');
-
-const message = 'Use of the Buffer() constructor has been deprecated. ' +
-                'Please use either Buffer.alloc(), Buffer.allocUnsafe(), ' +
-                'or Buffer.from()';
-
-new RuleTester().run('buffer-constructor', rule, {
-  valid: [
-    'Buffer.from(foo)'
-  ],
-  invalid: [
-    {
-      code: 'Buffer(foo)',
-      errors: [{ message }]
-    },
-    {
-      code: 'new Buffer(foo)',
-      errors: [{ message }]
-    }
-  ]
-});

test/root.status

@@ -31,7 +31,6 @@ parallel/test-domain-with-abort-on-uncaught-exception: SLOW
 parallel/test-env-var-no-warnings: SLOW
 parallel/test-error-reporting: SLOW
 parallel/test-eslint-alphabetize-errors: SLOW
-parallel/test-eslint-buffer-constructor: SLOW
 parallel/test-eslint-crypto-check: SLOW
 parallel/test-eslint-documented-errors: SLOW
 parallel/test-eslint-duplicate-requires: SLOW

tools/eslint-rules/buffer-constructor.js

@@ -1,20 +0,0 @@
-/**
- * @fileoverview Require use of new Buffer constructor methods in lib
- * @author James M Snell
- */
-'use strict';
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-const msg = 'Use of the Buffer() constructor has been deprecated. ' +
-            'Please use either Buffer.alloc(), Buffer.allocUnsafe(), ' +
-            'or Buffer.from()';
-const astSelector = 'NewExpression[callee.name="Buffer"],' +
-                    'CallExpression[callee.name="Buffer"]';
-
-module.exports = function(context) {
-  return {
-    [astSelector]: (node) => context.report(node, msg)
-  };
-};