Fix renameTo renaming React component prop name unexpectedly (#220)

See: #219

Author: baopham

src/collections/VariableDeclarator.js

@@ -114,6 +114,15 @@ const transformMethods = {
             return false;
           }
 
+          if (
+            types.JSXAttribute.check(parent) &&
+            parent.name === path.node &&
+            !parent.computed
+          ) {
+            // <Foo oldName={oldName} />
+            return false;
+          }
+
           return true;
         })
         .forEach(function(path) {

src/collections/__tests__/VariableDeclarator-test.js

@@ -53,6 +53,7 @@ describe('VariableDeclarators', function() {
       '    blah() {}',
       '  }',
       '}',
+      '<Component foo={foo} />',
     ].join('\n'), {parser: getParser()}).program];
   });
 
@@ -127,6 +128,17 @@ describe('VariableDeclarators', function() {
 
       expect(identifiers.length).toBe(1);
     });
+
+    it('does not rename React component prop name', function () {
+      const declarators = Collection.fromNodes(nodes)
+        .findVariableDeclarators('foo')
+        .renameTo('xyz');
+
+      const identifiers = Collection.fromNodes(nodes)
+        .find(types.JSXIdentifier, { name: 'foo' });
+
+      expect(identifiers.length).toBe(1);
+    });
   });
 
 });