fix(lowercase-name): ignore all top level describes when option is tr…

…ue (#614) closes #613
jest-community · Jun 23, 2020 · 624018a · 624018a
1 parent 9262f46
commit 624018a
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/src/rules/__tests__/lowercase-name.test.ts b/src/rules/__tests__/lowercase-name.test.ts
@@ -271,6 +271,22 @@ ruleTester.run('lowercase-name with ignoreTopLevelDescribe', rule, {
       `,
       options: [{ ignoreTopLevelDescribe: true }],
     },
+    {
+      code: dedent`
+        describe('Strings', () => {
+          it('are strings', () => {
+            expect('abc').toBe('abc');
+          });
+        });
+        
+        describe('Booleans', () => {
+          it('are booleans', () => {
+            expect(true).toBe(true);
+          });
+        });
+      `,
+      options: [{ ignoreTopLevelDescribe: true }],
+    },
   ],
   invalid: [
     {

diff --git a/src/rules/lowercase-name.ts b/src/rules/lowercase-name.ts
@@ -158,6 +158,11 @@ export default createRule<
           });
         }
       },
+      'CallExpression:exit'(node: TSESTree.CallExpression) {
+        if (isDescribe(node)) {
+          numberOfDescribeBlocks--;
+        }
+      },
     };
   },
 });