fixed bugs (#266)

Author: lunaruan

transforms/__testfixtures__/update-react-imports/react-type-default-export.input.js

@@ -0,0 +1,4 @@
+import type React from "react";
+import * as React from "react";
+
+<div>Hi</div>;

transforms/__testfixtures__/update-react-imports/react-type-default-export.output.js

@@ -0,0 +1,3 @@
+import type React from "react";
+
+<div>Hi</div>;

transforms/__tests__/update-react-imports-test.js

@@ -18,6 +18,7 @@ const tests = [
   'react-basic-default-export-jsx-element-react-variable',
   'react-basic-default-export-jsx-element',
   'react-basic-default-export',
+  'react-type-default-export',
 ];
 
 jest.mock('../update-react-imports', () => {

transforms/update-react-imports.js

@@ -27,14 +27,17 @@ module.exports = function(file, api, options) {
     .filter(path => {
       return (
         path.value.specifiers.length > 0 &&
+        path.value.importKind === 'value' &&
         path.value.specifiers.some(
           specifier => specifier.local.name === 'React',
         )
       );
     });
 
   if (reactImportPath.size() > 1) {
-    throw Error('There should only be one React import. Please remove the duplicate import and try again.');
+    throw Error(
+      'There should only be one React import. Please remove the duplicate import and try again.',
+    );
   }
 
   if (reactImportPath.size() === 0) {
@@ -95,6 +98,9 @@ module.exports = function(file, api, options) {
         } else {
           j(reactPath).remove();
         }
+      } else {
+        // nothing is transformed so we can return
+        return null;
       }
     }
     break;