Skip to content

Commit 107015b

Browse files
Yiyun Leitargos
Yiyun Lei
authored andcommitted
test: migrate message source map tests from Python to JS
Migrate the remaining source map tests in the `test/message` folder from Python to JS. Fixes: #47707 test: migrate message source map tests from Python to JS PR-URL: #49238 Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
1 parent b6522a2 commit 107015b

20 files changed

+65
-55
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
// Flags: --enable-source-maps
2+
3+
'use strict';
4+
require('../../../common');
5+
Error.stackTraceLimit = 5;
6+
7+
require('../enclosing-call-site-min.js');

test/message/source_map_enclosing_function.out test/fixtures/source-map/output/source_map_enclosing_function.snapshot

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
throw err
33
^
44

5+
56
Error: an error!
67
at functionD (*enclosing-call-site.js:16:17)
78
at functionC (*enclosing-call-site.js:10:3)
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Flags: --enable-source-maps
22

33
'use strict';
4-
require('../common');
4+
require('../../../common');
55
Error.stackTraceLimit = 2;
66

7-
require('../fixtures/source-map/tabs.js');
7+
require('../tabs.js');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
*tabs.coffee:26
2+
alert "I knew it!"
3+
^
4+
5+
6+
ReferenceError: alert is not defined
7+
at Object.<anonymous> (*tabs.coffee:26:2)
8+
at Object.<anonymous> (*tabs.coffee:1:14)
9+
10+
Node.js *

test/message/source_map_sourcemapping_url_string.js test/fixtures/source-map/output/source_map_sourcemapping_url_string.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
// Flags: --enable-source-maps
22

33
'use strict';
4-
require('../common');
4+
require('../../../common');
55
Error.stackTraceLimit = 2;
66

77
try {
8-
require('../fixtures/source-map/typescript-sourcemapping_url_string');
8+
require('../typescript-sourcemapping_url_string');
99
} catch (err) {
1010
setTimeout(() => {
1111
console.info(err);
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
Error: an exception.
2+
at Object.<anonymous> (*typescript-sourcemapping_url_string.ts:3:7)
3+
*

test/message/source_map_throw_catch.js test/fixtures/source-map/output/source_map_throw_catch.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
// Flags: --enable-source-maps
22

33
'use strict';
4-
require('../common');
4+
require('../../../common');
55
Error.stackTraceLimit = 2;
66

77
try {
8-
require('../fixtures/source-map/typescript-throw');
8+
require('../typescript-throw');
99
} catch (err) {
1010
setTimeout(() => {
1111
console.info(err);
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
reachable
2+
Error: an exception
3+
at branch (*typescript-throw.ts:18:11)
4+
at Object.<anonymous> (*typescript-throw.ts:24:1)
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Flags: --enable-source-maps
22

33
'use strict';
4-
require('../common');
4+
require('../../../common');
55
Error.stackTraceLimit = 2;
66

7-
require('../fixtures/source-map/icu');
7+
require('../icu');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
*icu.jsx:3
2+
("あ 🐕 🐕", throw Error("an error"));
3+
^
4+
5+
6+
Error: an error
7+
at Object.createElement (*icu.jsx:3:23)
8+
at Object.<anonymous> (*icu.jsx:9:5)
9+
10+
Node.js *
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
// Flags: --enable-source-maps
2+
3+
'use strict';
4+
require('../../../common');
5+
require('../uglify-throw');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
*uglify-throw-original.js:5
2+
throw Error('goodbye');
3+
^
4+
5+
6+
Error: goodbye
7+
at Hello (*uglify-throw-original.js:5:9)
8+
at Immediate.<anonymous> (*uglify-throw-original.js:9:3)
9+
at process.processImmediate (node:internal*timers:476:21)
10+
11+
Node.js *

test/message/source_map_enclosing_function.js

-7
This file was deleted.

test/message/source_map_reference_error_tabs.out

-9
This file was deleted.

test/message/source_map_sourcemapping_url_string.out

-3
This file was deleted.

test/message/source_map_throw_catch.out

-4
This file was deleted.

test/message/source_map_throw_icu.out

-9
This file was deleted.

test/message/source_map_throw_set_immediate.js

-5
This file was deleted.

test/message/source_map_throw_set_immediate.out

-10
This file was deleted.

test/parallel/test-node-output-sourcemaps.mjs

+6
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,15 @@ describe('sourcemaps output', { concurrency: true }, () => {
2828
const tests = [
2929
{ name: 'source-map/output/source_map_disabled_by_api.js' },
3030
{ name: 'source-map/output/source_map_enabled_by_api.js' },
31+
{ name: 'source-map/output/source_map_enclosing_function.js' },
3132
{ name: 'source-map/output/source_map_eval.js' },
3233
{ name: 'source-map/output/source_map_no_source_file.js' },
34+
{ name: 'source-map/output/source_map_reference_error_tabs.js' },
35+
{ name: 'source-map/output/source_map_sourcemapping_url_string.js' },
36+
{ name: 'source-map/output/source_map_throw_catch.js' },
3337
{ name: 'source-map/output/source_map_throw_first_tick.js' },
38+
{ name: 'source-map/output/source_map_throw_icu.js' },
39+
{ name: 'source-map/output/source_map_throw_set_immediate.js' },
3440
];
3541
for (const { name, transform } of tests) {
3642
it(name, async () => {

0 commit comments

Comments
 (0)