fix: fix error with other cases (#412)

gajus · Jun 13, 2019 · dd10c3f · dd10c3f
1 parent e8d640c
commit dd10c3f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/rules/useFlowType.js b/src/rules/useFlowType.js
@@ -36,8 +36,14 @@ const create = (context) => {
     TypeParameterDeclaration (node) {
       node.params.forEach((param) => {
         if (param.default && param.default.typeParameters) {
+          if (param.default.type === 'GenericTypeAnnotation') {
+            markTypeAsUsedWithGenericType(param.default);
+          }
+
           param.default.typeParameters.params.forEach((typeParameterNode) => {
-            markTypeAsUsedWithGenericType(typeParameterNode);
+            if (typeParameterNode.type === 'GenericTypeAnnotation') {
+              markTypeAsUsedWithGenericType(typeParameterNode);
+            }
           });
         }
       });

diff --git a/tests/rules/assertions/useFlowType.js b/tests/rules/assertions/useFlowType.js
@@ -40,6 +40,12 @@ const VALID_WITH_USE_FLOW_TYPE = [
     errors: [
       '\'A\' is defined but never used.'
     ]
+  },
+  {
+    code: 'import type A from "a"; type X<B = A<string>> = { b: B }; let x: X; console.log(x);',
+    errors: [
+      '\'A\' is defined but never used.'
+    ]
   }
 ];