test: migrate message source map tests from Python to JS · nodejs/node@107015b · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 632
• Pull requests 754
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 107015b

Browse filesBrowse the repository at this point in the historyBrowse files

Yiyun Lei

authored and

targos

committed

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
Copy full SHA for 107015b

20 files changed

+65-55Lines changed: 65 additions & 55 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test
  • fixtures/source-map/output
    • source_map_enclosing_function.js
    • source_map_enclosing_function.snapshot
    • source_map_reference_error_tabs.js
    • source_map_reference_error_tabs.snapshot
    • source_map_sourcemapping_url_string.js
    • source_map_sourcemapping_url_string.snapshot
    • source_map_throw_catch.js
    • source_map_throw_catch.snapshot
    • source_map_throw_icu.js
    • source_map_throw_icu.snapshot
    • source_map_throw_set_immediate.js
    • source_map_throw_set_immediate.snapshot
  • message
    • source_map_enclosing_function.js
    • source_map_reference_error_tabs.out
    • source_map_sourcemapping_url_string.out
    • source_map_throw_catch.out
    • source_map_throw_icu.out
    • source_map_throw_set_immediate.js
    • source_map_throw_set_immediate.out
  • parallel
    • test-node-output-sourcemaps.mjs

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/fixtures/source-map/output/source_map_enclosing_function.js‎

Copy file name to clipboard

+7Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff 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');

Collapse file

‎…essage/source_map_enclosing_function.out‎ ‎…t/source_map_enclosing_function.snapshot‎test/message/source_map_enclosing_function.out renamed to test/fixtures/source-map/output/source_map_enclosing_function.snapshot test/message/source_map_enclosing_function.out renamed to test/fixtures/source-map/output/source_map_enclosing_function.snapshot

Copy file name to clipboardExpand all lines: test/fixtures/source-map/output/source_map_enclosing_function.snapshot

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -2,6 +2,7 @@
2	2	throw err
3	3	^
4	4
	5	+
5	6	Error: an error!
6	7	at functionD (*enclosing-call-site.js:16:17)
7	8	at functionC (*enclosing-call-site.js:10:3)

Collapse file

‎…ssage/source_map_reference_error_tabs.js‎ ‎…utput/source_map_reference_error_tabs.js‎test/message/source_map_reference_error_tabs.js renamed to test/fixtures/source-map/output/source_map_reference_error_tabs.js test/message/source_map_reference_error_tabs.js renamed to test/fixtures/source-map/output/source_map_reference_error_tabs.js

Copy file name to clipboard

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -1,7 +1,7 @@
1	1	// Flags: --enable-source-maps
2	2
3	3	'use strict';
4		- require('../common');
	4	+ require('../../../common');
5	5	Error.stackTraceLimit = 2;
6	6
7		- require('../fixtures/source-map/tabs.js');
	7	+ require('../tabs.js');

Collapse file

‎test/fixtures/source-map/output/source_map_reference_error_tabs.snapshot‎

Copy file name to clipboard

+10Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff 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 *

Collapse file

‎…e/source_map_sourcemapping_url_string.js‎ ‎…t/source_map_sourcemapping_url_string.js‎test/message/source_map_sourcemapping_url_string.js renamed to test/fixtures/source-map/output/source_map_sourcemapping_url_string.js test/message/source_map_sourcemapping_url_string.js renamed to test/fixtures/source-map/output/source_map_sourcemapping_url_string.js

Copy file name to clipboardExpand all lines: test/fixtures/source-map/output/source_map_sourcemapping_url_string.js

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -1,11 +1,11 @@
1	1	// Flags: --enable-source-maps
2	2
3	3	'use strict';
4		- require('../common');
	4	+ require('../../../common');
5	5	Error.stackTraceLimit = 2;
6	6
7	7	try {
8		- require('../fixtures/source-map/typescript-sourcemapping_url_string');
	8	+ require('../typescript-sourcemapping_url_string');
9	9	} catch (err) {
10	10	setTimeout(() => {
11	11	console.info(err);

Collapse file

‎test/fixtures/source-map/output/source_map_sourcemapping_url_string.snapshot‎

Copy file name to clipboard

+3Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -0,0 +1,3 @@
	1	+ Error: an exception.
	2	+ at Object.<anonymous> (*typescript-sourcemapping_url_string.ts:3:7)
	3	+ *

Collapse file

‎test/message/source_map_throw_catch.js‎ ‎…rce-map/output/source_map_throw_catch.js‎test/message/source_map_throw_catch.js renamed to test/fixtures/source-map/output/source_map_throw_catch.js test/message/source_map_throw_catch.js renamed to test/fixtures/source-map/output/source_map_throw_catch.js

Copy file name to clipboardExpand all lines: test/fixtures/source-map/output/source_map_throw_catch.js

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -1,11 +1,11 @@
1	1	// Flags: --enable-source-maps
2	2
3	3	'use strict';
4		- require('../common');
	4	+ require('../../../common');
5	5	Error.stackTraceLimit = 2;
6	6
7	7	try {
8		- require('../fixtures/source-map/typescript-throw');
	8	+ require('../typescript-throw');
9	9	} catch (err) {
10	10	setTimeout(() => {
11	11	console.info(err);

Collapse file

‎test/fixtures/source-map/output/source_map_throw_catch.snapshot‎

Copy file name to clipboard

+4Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff 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)

Collapse file

‎test/message/source_map_throw_icu.js‎ ‎…ource-map/output/source_map_throw_icu.js‎test/message/source_map_throw_icu.js renamed to test/fixtures/source-map/output/source_map_throw_icu.js test/message/source_map_throw_icu.js renamed to test/fixtures/source-map/output/source_map_throw_icu.js

Copy file name to clipboard

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -1,7 +1,7 @@
1	1	// Flags: --enable-source-maps
2	2
3	3	'use strict';
4		- require('../common');
	4	+ require('../../../common');
5	5	Error.stackTraceLimit = 2;
6	6
7		- require('../fixtures/source-map/icu');
	7	+ require('../icu');

Collapse file

‎test/fixtures/source-map/output/source_map_throw_icu.snapshot‎

Copy file name to clipboard

+10Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff 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 *

0 commit comments

Comments

0 (0)