process: make getActive{Handles,Requests} official

doc/api/process.md

@@ -1041,6 +1041,33 @@ a code.
 Specifying a code to [`process.exit(code)`][`process.exit()`] will override any
 previous setting of `process.exitCode`.
 
+## process.getActiveHandles()
+<!-- YAML
+added: REPLACEME
+-->
+
+* Returns: {Array}
+
+Returns an array containing all handles that are preventing the Node.js process
+from exiting.
+
+```js
+console.log(`Active handles: ${util.inspect(process.getActiveHandles())}`);
+```
+
+## process.getActiveRequests()
+<!-- YAML
+added: REPLACEME
+-->
+
+* Returns: {Array}
+
+Returns an array containing all requests that are preventing the Node.js process
+from exiting.
+
+```js
+console.log(`Active requests: ${util.inspect(process.getActiveRequests())}`);
+```
 
 ## process.getegid()
 <!-- YAML

lib/internal/bootstrap_node.js

@@ -298,6 +298,8 @@
 
   function setupProcessObject() {
     process._setupProcessObject(pushValueToArray);
+    process._getActiveHandles = process.getActiveHandles;
+    process._getActiveRequests = process.getActiveRequests;
 
     function pushValueToArray() {
       for (var i = 0; i < arguments.length; i++)

src/node.cc

@@ -3195,8 +3195,8 @@ void SetupProcessObject(Environment* env,
   env->SetMethod(process,
                  "_stopProfilerIdleNotifier",
                  StopProfilerIdleNotifier);
-  env->SetMethod(process, "_getActiveRequests", GetActiveRequests);
-  env->SetMethod(process, "_getActiveHandles", GetActiveHandles);
+  env->SetMethod(process, "getActiveRequests", GetActiveRequests);
+  env->SetMethod(process, "getActiveHandles", GetActiveHandles);
   env->SetMethod(process, "reallyExit", Exit);
   env->SetMethod(process, "abort", Abort);
   env->SetMethod(process, "chdir", Chdir);

test/parallel/test-process-getactivehandles.js

@@ -30,7 +30,7 @@ function clientConnected(client) {
 
 
 function checkAll() {
-  const handles = process._getActiveHandles();
+  const handles = process.getActiveHandles();
 
   clients.forEach(function(item) {
     assert.ok(handles.includes(item));

test/parallel/test-timers-same-timeout-wrong-list-deleted.js

@@ -65,6 +65,6 @@ const handle1 = setTimeout(common.mustCall(function() {
 }), TIMEOUT);
 
 function getActiveTimers() {
-  const activeHandles = process._getActiveHandles();
+  const activeHandles = process.getActiveHandles();
   return activeHandles.filter((handle) => handle instanceof Timer);
 }

test/pseudo-tty/ref_keeps_node_running.js

@@ -12,7 +12,7 @@ handle.readStart();
 handle.onread = () => {};
 
 function isHandleActive(handle) {
-  return process._getActiveHandles().some((active) => active === handle);
+  return process.getActiveHandles().some((active) => active === handle);
 }
 
 strictEqual(isHandleActive(handle), true, 'TTY handle not initially active');

test/pummel/test-net-connect-econnrefused.js

@@ -51,8 +51,8 @@ function pummel() {
 
 function check() {
   setTimeout(function() {
-    assert.strictEqual(process._getActiveRequests().length, 0);
-    assert.strictEqual(process._getActiveHandles().length, 1); // the timer
+    assert.strictEqual(process.getActiveRequests().length, 0);
+    assert.strictEqual(process.getActiveHandles().length, 1); // the timer
     check_called = true;
   }, 0);
 }