tools: custom eslint autofix for inspector-check.js

rules-utils arguments check + spacing in test file Refs: nodejs#16636
shobhitchittora · Feb 17, 2018 · 3b1ce16 · 3b1ce16
1 parent c4b5e8b
commit 3b1ce16
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/test/parallel/test-eslint-inspector-check.js b/test/parallel/test-eslint-inspector-check.js
@@ -13,8 +13,8 @@ new RuleTester().run('inspector-check', rule, {
   valid: [
     'foo;',
     'require("common")\n' +
-    'common.skipIfInspectorDisabled();\n' +
-    'require("inspector")'
+      'common.skipIfInspectorDisabled();\n' +
+      'require("inspector")'
   ],
   invalid: [
     {

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