fix(no-jasmine-globals): Fix false positives for pending/fail/spyOn

Prior to this, similarly named local variables & parameters would trigger the
no-jasmine-globals warning

Fixes #156

Author: jdelStrother

rules/__tests__/no-jasmine-globals.test.js

@@ -15,6 +15,9 @@ ruleTester.run('no-jasmine-globals', rule, {
     'test("foo", function () {})',
     'foo()',
     `require('foo')('bar')`,
+    'function callback(fail) { fail() }',
+    'var spyOn = require("actions"); spyOn("foo")',
+    'function callback(pending) { pending() }',
   ],
   invalid: [
     {

rules/no-jasmine-globals.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { getDocsUrl, getNodeName } = require('./util');
+const { getDocsUrl, getNodeName, scopeHasLocalReference } = require('./util');
 
 const globalAlternatives = {
   spyOn: 'Illegal usage of global `spyOn`, prefer `jest.spyOn`',
@@ -32,6 +32,11 @@ module.exports = {
           calleeName === 'fail' ||
           calleeName === 'pending'
         ) {
+          if (scopeHasLocalReference(context.getScope(), calleeName)) {
+            // It's a local variable, not a jasmine global.
+            return;
+          }
+
           context.report({
             node,
             message: globalAlternatives[calleeName],