From 885ce1798d9b0bd5b6685675f8456062d34fc551 Mon Sep 17 00:00:00 2001 From: Simen Bekkhus Date: Sun, 27 May 2018 23:24:54 +0200 Subject: [PATCH] fix: handle callexpressions with anonymous calles --- rules/__tests__/no-jasmine-globals.test.js | 1 + rules/no-jasmine-globals.js | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/rules/__tests__/no-jasmine-globals.test.js b/rules/__tests__/no-jasmine-globals.test.js index 6dd66c9a4..13410035f 100644 --- a/rules/__tests__/no-jasmine-globals.test.js +++ b/rules/__tests__/no-jasmine-globals.test.js @@ -14,6 +14,7 @@ ruleTester.run('no-jasmine-globals', rule, { 'it("foo", function () {})', 'test("foo", function () {})', 'foo()', + `require('foo')('bar')`, ], invalid: [ { diff --git a/rules/no-jasmine-globals.js b/rules/no-jasmine-globals.js index 769a84775..40a2ed833 100644 --- a/rules/no-jasmine-globals.js +++ b/rules/no-jasmine-globals.js @@ -15,6 +15,10 @@ module.exports = { CallExpression(node) { const calleeName = getNodeName(node.callee); + if (!calleeName) { + return; + } + if (calleeName === 'spyOn' || calleeName === 'spyOnProperty') { context.report({ node,