From 2fba46f5b8ed556321ba88e565b53a4af3e13129 Mon Sep 17 00:00:00 2001 From: Jacek Kopecky Date: Fri, 25 May 2018 23:58:08 +0100 Subject: [PATCH] fix(rules): don't crash on require() (#114) --- rules/__tests__/no-jest-import.test.js | 1 + rules/no-jest-import.js | 6 +++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/rules/__tests__/no-jest-import.test.js b/rules/__tests__/no-jest-import.test.js index 8ebb05d2d..9907b20d6 100644 --- 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 index 4b8a7a153..45414994a 100644 --- 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: {