fix(rules): don't crash on require() (#115)

fixes #114
jest-community · May 27, 2018 · 523e2f4 · 523e2f4
1 parent 4586bdc
commit 523e2f4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/rules/__tests__/no-jest-import.test.js b/rules/__tests__/no-jest-import.test.js
@@ -12,6 +12,7 @@ ruleTester.run('no-jest-import', rule, {
       parserOptions: { sourceType: 'module' },
     },
     'require("somethingElse")',
+    'require()',
     'entirelyDifferent(fn)',
   ],
   invalid: [

diff --git a/rules/no-jest-import.js b/rules/no-jest-import.js
@@ -22,7 +22,11 @@ module.exports = {
       },
       CallExpression(node) {
         const calleeName = getNodeName(node.callee);
-        if (calleeName === 'require' && node.arguments[0].value === 'jest') {
+        if (
+          calleeName === 'require' &&
+          node.arguments[0] &&
+          node.arguments[0].value === 'jest'
+        ) {
           context.report({
             loc: {
               end: {