Removing the mapCoverage condition on reading inlineSourceMaps. Undoi…

…ng fixes done in #5117
jestjs · Dec 25, 2017 · 364d956 · 364d956
1 parent 4f8f6fb
commit 364d956
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/packages/jest-runtime/src/__tests__/script_transformer.test.js b/packages/jest-runtime/src/__tests__/script_transformer.test.js
@@ -357,7 +357,7 @@ describe('ScriptTransformer', () => {
     );
   });
 
-  it.skip('writes source maps if given by the transformer', () => {
+  it('writes source maps if given by the transformer', () => {
     config = Object.assign(config, {
       transform: [['^.+\\.js$', 'preprocessor-with-sourcemaps']],
     });

diff --git a/packages/jest-runtime/src/script_transformer.js b/packages/jest-runtime/src/script_transformer.js
@@ -242,12 +242,10 @@ export default class ScriptTransformer {
       }
     }
 
-    if (mapCoverage) {
-      if (!transformed.map) {
-        const inlineSourceMap = convertSourceMap.fromSource(transformed.code);
-        if (inlineSourceMap) {
-          transformed.map = inlineSourceMap.toJSON();
-        }
+    if (!transformed.map) {
+      const inlineSourceMap = convertSourceMap.fromSource(transformed.code);
+      if (inlineSourceMap) {
+        transformed.map = inlineSourceMap.toJSON();
       }
     }
 
@@ -261,7 +259,7 @@ export default class ScriptTransformer {
       code = transformed.code;
     }
 
-    if (instrument && mapCoverage && transformed.map) {
+    if (instrument && transformed.map) {
       const sourceMapContent =
         typeof transformed.map === 'string'
           ? transformed.map