eslint-plugin-jest 28.11.1 → 28.12.0

package/lib/rules/no-commented-out-tests.js

@@ -15,7 +15,7 @@ var _default = exports.default = (0, _utils.createRule)({
       description: 'Disallow commented out tests'
     },
     messages: {
-      commentedTests: 'Some tests seem to be commented'
+      commentedTests: 'Do not comment out tests'
     },
     schema: [],
     type: 'suggestion'

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

@@ -13,30 +13,20 @@ var _default = exports.default = (0, _utils.createRule)({
     },
     messages: {
       missingFunction: 'Test is missing function argument',
-      pending: 'Call to pending()',
-      pendingSuite: 'Call to pending() within test suite',
-      pendingTest: 'Call to pending() within test',
-      disabledSuite: 'Disabled test suite',
-      disabledTest: 'Disabled test'
+      skippedTest: 'Tests should not be skipped'
     },
     schema: [],
     type: 'suggestion'
   },
   defaultOptions: [],
   create(context) {
-    let suiteDepth = 0;
-    let testDepth = 0;
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
         if (!jestFnCall) {
           return;
         }
-        if (jestFnCall.type === 'describe') {
-          suiteDepth++;
-        }
         if (jestFnCall.type === 'test') {
-          testDepth++;
           if (node.arguments.length < 2 && jestFnCall.members.every(s => (0, _utils.getAccessorValue)(s) !== 'todo')) {
             context.report({
               messageId: 'missingFunction',
@@ -48,43 +38,19 @@ var _default = exports.default = (0, _utils.createRule)({
         // the only jest functions that are with "x" are "xdescribe", "xtest", and "xit"
         jestFnCall.name.startsWith('x') || jestFnCall.members.some(s => (0, _utils.getAccessorValue)(s) === 'skip')) {
           context.report({
-            messageId: jestFnCall.type === 'describe' ? 'disabledSuite' : 'disabledTest',
+            messageId: 'skippedTest',
             node
           });
         }
       },
-      'CallExpression:exit'(node) {
-        const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if (!jestFnCall) {
-          return;
-        }
-        if (jestFnCall.type === 'describe') {
-          suiteDepth--;
-        }
-        if (jestFnCall.type === 'test') {
-          testDepth--;
-        }
-      },
       'CallExpression[callee.name="pending"]'(node) {
         if ((0, _utils.resolveScope)((0, _utils.getScope)(context, node), 'pending')) {
           return;
         }
-        if (testDepth > 0) {
-          context.report({
-            messageId: 'pendingTest',
-            node
-          });
-        } else if (suiteDepth > 0) {
-          context.report({
-            messageId: 'pendingSuite',
-            node
-          });
-        } else {
-          context.report({
-            messageId: 'pending',
-            node
-          });
-        }
+        context.report({
+          messageId: 'skippedTest',
+          node
+        });
       }
     };
   }

package/lib/rules/prefer-lowercase-title.js

@@ -28,7 +28,7 @@ var _default = exports.default = (0, _utils.createRule)({
     },
     fixable: 'code',
     messages: {
-      unexpectedLowercase: '`{{ method }}`s should begin with lowercase'
+      unexpectedCase: '`{{ method }}`s should begin with lowercase'
     },
     schema: [{
       type: 'object',
@@ -94,7 +94,7 @@ var _default = exports.default = (0, _utils.createRule)({
           return;
         }
         context.report({
-          messageId: 'unexpectedLowercase',
+          messageId: 'unexpectedCase',
           node: node.arguments[0],
           data: {
             method: jestFnCall.name

package/package.json

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