fix(valid-title): check if node has arguments first (#436)

jest-community · Oct 23, 2019 · e961e21 · e961e21
1 parent 7eae625
commit e961e21
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/rules/__tests__/valid-title.test.ts b/src/rules/__tests__/valid-title.test.ts
@@ -9,6 +9,10 @@ const ruleTester = new TSESLint.RuleTester({
 
 ruleTester.run('no-accidental-space', rule, {
   valid: [
+    'it()',
+    'describe()',
+    'it("")',
+    'it.each()()',
     'describe("foo", function () {})',
     'describe(6, function () {})',
     'fdescribe("foo", function () {})',

diff --git a/src/rules/valid-title.ts b/src/rules/valid-title.ts
@@ -39,6 +39,8 @@ export default createRule({
       CallExpression(node) {
         if (!isDescribe(node) && !isTestCase(node)) return;
 
+        if (!node.arguments.length) return;
+
         const title = getNodeTitle(node);
         if (!title) return;