Improve source map handling when instrumenting transformed code (#5739)

e2e/__tests__/__snapshots__/stackTraceSourceMapsWithCoverage.test.ts.snap

@@ -0,0 +1,27 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`processes stack traces and code frames with source maps with coverage 1`] = `
+Object {
+  "rest": "FAIL __tests__/fails.ts
+  ✕ fails
+
+  ● fails
+
+    This did not work!
+
+      11 | 
+      12 | export function error() {
+    > 13 |   throw new Error('This did not work!');
+         |         ^
+      14 | }
+      15 | 
+
+      at Object.error (lib.ts:13:9)
+      at Object.<anonymous> (__tests__/fails.ts:10:3)",
+  "summary": "Test Suites: 1 failed, 1 total
+Tests:       1 failed, 1 total
+Snapshots:   0 total
+Time:        <<REPLACED>>
+Ran all test suites.",
+}
+`;

e2e/__tests__/stackTraceSourceMapsWithCoverage.test.ts

@@ -5,7 +5,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 import * as path from 'path';
-import {run} from '../Utils';
+import {extractSummary, run} from '../Utils';
 import runJest from '../runJest';
 
 it('processes stack traces and code frames with source maps with coverage', () => {
@@ -15,9 +15,5 @@ it('processes stack traces and code frames with source maps with coverage', () =
   );
   run('yarn', dir);
   const {stderr} = runJest(dir, ['--no-cache', '--coverage']);
-
-  // Should report an error at source line 13 in lib.ts at line 10 of the test
-  expect(stderr).toMatch("13 |   throw new Error('This did not work!');\n");
-  expect(stderr).toMatch(`at Object.error (lib.ts:13:9)
-      at Object.<anonymous> (__tests__/fails.ts:10:3)`);
+  expect(extractSummary(stderr)).toMatchSnapshot();
 });

packages/jest-runtime/src/index.ts

@@ -140,7 +140,6 @@ class Runtime {
   private _isolatedModuleRegistry: ModuleRegistry | null;
   private _moduleRegistry: ModuleRegistry;
   private _esmoduleRegistry: Map<string, Promise<VMModule>>;
-  private _needsCoverageMapped: Set<string>;
   private _resolver: Resolver;
   private _shouldAutoMock: boolean;
   private _shouldMockModuleCache: BooleanObject;