tools: auto fix custom eslint rule for crypto-check.js

Removes extra spaces ( lint-error ) + rules-utils argument check suggestoin Refs : #16636
nodejs · Feb 17, 2018 · 46206b7 · 46206b7
1 parent 1018b5b
commit 46206b7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/test/parallel/test-eslint-crypto-check.js b/test/parallel/test-eslint-crypto-check.js
@@ -31,7 +31,7 @@ new RuleTester().run('crypto-check', rule, {
               'require("crypto")'
     },
     {
-      code: 'require("common")\n' + 
+      code: 'require("common")\n' +
             'if (common.foo) {}\n' +
             'require("crypto")',
       errors: [{ message }],

diff --git a/tools/eslint-rules/rules-utils.js b/tools/eslint-rules/rules-utils.js
@@ -19,7 +19,8 @@ module.exports.isRequired = function(node, modules) {
 var commonModuleRegExp = new RegExp(/^(\.\.\/)*common(\.js)?$/);
 module.exports.isCommonModule = function(node) {
   return node.callee.name === 'require' &&
-    commonModuleRegExp.test(node.arguments[0].value);
+         node.arguments.length !== 0 &&
+         commonModuleRegExp.test(node.arguments[0].value);
 };
 
 /**