React-PropTypes-to-prop-types: ignore migrated named import (#188)

Author: simison

transforms/React-PropTypes-to-prop-types.js

@@ -220,7 +220,8 @@ module.exports = function(file, api, options) {
       .find(j.Identifier)
       .filter(path => (
         path.node.name === 'PropTypes' &&
-        path.parent.node.type === 'ImportSpecifier'
+        path.parent.node.type === 'ImportSpecifier' &&
+        path.parent.parent.node.source.value === 'react'
       ))
       .forEach(path => {
         hasModifications = true;

transforms/__testfixtures__/React-PropTypes-to-prop-types/already-migrated-named-as-import.input.js

@@ -0,0 +1,5 @@
+import { PropTypes as PT } from 'prop-types';
+import React from 'react';
+
+const object = PropTypes.object;
+const string = React.PropTypes.string;

transforms/__testfixtures__/React-PropTypes-to-prop-types/already-migrated-named-as-import.output.js

@@ -0,0 +1,5 @@
+import { PropTypes as PT } from 'prop-types';
+import React from 'react';
+
+const object = PropTypes.object;
+const string = PropTypes.string;

transforms/__testfixtures__/React-PropTypes-to-prop-types/already-migrated-named-import.input.js

@@ -0,0 +1,5 @@
+import { PropTypes } from 'prop-types';
+import React from 'react';
+
+const object = PropTypes.object;
+const string = React.PropTypes.string;

transforms/__testfixtures__/React-PropTypes-to-prop-types/already-migrated-named-import.output.js

@@ -0,0 +1,5 @@
+import { PropTypes } from 'prop-types';
+import React from 'react';
+
+const object = PropTypes.object;
+const string = PropTypes.string;

transforms/__tests__/React-PropTypes-to-prop-types-test.js

@@ -17,6 +17,8 @@ jest.mock('../React-PropTypes-to-prop-types', () => {
 });
 
 const tests = [
+  'already-migrated-named-as-import',
+  'already-migrated-named-import',
   'already-migrated-import',
   'already-migrated-require',
   'assigned-from-react-var',