Merge pull request #459 from KnisterPeter/prop-types-namespace-import

feat: support any name for namespace import of `prop-types`

Author: KnisterPeter

src/typings.ts

@@ -235,7 +235,7 @@ function getPropTypesName(ast: AstQuery): string|undefined {
       /:source StringLiteral[@value == 'prop-types']
     ]
     /:specifiers *[
-      / Identifier[@name == 'PropTypes']
+      ImportNamespaceSpecifier || / Identifier[@name == 'PropTypes']
     ]
     /:local Identifier
   `);

tests/parsing-test.ts

@@ -105,3 +105,6 @@ test('Parsing should create preact definition', t => {
 test('Parsing should suppport props-types repo', t => {
   compare(t, 'path', 'prop-types.jsx', 'prop-types.d.ts', {});
 });
+test('Parsing should suppport props-types repo (with a default import)', t => {
+  compare(t, 'path', 'prop-types-default-import.jsx', 'prop-types.d.ts', {});
+});

tests/prop-types-default-import.jsx

@@ -0,0 +1,15 @@
+import {Component} from 'react';
+import PropTypes from 'prop-types';
+
+export default class extends Component {
+
+    static propTypes = {
+      optionalString: PropTypes.string
+    };
+
+    render() {
+        return (
+            <div></div>
+        );
+    }
+}

tests/prop-types.jsx

@@ -1,10 +1,10 @@
 import {Component} from 'react';
-import * as PropTypes from 'prop-types';
+import * as P from 'prop-types';
 
 export default class extends Component {
 
     static propTypes = {
-      optionalString: PropTypes.string
+      optionalString: P.string
     };
 
     render() {