test: test case when --enable-source-maps is provided

Author: geeksilva97

lib/internal/test_runner/test.js

@@ -14,7 +14,6 @@ const {
   Number,
   NumberPrototypeToFixed,
   ObjectDefineProperty,
-  ObjectKeys,
   ObjectSeal,
   Promise,
   PromisePrototypeThen,
@@ -561,7 +560,7 @@ class Test extends AsyncResource {
         const map = lazyFindSourceMap(this.loc.file);
         const entry = map?.findEntry(this.loc.line - 1, this.loc.column - 1);
 
-        if (entry !== undefined && ObjectKeys(entry).length > 0) {
+        if (entry?.originalSource !== undefined) {
           this.loc.line = entry.originalLine + 1;
           this.loc.column = entry.originalColumn + 1;
           this.loc.file = entry.originalSource;

test/parallel/test-runner-coverage.js

@@ -289,6 +289,88 @@ test('coverage reports on lines, functions, and branches', skipIfNoInspector, as
   });
 });
 
+test('coverage with source maps', skipIfNoInspector, () => {
+  let report = [
+    '# start of coverage report',
+    '# --------------------------------------------------------------',
+    '# file          | line % | branch % | funcs % | uncovered lines',
+    '# --------------------------------------------------------------',
+    '# a.test.ts     |  53.85 |   100.00 |  100.00 | 8-13',  // part of a bundle
+    '# b.test.ts     |  55.56 |   100.00 |  100.00 | 1 7-9', // part of a bundle
+    '# index.test.js |  71.43 |    66.67 |  100.00 | 6-7',  // no source map
+    '# stdin.test.ts |  57.14 |   100.00 |  100.00 | 4-6',  // Source map without original file
+    '# --------------------------------------------------------------',
+    '# all files     |  58.33 |    87.50 |  100.00 | ',
+    '# --------------------------------------------------------------',
+    '# end of coverage report',
+  ].join('\n');
+
+  if (common.isWindows) {
+    report = report.replaceAll('/', '\\');
+  }
+
+  const fixture = fixtures.path('test-runner', 'coverage');
+  const args = [
+    '--test', '--experimental-test-coverage', '--test-reporter', 'tap',
+  ];
+  const result = spawnSync(process.execPath, args, { cwd: fixture });
+
+  assert.strictEqual(result.stderr.toString(), '');
+  assert(result.stdout.toString().includes(report));
+  assert.strictEqual(result.status, 1);
+});
+
+
+test('coverage with --enable-source-maps option', skipIfNoInspector, () => {
+  const fixture = fixtures.path('test-runner', 'coverage', 'stdin.test.js');
+  const args = [
+    '--test', '--experimental-test-coverage', '--enable-source-maps', '--test-reporter', 'tap',
+    fixture,
+  ];
+
+  let report = [
+    '# start of coverage report',
+    '# ------------------------------------------------------------------',
+    '# file              | line % | branch % | funcs % | uncovered lines',
+    '# ------------------------------------------------------------------',
+    '# test              |        |          |         | ',
+    '#  fixtures         |        |          |         | ',
+    '#   test-runner     |        |          |         | ',
+    '#    coverage       |        |          |         | ',
+    '#     stdin.test.ts |  57.14 |   100.00 |  100.00 | 4-6',
+    '# ------------------------------------------------------------------',
+    '# all files         |  57.14 |   100.00 |  100.00 | ',
+    '# ------------------------------------------------------------------',
+    '# end of coverage report',
+  ].join('\n');
+
+  if (common.isWindows) {
+    report = report.replaceAll('/', '\\');
+  }
+
+  const result = spawnSync(process.execPath, args);
+  const stdOut = result.stdout.toString();
+
+  assert.strictEqual(result.stderr.toString(), '');
+  assert(!stdOut.includes('TypeError'));
+  assert(stdOut.includes(report));
+  assert.strictEqual(result.status, 0);
+});
+
+test('coverage with source maps missing sources', skipIfNoInspector, () => {
+  const file = fixtures.path('test-runner', 'source-map-missing-sources', 'index.js');
+  const missing = fixtures.path('test-runner', 'source-map-missing-sources', 'nonexistent.js');
+  const result = spawnSync(process.execPath, [
+    '--test',
+    '--experimental-test-coverage',
+    file,
+  ]);
+
+  const error = `Cannot find '${pathToFileURL(missing)}' imported from the source map for '${pathToFileURL(file)}'`;
+  assert(result.stdout.toString().includes(error));
+  assert.strictEqual(result.status, 1);
+});
+
 test('coverage with ESM hook - source irrelevant', skipIfNoInspector, () => {
   let report = [
     '# start of coverage report',