eslint-plugin-jest 26.8.7 → 27.1.6

package/lib/rules/valid-title.js

@@ -4,32 +4,23 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 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;
   }
-
   if (binaryExp.left.type === _utils.AST_NODE_TYPES.BinaryExpression) {
     return doesBinaryExpressionContainStringNode(binaryExp.left);
   }
-
   return (0, _utils2.isStringNode)(binaryExp.left);
 };
-
 const quoteStringValue = node => node.type === _utils.AST_NODE_TYPES.TemplateLiteral ? `\`${node.quasis[0].value.raw}\`` : node.raw;
-
 const compileMatcherPattern = matcherMaybeWithMessage => {
   const [matcher, message] = Array.isArray(matcherMaybeWithMessage) ? matcherMaybeWithMessage : [matcherMaybeWithMessage];
   return [new RegExp(matcher, 'u'), message];
 };
-
 const compileMatcherPatterns = matchers => {
   if (typeof matchers === 'string' || Array.isArray(matchers)) {
     const compiledMatcher = compileMatcherPattern(matchers);
@@ -39,14 +30,12 @@ const compileMatcherPatterns = matchers => {
       it: compiledMatcher
     };
   }
-
   return {
     describe: matchers.describe ? compileMatcherPattern(matchers.describe) : null,
     test: matchers.test ? compileMatcherPattern(matchers.test) : null,
     it: matchers.it ? compileMatcherPattern(matchers.it) : null
   };
 };
