tools: simplify lowercase-name-for-primitive rule

PR-URL: https://github.com/nodejs/node/pull/17653
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
This commit is contained in:
cjihrig 2017-12-13 11:14:02 -05:00
parent eecea2e0e5
commit 203ce5f9e0
No known key found for this signature in database
GPG Key ID: 7434390BDBE9B9C5

View File

@ -9,41 +9,38 @@
// Rule Definition // Rule Definition
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------
const astSelector = 'NewExpression[callee.property.name="TypeError"]' +
'[arguments.0.value="ERR_INVALID_ARG_TYPE"]';
const primitives = [ const primitives = [
'number', 'string', 'boolean', 'null', 'undefined' 'number', 'string', 'boolean', 'null', 'undefined'
]; ];
module.exports = function(context) { module.exports = function(context) {
return { function checkNamesArgument(node) {
NewExpression(node) { const names = node.arguments[2];
if (
node.callee.property &&
node.callee.property.name === 'TypeError' &&
node.arguments[0].value === 'ERR_INVALID_ARG_TYPE'
) {
checkNamesArgument(node.arguments[2]);
}
function checkNamesArgument(names) { switch (names.type) {
switch (names.type) { case 'Literal':
case 'Literal': checkName(node, names.value);
checkName(names.value); break;
break; case 'ArrayExpression':
case 'ArrayExpression': names.elements.forEach((name) => {
names.elements.forEach((name) => { checkName(node, name.value);
checkName(name.value); });
}); break;
break;
}
}
function checkName(name) {
const lowercaseName = name.toLowerCase();
if (primitives.includes(lowercaseName) && !primitives.includes(name)) {
const msg = `primitive should use lowercase: ${name}`;
context.report(node, msg);
}
}
} }
}
function checkName(node, name) {
const lowercaseName = name.toLowerCase();
if (primitives.includes(lowercaseName) && !primitives.includes(name)) {
const msg = `primitive should use lowercase: ${name}`;
context.report(node, msg);
}
}
return {
[astSelector]: (node) => checkNamesArgument(node)
}; };
}; };