Remove the concept of using Jest's stored sourceMaps to map BabelCove…

…rage - inputSourceMap should always be available.
jestjs · Apr 18, 2020 · f1f3448 · f1f3448
1 parent 965c4b4
commit f1f3448
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 16 deletions.
diff --git a/packages/jest-reporters/src/coverage_reporter.ts b/packages/jest-reporters/src/coverage_reporter.ts
@@ -215,13 +215,6 @@ export default class CoverageReporter extends BaseReporter {
               ]);
             } else {
               this._coverageMap.addFileCoverage(result.coverage);
-
-              if (result.sourceMapPath) {
-                this._sourceMapStore.registerURL(
-                  filename,
-                  result.sourceMapPath,
-                );
-              }
             }
           }
         } catch (error) {

diff --git a/packages/jest-reporters/src/generateEmptyCoverage.ts b/packages/jest-reporters/src/generateEmptyCoverage.ts
@@ -18,7 +18,6 @@ export type CoverageWorkerResult =
   | {
       kind: 'BabelCoverage';
       coverage: FileCoverage;
-      sourceMapPath?: string | null;
     }
   | {
       kind: 'V8Coverage';
@@ -66,17 +65,18 @@ export default function (
     }
 
     // Transform file with instrumentation to make sure initial coverage data is well mapped to original code.
-    const {code, mapCoverage, sourceMapPath} = new ScriptTransformer(
-      config,
-    ).transformSource(filename, source, true);
+    const {code} = new ScriptTransformer(config).transformSource(
+      filename,
+      source,
+      true,
+    );
     // TODO: consider passing AST
     const extracted = readInitialCoverage(code);
     // Check extracted initial coverage is not null, this can happen when using /* istanbul ignore file */
     if (extracted) {
       coverageWorkerResult = {
         coverage: createFileCoverage(extracted.coverageData),
         kind: 'BabelCoverage',
-        sourceMapPath: mapCoverage ? sourceMapPath : null,
       };
     }
   }

diff --git a/packages/jest-transform/src/ScriptTransformer.ts b/packages/jest-transform/src/ScriptTransformer.ts
@@ -296,7 +296,6 @@ export default class ScriptTransformer {
       // two separate processes write concurrently to the same cache files.
       return {
         code,
-        mapCoverage: false,
         originalCode: content,
         sourceMapPath,
       };
@@ -385,7 +384,6 @@ export default class ScriptTransformer {
 
     return {
       code,
-      mapCoverage: false,
       originalCode: content,
       sourceMapPath,
     };
@@ -431,7 +429,6 @@ export default class ScriptTransformer {
 
       return {
         code,
-        mapCoverage: false,
         originalCode: content,
         sourceMapPath,
       };

diff --git a/packages/jest-types/src/Transform.ts b/packages/jest-types/src/Transform.ts
@@ -9,6 +9,5 @@
 export type TransformResult = {
   code: string;
   originalCode: string;
-  mapCoverage: boolean;
   sourceMapPath: string | null;
 };