-
 const MatcherAndMessageSchema = {
   type: 'array',
   items: {
@@ -56,7 +45,6 @@ const MatcherAndMessageSchema = {
   maxItems: 2,
   additionalItems: false
 };
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -116,7 +104,6 @@ var _default = (0, _utils2.createRule)({
     ignoreTypeOfDescribeName: false,
     disallowedWords: []
   }],
-
   create(context, [{
     ignoreTypeOfDescribeName,
     disallowedWords = [],
@@ -124,41 +111,31 @@ var _default = (0, _utils2.createRule)({
     mustMatch
   }]) {
     const disallowedWordsRegexp = new RegExp(`\\b(${disallowedWords.join('|')})\\b`, 'iu');
-    const mustNotMatchPatterns = compileMatcherPatterns(mustNotMatch !== null && mustNotMatch !== void 0 ? mustNotMatch : {});
-    const mustMatchPatterns = compileMatcherPatterns(mustMatch !== null && mustMatch !== void 0 ? mustMatch : {});
+    const mustNotMatchPatterns = compileMatcherPatterns(mustNotMatch ?? {});
+    const mustMatchPatterns = compileMatcherPatterns(mustMatch ?? {});
     return {
       CallExpression(node) {
-        var _mustNotMatchPatterns, _mustMatchPatterns$je;
-
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-
         if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'describe' && (jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'test') {
           return;
         }
-
         const [argument] = node.arguments;
-
         if (!argument) {
           return;
         }
-
         if (!(0, _utils2.isStringNode)(argument)) {
           if (argument.type === _utils.AST_NODE_TYPES.BinaryExpression && doesBinaryExpressionContainStringNode(argument)) {
             return;
           }
-
           if (argument.type !== _utils.AST_NODE_TYPES.TemplateLiteral && !(ignoreTypeOfDescribeName && jestFnCall.type === 'describe')) {
             context.report({
               messageId: 'titleMustBeString',
               loc: argument.loc
             });
           }
-
           return;
         }
-
         const title = (0, _utils2.getStringValue)(argument);
-
         if (!title) {
           context.report({
             messageId: 'emptyTitle',
@@ -169,10 +146,8 @@ var _default = (0, _utils2.createRule)({
           });
           return;
         }
-
         if (disallowedWords.length > 0) {
           const disallowedMatch = disallowedWordsRegexp.exec(title);
-
           if (disallowedMatch) {
             context.report({
               data: {
@@ -184,7 +159,6 @@ var _default = (0, _utils2.createRule)({
             return;
           }
         }
-
         if (title.trim().length !== title.length) {
           context.report({
             messageId: 'accidentalSpace',
@@ -192,10 +166,8 @@ var _default = (0, _utils2.createRule)({
             fix: fixer => [fixer.replaceTextRange(argument.range, quoteStringValue(argument).replace(/^([`'"]) +?/u, '$1').replace(/ +?([`'"])$/u, '$1'))]
           });
         }
-
         const unprefixedName = trimFXprefix(jestFnCall.name);
         const [firstWord] = title.split(' ');
-
         if (firstWord.toLowerCase() === unprefixedName) {
           context.report({
             messageId: 'duplicatePrefix',
@@ -203,10 +175,8 @@ var _default = (0, _utils2.createRule)({
             fix: fixer => [fixer.replaceTextRange(argument.range, quoteStringValue(argument).replace(/^([`'"]).+? /u, '$1'))]
           });
         }
-
         const jestFunctionName = unprefixedName;
-        const [mustNotMatchPattern, mustNotMatchMessage] = (_mustNotMatchPatterns = mustNotMatchPatterns[jestFunctionName]) !== null && _mustNotMatchPatterns !== void 0 ? _mustNotMatchPatterns : [];
-
+        const [mustNotMatchPattern, mustNotMatchMessage] = mustNotMatchPatterns[jestFunctionName] ?? [];
         if (mustNotMatchPattern) {
           if (mustNotMatchPattern.test(title)) {
             context.report({
@@ -221,9 +191,7 @@ var _default = (0, _utils2.createRule)({
             return;
           }
         }
-
-        const [mustMatchPattern, mustMatchMessage] = (_mustMatchPatterns$je = mustMatchPatterns[jestFunctionName]) !== null && _mustMatchPatterns$je !== void 0 ? _mustMatchPatterns$je : [];
-
+        const [mustMatchPattern, mustMatchMessage] = mustMatchPatterns[jestFunctionName] ?? [];
         if (mustMatchPattern) {
           if (!mustMatchPattern.test(title)) {
             context.report({
@@ -239,10 +207,7 @@ var _default = (0, _utils2.createRule)({
           }
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "26.8.7",
+  "version": "27.1.6",
   "description": "ESLint rules for Jest",
   "keywords": [
     "eslint",
@@ -29,7 +29,7 @@
     "prettier:write": "prettier --write 'docs/**/*.md' README.md '.github/**' package.json tsconfig.json src/globals.json .yarnrc.yml",
     "postpublish": "pinst --enable",
     "test": "jest",
-    "tools:regenerate-docs": "ts-node -T tools/regenerate-docs",
+    "tools:regenerate-docs": "yarn prepack && eslint-doc-generator && yarn prettier:write",
     "typecheck": "tsc -p ."
   },
   "commitlint": {
@@ -103,34 +103,38 @@
     "@babel/core": "^7.4.4",
     "@babel/preset-env": "^7.4.4",
     "@babel/preset-typescript": "^7.3.3",
-    "@commitlint/cli": "^16.0.0",
-    "@commitlint/config-conventional": "^16.0.0",
+    "@commitlint/cli": "^17.0.3",
+    "@commitlint/config-conventional": "^17.0.3",
     "@schemastore/package": "^0.0.6",
     "@semantic-release/changelog": "^6.0.0",
     "@semantic-release/git": "^10.0.0",
+    "@tsconfig/node14": "^1.0.3",
     "@types/dedent": "^0.7.0",
-    "@types/jest": "^28.0.0",
-    "@types/node": "^16.0.0",
+    "@types/eslint": "^8.4.6",
+    "@types/jest": "^29.0.0",
+    "@types/node": "^14.18.26",
     "@types/prettier": "^2.0.0",
     "@typescript-eslint/eslint-plugin": "^5.0.0",
     "@typescript-eslint/parser": "^5.0.0",
-    "babel-jest": "^28.0.0",
+    "babel-jest": "^29.0.0",
     "babel-plugin-replace-ts-export-assignment": "^0.0.2",
     "dedent": "^0.7.0",
     "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0",
     "eslint-config-prettier": "^8.3.0",
+    "eslint-doc-generator": "^0.26.0",
     "eslint-plugin-eslint-comments": "^3.1.2",
-    "eslint-plugin-eslint-plugin": "^4.0.0",
+    "eslint-plugin-eslint-plugin": "^5.0.6",
     "eslint-plugin-import": "^2.25.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-prettier": "^3.4.1",
+    "eslint-plugin-prettier": "^4.2.1",
     "eslint-remote-tester": "^3.0.0",
     "eslint-remote-tester-repositories": "~0.0.5",
-    "husky": "^7.0.2",
+    "husky": "^8.0.1",
     "is-ci": "^3.0.0",
-    "jest": "^28.0.0",
-    "jest-runner-eslint": "^1.0.0",
-    "lint-staged": "^12.0.0",
+    "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",
@@ -141,7 +145,7 @@
   },
   "peerDependencies": {
     "@typescript-eslint/eslint-plugin": "^5.0.0",
-    "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0"
+    "eslint": "^7.0.0 || ^8.0.0"
   },
   "peerDependenciesMeta": {
     "@typescript-eslint/eslint-plugin": {
@@ -151,8 +155,8 @@
       "optional": true
     }
   },
-  "packageManager": "yarn@3.2.2",
+  "packageManager": "yarn@3.3.0",
   "engines": {
-    "node": "^12.22.0 || ^14.17.0 || >=16.0.0"
+    "node": "^14.15.0 || ^16.10.0 || >=18.0.0"
   }
 }

package/docs/rules/no-jest-import.md

@@ -1,20 +0,0 @@
-# Disallow importing Jest (`no-jest-import`)
-
-The `jest` object is automatically in scope within every test file. The methods
-in the `jest` object help create mocks and let you control Jest's overall
-behavior. It is therefore completely unnecessary to import in `jest`, as Jest
-doesn't export anything in the first place.
-
-### Rule details
-
-This rule reports on any importing of Jest.
-
-To name a few: `var jest = require('jest');` `const jest = require('jest');`
-`import jest from 'jest';` `import {jest as test} from 'jest';`
-
-There is no correct usage of this code, other than to not import `jest` in the
-first place.
-
-## Further Reading
-
-- [The Jest Object](https://facebook.github.io/jest/docs/en/jest-object.html)

package/lib/rules/no-jest-import.js

@@ -1,48 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = void 0;
-
-var _utils = require("./utils");
-
-var _default = (0, _utils.createRule)({
-  name: __filename,
-  meta: {
-    type: 'problem',
-    docs: {
-      description: 'Disallow importing Jest',
-      category: 'Best Practices',
-      recommended: 'error'
-    },
-    messages: {
-      unexpectedImport: `Jest is automatically in scope. Do not import "jest", as Jest doesn't export anything.`
-    },
-    schema: []
-  },
-  defaultOptions: [],
-
-  create(context) {
-    return {
-      'ImportDeclaration[source.value="jest"]'(node) {
-        context.report({
-          node,
-          messageId: 'unexpectedImport'
-        });
-      },
-
-      'CallExpression[callee.name="require"][arguments.0.value="jest"]'(node) {
-        context.report({
-          loc: node.arguments[0].loc,
-          messageId: 'unexpectedImport',
-          node
-        });
-      }
-
-    };
-  }
-
-});
-
-exports.default = _default;