Skip to content

Commit 8e5e419

Browse files
Izhakisindresorhus
authored andcommitted
Fix process.exit within process.once throwing no-process-exit - fixes #133 (#135)
Support process.exit process.once callback, just like with process.on.
1 parent 4847a3d commit 8e5e419

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

rules/no-process-exit.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const create = context => {
1414
const callee = node.callee;
1515

1616
if (callee.type === 'MemberExpression' && callee.object.name === 'process') {
17-
if (callee.property.name === 'on') {
17+
if (callee.property.name === 'on' || callee.property.name === 'once') {
1818
processEventHandler = node;
1919
return;
2020
}

test/no-process-exit.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,11 @@ ruleTester.run('no-process-exit', rule, {
2424
'process.on("SIGINT", () => { process.exit(1); })',
2525
'process.on("SIGINT", () => process.exit(1))',
2626
'process.on("SIGINT", () => { if (true) { process.exit(1); } })',
27+
'process.once("SIGINT", function() { process.exit(1); })',
28+
'process.once("SIGKILL", function() { process.exit(1); })',
29+
'process.once("SIGINT", () => { process.exit(1); })',
30+
'process.once("SIGINT", () => process.exit(1))',
31+
'process.once("SIGINT", () => { if (true) { process.exit(1); } })',
2732
''
2833
],
2934
invalid: [

0 commit comments

Comments
 (0)