Skip to content

Commit 0f7e3f0

Browse files
geeksilva97aduh95
authored andcommitted
test_runner: assert entry is a valid object
PR-URL: #55231 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
1 parent f0b4412 commit 0f7e3f0

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

lib/internal/test_runner/test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -560,7 +560,7 @@ class Test extends AsyncResource {
560560
const map = lazyFindSourceMap(this.loc.file);
561561
const entry = map?.findEntry(this.loc.line - 1, this.loc.column - 1);
562562

563-
if (entry !== undefined) {
563+
if (entry?.originalSource !== undefined) {
564564
this.loc.line = entry.originalLine + 1;
565565
this.loc.column = entry.originalColumn + 1;
566566
this.loc.file = entry.originalSource;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
'use strict';
2+
require('../common');
3+
const assert = require('node:assert');
4+
const { spawnSync } = require('node:child_process');
5+
const { test } = require('node:test');
6+
const fixtures = require('../common/fixtures');
7+
8+
test('ensures --enable-source-maps does not throw an error', () => {
9+
const fixture = fixtures.path('test-runner', 'coverage', 'stdin.test.js');
10+
const args = ['--enable-source-maps', fixture];
11+
12+
const result = spawnSync(process.execPath, args);
13+
14+
assert.strictEqual(result.stderr.toString(), '');
15+
assert.strictEqual(result.status, 0);
16+
});

0 commit comments

Comments
 (0)