Switch default importer to ignoreImports, fix type sigs w/ babel options, update tests

Author: phated

src/__tests__/main-test.js

@@ -8,7 +8,7 @@
 
 import fs from 'fs';
 import path from 'path';
-import { handlers, parse } from '../main';
+import { handlers, parse, importers } from '../main';
 import { ERROR_MISSING_DEFINITION } from '../parse';
 
 describe('main', () => {
@@ -216,6 +216,7 @@ describe('main', () => {
     });
   });
 
+  // Fixures uses the resolveImports importer even though it is not the default
   describe('fixtures', () => {
     const fixturePath = path.join(__dirname, 'fixtures');
     const fileNames = fs.readdirSync(fixturePath);
@@ -227,6 +228,7 @@ describe('main', () => {
         let result;
         expect(() => {
           result = parse(fileContent, null, null, {
+            importer: importers.resolveImports,
             filename: filePath,
             babelrc: false,
           });

src/babelParser.js

@@ -66,8 +66,9 @@ type BabelOptions = {
   babelrcRoots?: true | string | string[],
 };
 
-export type Options = BabelOptions & {
-  parserOptions?: ParserOptions,
+export type Options = {
+  ...$Exact<BabelOptions>,
+  ...{| parserOptions?: ParserOptions |},
 };
 
 function buildOptions(

src/main.js

@@ -30,7 +30,7 @@ const defaultHandlers = [
   allHandlers.componentMethodsHandler,
   allHandlers.componentMethodsJsDocHandler,
 ];
-const defaultImporter = AllImporter.resolveImports;
+const defaultImporter = AllImporter.ignoreImports;
 
 /**
  * See `lib/parse.js` for more information about the arguments. This function
@@ -58,9 +58,9 @@ function defaultParse(
     handlers = defaultHandlers;
   }
 
-  const { importer = defaultImporter } = options;
+  const { importer = defaultImporter, ...opts } = options;
 
-  return parse(String(src), resolver, handlers, importer, options);
+  return parse(String(src), resolver, handlers, importer, opts);
 }
 
 export {