fix(prefer-todo): ensure argument exists before trying to access it (#…

…462)
jest-community · Oct 28, 2019 · a87c8c2 · a87c8c2
1 parent b1b6dca
commit a87c8c2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/rules/__tests__/prefer-todo.test.ts b/src/rules/__tests__/prefer-todo.test.ts
@@ -9,6 +9,7 @@ const ruleTester = new TSESLint.RuleTester({
 
 ruleTester.run('prefer-todo', rule, {
   valid: [
+    'test()',
     'test.todo("i need to write this test");',
     'test(obj)',
     'fit("foo")',

diff --git a/src/rules/prefer-todo.ts b/src/rules/prefer-todo.ts
@@ -76,7 +76,7 @@ export default createRule({
       CallExpression(node) {
         const [firstArg, secondArg] = node.arguments;
 
-        if (!isTargetedTestCase(node) || !isStringNode(firstArg)) {
+        if (!firstArg || !isTargetedTestCase(node) || !isStringNode(firstArg)) {
           return;
         }