flow: Ignore import statements when using 'import type'

Fixes #931

src/rules/no-extraneous-dependencies.js

@@ -55,9 +55,15 @@ function optDepErrorMessage(packageName) {
 }
 
 function reportIfMissing(context, deps, depsOptions, node, name) {
+  // Do not report when importing types
+  if (node.importKind === 'type') {
+    return
+  }
+
   if (importType(name, context) !== 'external') {
     return
   }
+
   const resolved = resolve(name, context)
   if (!resolved) {
     return

tests/files/with-flow-typed/flow-typed/npm/myflowtyped_v1.x.x.js

@@ -0,0 +1,4 @@
+// flow-typed signature: ____
+// flow-typed version: ____/myflowtyped_v1.x.x/flow_>=v0.33.x
+
+declare module 'myflowtyped' {}

tests/files/with-flow-typed/package.json

@@ -0,0 +1,3 @@
+{
+  "dependencies": {}
+}

tests/src/rules/no-extraneous-dependencies.js

@@ -14,6 +14,7 @@ const packageFileWithSyntaxErrorMessage = (() => {
     return error.message
   }
 })()
+const packageDirWithFlowTyped = path.join(__dirname, '../../files/with-flow-typed')
 
 ruleTester.run('no-extraneous-dependencies', rule, {
   valid: [
@@ -69,6 +70,11 @@ ruleTester.run('no-extraneous-dependencies', rule, {
       code: 'import "doctrine"',
       options: [{packageDir: path.join(__dirname, '../../../')}],
     }),
+    test({
+      code: 'import type MyType from "myflowtyped";',
+      options: [{packageDir: packageDirWithFlowTyped}],
+      parser: 'babel-eslint',
+    }),
   ],
   invalid: [
     test({