From 0ade4888f2744e764c87b20f658e4508ef55cc84 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Mon, 4 Dec 2017 05:46:40 -0800
Subject: [PATCH] test: forbid `common.mustCall*()` in process exit handlers

`common.mustCall()` and `common.mustCallAtLeast()` need to be called
before process exit handlers to work because checks are done inside a
process exit handler. Detect if being used inside a process exit handler
and throw.

PR-URL: https://github.com/nodejs/node/pull/17453
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Khaidi Chu <i@2333.moe>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 test/common/index.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/common/index.js b/test/common/index.js
index 2cf188abe5ffc9..8ba6c15aa49b40 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -493,6 +493,8 @@ exports.mustCallAtLeast = function(fn, minimum) {
 };
 
 function _mustCallInner(fn, criteria = 1, field) {
+  if (process._exiting)
+    throw new Error('Cannot use common.mustCall*() in process exit handler');
   if (typeof fn === 'number') {
     criteria = fn;
     fn = noop;