eslint-plugin-jest 27.0.0 → 27.0.3

package/README.md

@@ -257,7 +257,7 @@ that are powered by type-checking information provided by TypeScript.
 
 In order to use these rules, you must be using `@typescript-eslint/parser` &
 adjust your eslint config as outlined
-[here](https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/TYPED_LINTING.md)
+[here](https://github.com/typescript-eslint/typescript-eslint/blob/main/docs/linting/TYPED_LINTING.md).
 
 Note that unlike the type-checking rules in `@typescript-eslint/eslint-plugin`,
 the rules here will fallback to doing nothing if type information is not

package/docs/rules/no-deprecated-functions.md

@@ -20,13 +20,12 @@ This rule can also autofix a number of these deprecations for you.
 
 ### `jest.resetModuleRegistry`
 
-This function was renamed to `resetModules` in Jest 15, and is scheduled for
-removal in Jest 27.
+This function was renamed to `resetModules` in Jest 15 and removed in Jest 27.
 
 ### `jest.addMatchers`
 
-This function was replaced with `expect.extend` in Jest 17, and is scheduled for
-removal in Jest 27.
+This function was replaced with `expect.extend` in Jest 17 and removed in
+Jest 27.
 
 ### `require.requireActual` & `require.requireMock`
 
@@ -41,10 +40,10 @@ the release of Jest 26 saw them removed from the `require` function altogether.
 
 ### `jest.runTimersToTime`
 
-This function was renamed to `advanceTimersByTime` in Jest 22, and is scheduled
-for removal in Jest 27.
+This function was renamed to `advanceTimersByTime` in Jest 22 and removed in
+Jest 27.
 
 ### `jest.genMockFromModule`
 
 This function was renamed to `createMockFromModule` in Jest 26, and is scheduled
-for removal in a future version of Jest.
+for removal in Jest 30.

package/lib/index.js

@@ -48,15 +48,12 @@ module.exports = {
   configs: {
     all: createConfig(allRules),
     recommended: createConfig(recommendedRules),
-    style: {
-      plugins: ['jest'],
-      rules: {
-        'jest/no-alias-methods': 'warn',
-        'jest/prefer-to-be': 'error',
-        'jest/prefer-to-contain': 'error',
-        'jest/prefer-to-have-length': 'error'
-      }
-    }
+    style: createConfig({
+      'jest/no-alias-methods': 'warn',
+      'jest/prefer-to-be': 'error',
+      'jest/prefer-to-contain': 'error',
+      'jest/prefer-to-have-length': 'error'
+    })
   },
   environments: {
     globals: {

package/lib/rules/no-done-callback.js

@@ -33,8 +33,7 @@ var _default = (0, _utils2.createRule)({
     docs: {
       category: 'Best Practices',
       description: 'Avoid using a callback in asynchronous tests and hooks',
-      recommended: 'error',
-      suggestion: true
+      recommended: 'error'
     },
     messages: {
       noDoneCallback: 'Return a Promise instead of relying on callback parameter',

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

@@ -15,8 +15,7 @@ var _default = (0, _utils2.createRule)({
     docs: {
       category: 'Best Practices',
       description: 'Disallow focused tests',
-      recommended: 'error',
-      suggestion: true
+      recommended: 'error'
     },
     messages: {
       focusedTest: 'Unexpected focused test.',

package/lib/rules/no-restricted-matchers.js

@@ -7,6 +7,14 @@ exports.default = void 0;
 
 var _utils = require("./utils");
 
+const isChainRestricted = (chain, restriction) => {
+  if (_utils.ModifierName.hasOwnProperty(restriction) || restriction.endsWith('.not')) {
+    return chain.startsWith(restriction);
+  }
+
+  return chain === restriction;
+};
+
 var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
@@ -41,7 +49,7 @@ var _default = (0, _utils.createRule)({
         const chain = jestFnCall.members.map(nod => (0, _utils.getAccessorValue)(nod)).join('.');
 
         for (const [restriction, message] of Object.entries(restrictedChains)) {
-          if (chain.startsWith(restriction)) {
+          if (isChainRestricted(chain, restriction)) {
             context.report({
               messageId: message ? 'restrictedChainWithMessage' : 'restrictedChain',
               data: {

package/lib/rules/prefer-equality-matcher.js

@@ -15,8 +15,7 @@ var _default = (0, _utils2.createRule)({
     docs: {
       category: 'Best Practices',
       description: 'Suggest using the built-in equality matchers',
-      recommended: false,
-      suggestion: true
+      recommended: false
     },
     messages: {
       useEqualityMatcher: 'Prefer using one of the equality matchers instead',

package/lib/rules/prefer-expect-assertions.js

@@ -13,10 +13,16 @@ const isFirstStatement = node => {
   let parent = node;
 
   while (parent) {
-    var _parent$parent;
+    var _parent$parent, _parent$parent2;
 
     if (((_parent$parent = parent.parent) === null || _parent$parent === void 0 ? void 0 : _parent$parent.type) === _utils.AST_NODE_TYPES.BlockStatement) {
       return parent.parent.body[0] === parent;
+    } // if we've hit an arrow function, then it must have a single expression
+    // as its body, as otherwise we would have hit the block statement already
+
+
+    if (((_parent$parent2 = parent.parent) === null || _parent$parent2 === void 0 ? void 0 : _parent$parent2.type) === _utils.AST_NODE_TYPES.ArrowFunctionExpression) {
+      return true;
     }
 
     parent = parent.parent;
@@ -32,18 +38,13 @@ const suggestRemovingExtraArguments = (args, extraArgsStartAt) => ({
   fix: fixer => fixer.removeRange([args[extraArgsStartAt].range[0] - Math.sign(extraArgsStartAt), args[args.length - 1].range[1]])
 });
 
-// const suggestions: Array<[MessageIds, string]> = [
-//   ['suggestAddingHasAssertions', 'expect.hasAssertions();'],
-//   ['suggestAddingAssertions', 'expect.assertions();'],
-// ];
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
       category: 'Best Practices',
       description: 'Suggest using `expect.assertions()` OR `expect.hasAssertions()`',
-      recommended: false,
-      suggestion: true
+      recommended: false
     },
     messages: {
       hasAssertionsTakesNoArguments: '`expect.hasAssertions` expects no arguments',

package/lib/rules/prefer-strict-equal.js

@@ -13,8 +13,7 @@ var _default = (0, _utils.createRule)({
     docs: {
       category: 'Best Practices',
       description: 'Suggest using `toStrictEqual()`',
-      recommended: false,
-      suggestion: true
+      recommended: false
     },
     messages: {
       useToStrictEqual: 'Use `toStrictEqual()` instead',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "27.0.0",
+  "version": "27.0.3",
   "description": "ESLint rules for Jest",
   "keywords": [
     "eslint",
@@ -109,7 +109,7 @@
     "@semantic-release/changelog": "^6.0.0",
     "@semantic-release/git": "^10.0.0",
     "@types/dedent": "^0.7.0",
-    "@types/jest": "^28.0.0",
+    "@types/jest": "^29.0.0",
     "@types/node": "^14.18.26",
     "@types/prettier": "^2.0.0",
     "@typescript-eslint/eslint-plugin": "^5.0.0",
@@ -131,6 +131,7 @@
     "jest": "^29.0.0",
     "jest-runner-eslint": "^1.1.0",
     "lint-staged": "^13.0.3",
+    "markdown-link-check": "^3.10.2",
     "pinst": "^3.0.0",
     "prettier": "^2.0.5",
     "rimraf": "^3.0.0",