Ensure cached content goes through module._compile() too.

standard-things · Nov 17, 2017 · ae59e73 · ae59e73
1 parent 87e9a9e
commit ae59e73
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 15 deletions.
diff --git a/src/hook/module.js b/src/hook/module.js
@@ -130,11 +130,6 @@ function hook(Module, parent, options) {
       }
     }
 
-    if (isObject(cached)) {
-      tryCompileCached(mod, cached, filePath, runtimeAlias, options)
-      return
-    }
-
     const { _compile } = mod
     const shouldOverwrite = env.cli
     const shouldRestore = shouldOverwrite && has(mod, "_compile")
@@ -148,15 +143,17 @@ function hook(Module, parent, options) {
         }
       }
 
-      cached = tryCompileCode(manager, content, {
-        cacheFileName,
-        cachePath,
-        filePath,
-        hint,
-        pkgInfo,
-        runtimeAlias,
-        type
-      })
+      if (! isObject(cached)) {
+        cached = tryCompileCode(manager, content, {
+          cacheFileName,
+          cachePath,
+          filePath,
+          hint,
+          pkgInfo,
+          runtimeAlias,
+          type
+        })
+      }
 
       if (cached.warnings) {
         for (const warning of cached.warnings) {

diff --git a/test/node_modules/babel-register/index.js b/test/node_modules/babel-register/index.js