Skip to content

Commit 523e2f4

Browse files
jacekkopeckySimenB
authored andcommittedMay 27, 2018
fix(rules): don't crash on require() (#115)
fixes #114
1 parent 4586bdc commit 523e2f4

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed
 

Diff for: ‎rules/__tests__/no-jest-import.test.js

+1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ ruleTester.run('no-jest-import', rule, {
1212
parserOptions: { sourceType: 'module' },
1313
},
1414
'require("somethingElse")',
15+
'require()',
1516
'entirelyDifferent(fn)',
1617
],
1718
invalid: [

Diff for: ‎rules/no-jest-import.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,11 @@ module.exports = {
2222
},
2323
CallExpression(node) {
2424
const calleeName = getNodeName(node.callee);
25-
if (calleeName === 'require' && node.arguments[0].value === 'jest') {
25+
if (
26+
calleeName === 'require' &&
27+
node.arguments[0] &&
28+
node.arguments[0].value === 'jest'
29+
) {
2630
context.report({
2731
loc: {
2832
end: {

0 commit comments

Comments
 (0)
Please sign in to comment.