eslint-plugin-jest 29.8.0 → 29.9.0

package/docs/rules/no-unnecessary-assertion.md

@@ -10,9 +10,9 @@ provided that the types are accurate.
 
 ## Rule details
 
-This rule warns when you do an assertion about being `null` or `undefined` on
-something that cannot be those types, as that indicates either the assertion can
-be removed or the types need to be adjusted.
+This rule warns when you do an assertion about being `null`, `undefined`, or
+`NaN` on something that cannot be those types, as that indicates either the
+assertion can be removed or the types need to be adjusted.
 
 The following patterns are considered warnings:
 
@@ -22,6 +22,8 @@ expect('hello world'.match('sunshine') ?? []).toBeNull();
 expect(User.findOrThrow(1)).toBeDefined();
 
 expect(map.getOrInsert('key', 'default')).not.toBeUndefined();
+
+expect(user.name).not.toBeNaN();
 ```
 
 The following patterns are not considered warnings:
@@ -32,4 +34,6 @@ expect('hello world'.match('sunshine')).toBeNull();
 expect(User.findOrNull(1)).toBeDefined();
 
 expect(map.get('key')).not.toBeUndefined();
+
+expect(user.age).not.toBeNaN();
 ```

package/lib/rules/no-error-equal.js

@@ -6,70 +6,6 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var _utils = require("@typescript-eslint/utils");
 var _utils2 = require("./utils");
