Fix prefer-class-properties to ignore constructor overload declarations

.changeset/healthy-llamas-mix.md

@@ -0,0 +1,5 @@
+---
+'@shopify/eslint-plugin': patch
+---
+
+Fix prefer-class-properties to ignore empty constructor overload declarations

packages/eslint-plugin/lib/rules/prefer-class-properties.js

@@ -75,7 +75,8 @@ module.exports = {
       return classNode.body.body.find((propertyNode) => {
         return (
           propertyNode.type === 'MethodDefinition' &&
-          propertyNode.key.name === 'constructor'
+          propertyNode.key.name === 'constructor' &&
+          propertyNode.value.body
         );
       });
     }

packages/eslint-plugin/tests/lib/rules/prefer-class-properties.test.js

@@ -3,7 +3,7 @@ const {RuleTester} = require('eslint');
 const rule = require('../../../lib/rules/prefer-class-properties');
 
 const ruleTester = new RuleTester({
-  parser: require.resolve('@babel/eslint-parser'),
+  parser: require.resolve('@typescript-eslint/parser'),
   parserOptions: {
     ecmaVersion: 6,
   },
@@ -127,6 +127,15 @@ ruleTester.run('prefer-class-properties', rule, {
       }`,
       options: ['always'],
     },
+    {
+      code: `class Foo {
+        constructor();
+        constructor() {
+          this.foo = {[foo]: 123};
+        }
+      }`,
+      options: ['always'],
+    },
   ],
   invalid: [
     {
@@ -243,5 +252,15 @@ ruleTester.run('prefer-class-properties', rule, {
       errors: assignErrors,
       options: ['always'],
     },
+    {
+      code: `class Foo {
+        constructor();
+        constructor(a: number) {
+          this['foo'] = 123;
+        }
+      }`,
+      errors: assignErrors,
+      options: ['always'],
+    },
   ],
 });