Skip to content

Commit 46206b7

Browse files
tools: auto fix custom eslint rule for crypto-check.js
Removes extra spaces ( lint-error ) + rules-utils argument check suggestoin Refs : nodejs#16636
1 parent 1018b5b commit 46206b7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

test/parallel/test-eslint-crypto-check.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ new RuleTester().run('crypto-check', rule, {
3131
'require("crypto")'
3232
},
3333
{
34-
code: 'require("common")\n' +
34+
code: 'require("common")\n' +
3535
'if (common.foo) {}\n' +
3636
'require("crypto")',
3737
errors: [{ message }],

tools/eslint-rules/rules-utils.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@ module.exports.isRequired = function(node, modules) {
1919
var commonModuleRegExp = new RegExp(/^(\.\.\/)*common(\.js)?$/);
2020
module.exports.isCommonModule = function(node) {
2121
return node.callee.name === 'require' &&
22-
commonModuleRegExp.test(node.arguments[0].value);
22+
node.arguments.length !== 0 &&
23+
commonModuleRegExp.test(node.arguments[0].value);
2324
};
2425

2526
/**

0 commit comments

Comments
 (0)