fix: Fix the double plugin bug (#146)

tleunen · Mar 30, 2017 · 4e19188 · 4e19188
1 parent ed54855
commit 4e19188
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 8 deletions.
diff --git a/src/getRealPath.js b/src/getRealPath.js
@@ -98,7 +98,7 @@ export default function getRealPath(sourcePath, currentFile, opts) {
 
   const { cwd, extensions, pluginOpts } = opts;
   const rootDirs = pluginOpts.root || [];
-  const regExps = pluginOpts.regExps || [];
+  const regExps = pluginOpts.regExps;
   const alias = pluginOpts.alias || {};
 
   const sourceFileFromRoot = getRealPathFromRootConfig(

diff --git a/src/index.js b/src/index.js
@@ -82,15 +82,11 @@ export default ({ types: t }) => {
   };
 
   return {
-    manipulateOptions(babelOptions) {
-      let findPluginOptions = babelOptions.plugins.find(plugin => plugin[0] === this)[1];
-      findPluginOptions = manipulatePluginOptions(findPluginOptions);
+    pre(file) {
+      manipulatePluginOptions(this.opts);
 
-      this.customCWD = findPluginOptions.cwd;
-    },
+      let customCWD = this.opts.cwd;
 
-    pre(file) {
-      let { customCWD } = this.plugin;
       if (customCWD === 'babelrc') {
         const startPath = (file.opts.filename === 'unknown')
           ? './'

diff --git a/test/index.test.js b/test/index.test.js
@@ -435,6 +435,27 @@ describe('module-resolver', () => {
         );
       });
     });
+
+    describe('with the plugin applied twice', () => {
+      const doubleAliasTransformerOpts = {
+        plugins: [
+          plugin,
+          [plugin, {
+            alias: {
+              '^@namespace/foo-(.+)': 'packages/\\1',
+            },
+          }],
+        ],
+      };
+
+      describe('should support replacing parts of a path', () => {
+        testRequireImport(
+          '@namespace/foo-bar',
+          'packages/bar',
+          doubleAliasTransformerOpts,
+        );
+      });
+    });
   });
 
   describe('with custom cwd', () => {