test: work scheduled in process.nextTick can keep the event loop alive · nodejs/node@8850555 · GitHub

nodejs / node Public

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

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

Additional navigation options

Commit 8850555

Browse filesBrowse the repository at this point in the historyBrowse files

andreubotella

authored and

danielleadams

committed

test: work scheduled in process.nextTick can keep the event loop alive

PR-URL: #43787 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>

1 parent cd0d9dd commit 8850555
Copy full SHA for 8850555

1 file changed

+10-1Lines changed: 10 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-process-beforeexit.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-process-beforeexit.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-process-beforeexit.js

+10-1Lines changed: 10 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -59,12 +59,21 @@ function tryRepeatedTimer() {
59	59	if (++n < N)
60	60	setTimeout(repeatedTimer, 1);
61	61	else // n == N
62		- process.once('beforeExit', common.mustCall(tryNextTick));
	62	+ process.once('beforeExit', common.mustCall(tryNextTickSetImmediate));
63	63	}, N);
64	64	setTimeout(repeatedTimer, 1);
65	65	}
66	66
67	67	// Test if the callback of `process.nextTick` can be invoked.
	68	+ function tryNextTickSetImmediate() {
	69	+ process.nextTick(common.mustCall(function() {
	70	+ setImmediate(common.mustCall(() => {
	71	+ process.once('beforeExit', common.mustCall(tryNextTick));
	72	+ }));
	73	+ }));
	74	+ }
	75	+
	76	+ // Test that `process.nextTick` won't keep the event loop running by itself.
68	77	function tryNextTick() {
69	78	process.nextTick(common.mustCall(function() {
70	79	process.once('beforeExit', common.mustNotCall());

0 commit comments

Comments

0 (0)