eslint-plugin-jest 27.1.4 → 27.1.5

package/lib/rules/no-disabled-tests.js

@@ -68,7 +68,7 @@ var _default = (0, _utils.createRule)({
         }
       },
       'CallExpression[callee.name="pending"]'(node) {
-        if ((0, _utils.scopeHasLocalReference)(context.getScope(), 'pending')) {
+        if ((0, _utils.resolveScope)(context.getScope(), 'pending')) {
           return;
         }
         if (testDepth > 0) {

package/lib/rules/no-jasmine-globals.js

@@ -37,7 +37,7 @@ var _default = (0, _utils2.createRule)({
           return;
         }
         if (calleeName === 'spyOn' || calleeName === 'spyOnProperty' || calleeName === 'fail' || calleeName === 'pending') {
-          if ((0, _utils2.scopeHasLocalReference)(context.getScope(), calleeName)) {
+          if ((0, _utils2.resolveScope)(context.getScope(), calleeName)) {
             // It's a local variable, not a jasmine global.
             return;
           }

package/lib/rules/utils/parseJestFnCall.js

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.getNodeChain = getNodeChain;
-exports.scopeHasLocalReference = exports.parseJestFnCallWithReason = exports.parseJestFnCall = exports.isTypeOfJestFnCall = void 0;
+exports.resolveScope = exports.parseJestFnCallWithReason = exports.parseJestFnCall = exports.isTypeOfJestFnCall = void 0;
 var _utils = require("@typescript-eslint/utils");
 var _utils2 = require("../utils");
 const isTypeOfJestFnCall = (node, context, types) => {
@@ -286,55 +286,23 @@ const describePossibleImportDef = def => {
   }
   return null;
 };
-const collectReferences = scope => {
-  const locals = new Set();
-  const imports = new Map();
-  const unresolved = new Set();
-  let currentScope = scope;
-  while (currentScope !== null) {
-    for (const ref of currentScope.variables) {
-      if (ref.defs.length === 0) {
-        continue;
-      }
-      const def = ref.defs[ref.defs.length - 1];
-      const importDetails = describePossibleImportDef(def);
-      if (importDetails) {
-        imports.set(importDetails.local, importDetails);
-        continue;
-      }
-      locals.add(ref.name);
-    }
-    for (const ref of currentScope.through) {
-      unresolved.add(ref.identifier.name);
-    }
-    currentScope = currentScope.upper;
-  }
-  return {
-    locals,
-    imports,
-    unresolved
-  };
-};
 const resolveScope = (scope, identifier) => {
   let currentScope = scope;
   while (currentScope !== null) {
-    for (const ref of currentScope.variables) {
-      if (ref.defs.length === 0) {
-        continue;
-      }
+    const ref = currentScope.set.get(identifier);
+    if (ref && ref.defs.length > 0) {
       const def = ref.defs[ref.defs.length - 1];
       const importDetails = describePossibleImportDef(def);
       if ((importDetails === null || importDetails === void 0 ? void 0 : importDetails.local) === identifier) {
         return importDetails;
       }
-      if (ref.name === identifier) {
-        return 'local';
-      }
+      return 'local';
     }
     currentScope = currentScope.upper;
   }
   return null;
 };
+exports.resolveScope = resolveScope;
 const resolveToJestFn = (context, identifier) => {
   const maybeImport = resolveScope(context.getScope(), identifier);
 
@@ -360,17 +328,4 @@ const resolveToJestFn = (context, identifier) => {
     local: identifier,
     type: 'global'
   };
-};
-const scopeHasLocalReference = (scope, referenceName) => {
-  const references = collectReferences(scope);
-  return (
-    // referenceName was found as a local variable or function declaration.
-    references.locals.has(referenceName) ||
-    // referenceName was found as an imported identifier
-    references.imports.has(referenceName) ||
-    // referenceName was not found as an unresolved reference,
-    // meaning it is likely not an implicit global reference.
-    !references.unresolved.has(referenceName)
-  );
-};
-exports.scopeHasLocalReference = scopeHasLocalReference;
+};

package/lib/rules/valid-title.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var _utils = require("@typescript-eslint/utils");
 var _utils2 = require("./utils");
-const trimFXprefix = word => ['f', 'x'].includes(word.charAt(0)) ? word.substr(1) : word;
+const trimFXprefix = word => ['f', 'x'].includes(word.charAt(0)) ? word.substring(1) : word;
 const doesBinaryExpressionContainStringNode = binaryExp => {
   if ((0, _utils2.isStringNode)(binaryExp.right)) {
     return true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "27.1.4",
+  "version": "27.1.5",
   "description": "ESLint rules for Jest",
   "keywords": [
     "eslint",