Merge pull request #1600 from kevinzwhuang/fix-unused-prop-types

Add empty params check for unused prop types rule to fix empty proptype functions from causing crashes

Author: ljharb

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()) {

tests/lib/rules/no-unused-prop-types.js

@@ -2440,6 +2440,46 @@ ruleTester.run('no-unused-prop-types', rule, {
       }
       `,
       parser: 'babel-eslint'
+    },
+    {
+      code: `
+        class MyComponent extends React.Component {
+          render() {
+            return <div>{ this.props.other }</div>
+          }
+        }
+        MyComponent.propTypes = { other: () => {} };
+      `
+    },
+    {
+      code: `
+        class MyComponent extends React.Component {
+          render() {
+            return <div>{ this.props.other }</div>
+          }
+        }
+        MyComponent.propTypes = { other() {} };
+      `
+    },
+    {
+      code: `
+        class MyComponent extends React.Component {
+          render() {
+            return <div>{ this.props.other }</div>
+          }
+        }
+        MyComponent.propTypes = { other: function () {} };
+      `
+    },
+    {
+      code: `
+        class MyComponent extends React.Component {
+          render() {
+            return <div>{ this.props.other }</div>
+          }
+        }
+        MyComponent.propTypes = { * other() {} };
+      `
     }
   ],