-let SymbolFlags;
-let TypeFlags;
-function isSymbolFromDefaultLibrary(program, symbol) {
-  /* istanbul ignore next */
-  const declarations = symbol.getDeclarations() ?? [];
-  for (const declaration of declarations) {
-    const sourceFile = declaration.getSourceFile();
-
-    /* istanbul ignore else */
-    if (program.isSourceFileDefaultLibrary(sourceFile)) {
-      return true;
-    }
-  }
-
-  /* istanbul ignore next */
-  return false;
-}
-function isBuiltinSymbolLike(program, type, symbolName) {
-  return isBuiltinSymbolLikeRecurser(program, type, subType => {
-    const symbol = subType.getSymbol();
-    if (!symbol) {
-      return false;
-    }
-    const actualSymbolName = symbol.getName();
-    if (actualSymbolName === symbolName && isSymbolFromDefaultLibrary(program, symbol)) {
-      return true;
-    }
-    return null;
-  });
-}
-function isBuiltinSymbolLikeRecurser(program, type, predicate) {
-  if (type.isIntersection()) {
-    return type.types.some(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
-  }
-  if (type.isUnion()) {
-    return type.types.every(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
-  }
-  if (isTypeParameter(type)) {
-    const t = type.getConstraint();
-    if (t) {
-      return isBuiltinSymbolLikeRecurser(program, t, predicate);
-    }
-    return false;
-  }
-  const predicateResult = predicate(type);
-  if (typeof predicateResult === 'boolean') {
-    return predicateResult;
-  }
-  const symbol = type.getSymbol();
-
-  /* istanbul ignore next */
-  if (symbol && symbol.flags & (SymbolFlags.Class | SymbolFlags.Interface)) {
-    const checker = program.getTypeChecker();
-    for (const baseType of checker.getBaseTypes(type)) {
-      if (isBuiltinSymbolLikeRecurser(program, baseType, predicate)) {
-        return true;
-      }
-    }
-  }
-  return false;
-}
-function isTypeParameter(type) {
-  return (type.flags & TypeFlags.TypeParameter) !== 0;
-}
 var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -86,12 +22,6 @@ var _default = exports.default = (0, _utils2.createRule)({
   defaultOptions: [],
   create(context) {
     const services = _utils.ESLintUtils.getParserServices(context);
-
-    // eslint-disable-next-line @typescript-eslint/no-require-imports
-    ({
-      TypeFlags,
-      SymbolFlags
-    } = require('typescript'));
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
@@ -99,7 +29,7 @@ var _default = exports.default = (0, _utils2.createRule)({
           return;
         }
         const [argument] = jestFnCall.head.node.parent.arguments;
-        if (isBuiltinSymbolLike(services.program, services.getTypeAtLocation(argument), 'Error')) {
+        if ((0, _utils2.isBuiltinSymbolLike)(services.program, services.getTypeAtLocation(argument), 'Error')) {
           context.report({
             messageId: 'equalError',
             node

package/lib/rules/no-unnecessary-assertion.js

@@ -58,7 +58,7 @@ var _default = exports.default = (0, _utils2.createRule)({
           return;
         }
         const matcherName = (0, _utils2.getAccessorValue)(jestFnCall.matcher);
-        if (!['toBeNull', 'toBeDefined', 'toBeUndefined'].includes(matcherName)) {
+        if (!['toBeNull', 'toBeDefined', 'toBeUndefined', 'toBeNaN'].includes(matcherName)) {
           return;
         }
 
@@ -67,12 +67,12 @@ var _default = exports.default = (0, _utils2.createRule)({
           return;
         }
         const [argument] = jestFnCall.head.node.parent.arguments;
-        const isNullable = canBe(services.getTypeAtLocation(argument), matcherName === 'toBeNull' ? TypeFlags.Null : TypeFlags.Undefined);
-        if (!isNullable) {
+        const isTypePossible = canBe(services.getTypeAtLocation(argument), matcherName === 'toBeNaN' ? TypeFlags.NumberLike : matcherName === 'toBeNull' ? TypeFlags.Null : TypeFlags.Undefined);
+        if (!isTypePossible) {
           context.report({
             messageId: 'unnecessaryAssertion',
             data: {
-              thing: matcherName === 'toBeNull' ? 'null' : 'undefined'
+              thing: matcherName === 'toBeNaN' ? 'a number' : matcherName === 'toBeNull' ? 'null' : 'undefined'
             },
             node
           });

package/lib/rules/utils/index.js

@@ -57,4 +57,15 @@ Object.keys(_parseJestFnCall).forEach(function (key) {
       return _parseJestFnCall[key];
     }
   });
+});
+var _ts = require("./ts");
+Object.keys(_ts).forEach(function (key) {
+  if (key === "default" || key === "__esModule") return;
+  if (key in exports && exports[key] === _ts[key]) return;
+  Object.defineProperty(exports, key, {
+    enumerable: true,
+    get: function () {
+      return _ts[key];
+    }
+  });
 });

package/lib/rules/utils/ts.js

@@ -0,0 +1,73 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.isBuiltinSymbolLike = isBuiltinSymbolLike;
+let SymbolFlags;
+let TypeFlags;
+function isSymbolFromDefaultLibrary(program, symbol) {
+  /* istanbul ignore next */
+  const declarations = symbol.getDeclarations() ?? [];
+  for (const declaration of declarations) {
+    const sourceFile = declaration.getSourceFile();
+
+    /* istanbul ignore else */
+    if (program.isSourceFileDefaultLibrary(sourceFile)) {
+      return true;
+    }
+  }
+
+  /* istanbul ignore next */
+  return false;
+}
+function isBuiltinSymbolLike(program, type, symbolName) {
+  // eslint-disable-next-line @typescript-eslint/no-require-imports
+  ({
+    TypeFlags,
+    SymbolFlags
+  } = require('typescript'));
+  return isBuiltinSymbolLikeRecurser(program, type, subType => {
+    const symbol = subType.getSymbol();
+    if (!symbol) {
+      return false;
+    }
+    const actualSymbolName = symbol.getName();
+    if (actualSymbolName === symbolName && isSymbolFromDefaultLibrary(program, symbol)) {
+      return true;
+    }
+    return null;
+  });
+}
+function isBuiltinSymbolLikeRecurser(program, type, predicate) {
+  if (type.isIntersection()) {
+    return type.types.some(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
+  }
+  if (type.isUnion()) {
+    return type.types.every(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
+  }
+  if (isTypeParameter(type)) {
+    const t = type.getConstraint();
+    if (t) {
+      return isBuiltinSymbolLikeRecurser(program, t, predicate);
+    }
+    return false;
+  }
+  const predicateResult = predicate(type);
+  if (typeof predicateResult === 'boolean') {
+    return predicateResult;
+  }
+  const symbol = type.getSymbol();
+  if (symbol && symbol.flags & (SymbolFlags.Class | SymbolFlags.Interface)) {
+    const checker = program.getTypeChecker();
+    for (const baseType of checker.getBaseTypes(type)) {
+      if (isBuiltinSymbolLikeRecurser(program, baseType, predicate)) {
+        return true;
+      }
+    }
+  }
+  return false;
+}
+function isTypeParameter(type) {
+  return (type.flags & TypeFlags.TypeParameter) !== 0;
+}

package/lib/rules/valid-expect-with-promise.js

@@ -6,70 +6,6 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var _utils = require("@typescript-eslint/utils");
 var _utils2 = require("./utils");
-let SymbolFlags;
-let TypeFlags;
-function isSymbolFromDefaultLibrary(program, symbol) {
-  /* istanbul ignore next */
-  const declarations = symbol.getDeclarations() ?? [];
-  for (const declaration of declarations) {
-    const sourceFile = declaration.getSourceFile();
-
-    /* istanbul ignore else */
-    if (program.isSourceFileDefaultLibrary(sourceFile)) {
-      return true;
-    }
-  }
-
-  /* istanbul ignore next */
-  return false;
-}
-function isBuiltinSymbolLike(program, type, symbolName) {
-  return isBuiltinSymbolLikeRecurser(program, type, subType => {
-    const symbol = subType.getSymbol();
-    if (!symbol) {
-      return false;
-    }
-    const actualSymbolName = symbol.getName();
-    if (actualSymbolName === symbolName && isSymbolFromDefaultLibrary(program, symbol)) {
-      return true;
-    }
-    return null;
-  });
-}
-function isBuiltinSymbolLikeRecurser(program, type, predicate) {
-  if (type.isIntersection()) {
-    return type.types.some(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
-  }
-  if (type.isUnion()) {
-    return type.types.every(t => isBuiltinSymbolLikeRecurser(program, t, predicate));
-  }
-  if (isTypeParameter(type)) {
-    const t = type.getConstraint();
-    if (t) {
-      return isBuiltinSymbolLikeRecurser(program, t, predicate);
-    }
-    return false;
-  }
-  const predicateResult = predicate(type);
-  if (typeof predicateResult === 'boolean') {
-    return predicateResult;
-  }
-  const symbol = type.getSymbol();
-
-  /* istanbul ignore next */
-  if (symbol && symbol.flags & (SymbolFlags.Class | SymbolFlags.Interface)) {
-    const checker = program.getTypeChecker();
-    for (const baseType of checker.getBaseTypes(type)) {
-      if (isBuiltinSymbolLikeRecurser(program, baseType, predicate)) {
-        return true;
-      }
-    }
-  }
-  return false;
-}
-function isTypeParameter(type) {
-  return (type.flags & TypeFlags.TypeParameter) !== 0;
-}
 var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -87,12 +23,6 @@ var _default = exports.default = (0, _utils2.createRule)({
   defaultOptions: [],
   create(context) {
     const services = _utils.ESLintUtils.getParserServices(context);
-
-    // eslint-disable-next-line @typescript-eslint/no-require-imports
-    ({
-      TypeFlags,
-      SymbolFlags
-    } = require('typescript'));
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
@@ -100,7 +30,7 @@ var _default = exports.default = (0, _utils2.createRule)({
           return;
         }
         const [argument] = jestFnCall.head.node.parent.arguments;
-        const isPromiseLike = isBuiltinSymbolLike(services.program, services.getTypeAtLocation(argument), 'Promise');
+        const isPromiseLike = (0, _utils2.isBuiltinSymbolLike)(services.program, services.getTypeAtLocation(argument), 'Promise');
         const promiseModifier = jestFnCall.modifiers.find(nod => (0, _utils2.getAccessorValue)(nod) !== 'not');
         if (isPromiseLike && !promiseModifier) {
           context.report({

package/package.json

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