test: add test case for process.dlopen with undefined · nodejs/node@8336e4f · 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 8336e4f

Browse filesBrowse the repository at this point in the historyBrowse files

Leko

authored and

MylesBorins

committed

test: add test case for process.dlopen with undefined

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 f060881 commit 8336e4f
Copy full SHA for 8336e4f

1 file changed

+10Lines changed: 10 additions & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-process-dlopen-undefined-exports.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-process-dlopen-undefined-exports.js‎

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	+ 'use strict';
	2	+
	3	+ require('../common');
	4	+ const assert = require('assert');
	5	+
	6	+ const someBindingPath = './test/addons/hello-world/build/Release/binding.node';
	7	+
	8	+ assert.throws(() => {
	9	+ process.dlopen({ exports: undefined }, someBindingPath);
	10	+ }, Error);

0 commit comments

Comments

0 (0)