test: add regression test for signal handler removal in exit · nodejs/node@f5ac4ec · GitHub

nodejs / node Public

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

• Code
• Issues 633
• Pull requests 756
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit f5ac4ec

Browse filesBrowse the repository at this point in the historyBrowse files

addaleax

authored and

MylesBorins

committed

test: add regression test for signal handler removal in exit

Refs: #30581 Refs: #30582 PR-URL: #30589 Refs: #30581 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 parent 4d396fd commit f5ac4ec
Copy full SHA for f5ac4ec

1 file changed

+9Lines changed: 9 additions & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-signal-handler-remove-on-exit.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-signal-handler-remove-on-exit.js‎

Copy file name to clipboard

+9Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -0,0 +1,9 @@
	1	+ 'use strict';
	2	+ require('../common');
	3	+
	4	+ // Regression test for https://github.com/nodejs/node/issues/30581
	5	+ // This script should not crash.
	6	+
	7	+ function dummy() {}
	8	+ process.on('SIGINT', dummy);
	9	+ process.on('exit', () => process.removeListener('SIGINT', dummy));

0 commit comments

Comments

0 (0)