[Fix] no-unused-prop-types: make markPropTypesAsUsed work with `T…

…SEmptyBodyFunctionExpression` AST node Fixes #2559.
jsx-eslint · Feb 1, 2020 · b517b9e · b517b9e
1 parent 66c0d66
commit b517b9e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/util/usedPropTypes.js b/lib/util/usedPropTypes.js
@@ -339,6 +339,8 @@ module.exports = function usedPropTypesInstructions(context, components, utils)
         type = 'destructuring';
         properties = node.properties;
         break;
+      case 'TSEmptyBodyFunctionExpression':
+        break;
       default:
         throw new Error(`${node.type} ASTNodes are not handled by markPropTypesAsUsed`);
     }

diff --git a/tests/lib/rules/no-unused-prop-types.js b/tests/lib/rules/no-unused-prop-types.js
@@ -3202,6 +3202,15 @@ ruleTester.run('no-unused-prop-types', rule, {
         }
       `,
       parser: parsers.TYPESCRIPT_ESLINT
+    },
+    {
+      code: [
+        'declare class Thing {',
+        '  constructor({ id }: { id: string });',
+        '}',
+        'export default Thing;'
+      ].join('\n'),
+      parser: parsers.TYPESCRIPT_ESLINT
     }
   ],