fix(no-disabled-tests): support describe.skip.each & `xdescribe.eac…

…h` (#778)
jest-community · Mar 6, 2021 · 6a32e87 · 6a32e87
1 parent 92a8826
commit 6a32e87
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/rules/__tests__/no-disabled-tests.test.ts b/src/rules/__tests__/no-disabled-tests.test.ts
@@ -67,6 +67,14 @@ ruleTester.run('no-disabled-tests', rule, {
       code: 'describe.skip("foo", function () {})',
       errors: [{ messageId: 'skippedTestSuite', column: 1, line: 1 }],
     },
+    {
+      code: 'describe.skip.each([1, 2, 3])("%s", (a, b) => {});',
+      errors: [{ messageId: 'skippedTestSuite', column: 1, line: 1 }],
+    },
+    {
+      code: 'xdescribe.each([1, 2, 3])("%s", (a, b) => {});',
+      errors: [{ messageId: 'skippedTestSuite', column: 1, line: 1 }],
+    },
     {
       code: 'describe[`skip`]("foo", function () {})',
       errors: [{ messageId: 'skippedTestSuite', column: 1, line: 1 }],

diff --git a/src/rules/no-disabled-tests.ts b/src/rules/no-disabled-tests.ts
@@ -45,6 +45,8 @@ export default createRule({
         }
 
         switch (functionName) {
+          case 'describe.skip.each':
+          case 'xdescribe.each':
           case 'describe.skip':
             context.report({ messageId: 'skippedTestSuite', node });
             break;