test: add test case for throwing an exception with vm.Script · nodejs/node@f060881 · GitHub

nodejs / node Public

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

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

Additional navigation options

Commit f060881

Browse filesBrowse the repository at this point in the historyBrowse files

Leko

authored and

MylesBorins

committed

test: add test case for throwing an exception with vm.Script

It failed with `FATAL ERROR: v8::ToLocalChecked Empty MaybeLocal`. PR-URL: #17343 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>

1 parent 419cde7 commit f060881
Copy full SHA for f060881

1 file changed

+14Lines changed: 14 additions & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-vm-script-throw-in-tostring.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-vm-script-throw-in-tostring.js‎

Copy file name to clipboard

+14Lines changed: 14 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -0,0 +1,14 @@
	1	+ 'use strict';
	2	+
	3	+ require('../common');
	4	+ const assert = require('assert');
	5	+
	6	+ const vm = require('vm');
	7	+
	8	+ assert.throws(() => {
	9	+ new vm.Script({
	10	+ toString() {
	11	+ throw new Error();
	12	+ }
	13	+ });
	14	+ }, Error);

0 commit comments

Comments

0 (0)