Skip to content

Commit a87c8c2

Browse files
authored
fix(prefer-todo): ensure argument exists before trying to access it (#462)
1 parent b1b6dca commit a87c8c2

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/rules/__tests__/prefer-todo.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ const ruleTester = new TSESLint.RuleTester({
99

1010
ruleTester.run('prefer-todo', rule, {
1111
valid: [
12+
'test()',
1213
'test.todo("i need to write this test");',
1314
'test(obj)',
1415
'fit("foo")',

src/rules/prefer-todo.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export default createRule({
7676
CallExpression(node) {
7777
const [firstArg, secondArg] = node.arguments;
7878

79-
if (!isTargetedTestCase(node) || !isStringNode(firstArg)) {
79+
if (!firstArg || !isTargetedTestCase(node) || !isStringNode(firstArg)) {
8080
return;
8181
}
8282

0 commit comments

Comments
 (0)