Add empty params check for unused prop types rule to fix empty propty…

…pe functions from causing crashes
jsx-eslint · Dec 13, 2017 · d331af6 · d331af6
1 parent f6e4c89
commit d331af6
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/lib/rules/no-unused-prop-types.js b/lib/rules/no-unused-prop-types.js
@@ -652,6 +652,9 @@ module.exports = {
         case 'ArrowFunctionExpression':
         case 'FunctionDeclaration':
         case 'FunctionExpression':
+          if (node.params.length === 0) {
+            break;
+          }
           type = 'destructuring';
           properties = node.params[0].properties;
           if (inSetStateUpdater()) {

diff --git a/tests/lib/rules/no-unused-prop-types.js b/tests/lib/rules/no-unused-prop-types.js
@@ -2440,6 +2440,17 @@ ruleTester.run('no-unused-prop-types', rule, {
       }
       `,
       parser: 'babel-eslint'
+    },
+    {
+      code: `
+        class MyComponent extends React.Component<Props> {
+          render() {
+            return <div>{ this.props.other }</div>
+          }
+        }
+        MyComponent.propTypes = { other: () => {} };
+      `,
+      parser: 'babel-eslint'
     }
   ],