Ignore private class properties (#347)

Author: danez

src/utils/__tests__/getClassMemberValuePath-test.js

@@ -92,4 +92,29 @@ describe('getClassMemberValuePath', () => {
       );
     });
   });
+
+  describe('PrivateClassProperty', () => {
+    it('ignores private class properties', () => {
+      const def = statement(`
+        class Foo {
+          #foo = 42;
+        }
+      `);
+
+      expect(getClassMemberValuePath(def, 'foo')).toBe(undefined);
+    });
+
+    it('finds "normal" class properties with private present', () => {
+      const def = statement(`
+        class Foo {
+          #private = 54;
+          foo = 42;
+        }
+      `);
+
+      expect(getClassMemberValuePath(def, 'foo')).toBe(
+        def.get('body', 'body', 1, 'value'),
+      );
+    });
+  });
 });

src/utils/getClassMemberValuePath.js

@@ -27,6 +27,7 @@ export default function getClassMemberValuePath(
       memberPath =>
         (!memberPath.node.computed ||
           types.Literal.check(memberPath.node.key)) &&
+        !types.PrivateName.check(memberPath.node.key) &&
         getNameOrValue(memberPath.get('key')) === memberName &&
         memberPath.node.kind !== 'set',
     )