chore: rename to os.guessFileDescriptorType

Author: vladfrangu

doc/api/os.md

@@ -508,20 +508,20 @@ On POSIX systems, the operating system release is determined by calling
 available, `GetVersionExW()` will be used. See
 <https://en.wikipedia.org/wiki/Uname#Examples> for more information.
 
-## `os.guessHandleType(handle)`
+## `os.guessFileDescriptorType(fd)`
 
 <!-- YAML
 added: REPLACEME
 -->
 
-* `handle` {integer} The handle number to try and guess the type of.
+* `fd` {integer} The file descriptor number to try and guess the type of.
 
 * Returns: {string}
 
-Returns the type of the handle passed in, or `'INVALID'` if the provided handle
+Returns the type of the file descriptor passed in, or `'INVALID'` if the provided file descriptor
 is invalid.
 
-Currently, the following types for a handle can be returned:
+Currently, the following types for a file descriptor can be returned:
 
 * `'TCP'`
 * `'TTY'`

lib/os.js

@@ -328,7 +328,7 @@ module.exports = {
   uptime: getUptime,
   version: getOSVersion,
   machine: getMachine,
-  guessHandleType: _guessHandleType,
+  guessFileDescriptorType: _guessHandleType,
 };
 
 ObjectDefineProperties(module.exports, {

test/pseudo-tty/test-os-guessFileDescriptorType.js

@@ -0,0 +1,17 @@
+'use strict';
+
+require('../common');
+const { strictEqual } = require('assert');
+const { guessFileDescriptorType } = require('os');
+
+strictEqual(guessFileDescriptorType(0), 'TTY', 'stdin reported to not be a tty, but it is');
+strictEqual(guessFileDescriptorType(1), 'TTY', 'stdout reported to not be a tty, but it is');
+strictEqual(guessFileDescriptorType(2), 'TTY', 'stderr reported to not be a tty, but it is');
+
+strictEqual(guessFileDescriptorType(-1), 'INVALID', '-1 reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType(55555), 'UNKNOWN', '55555 reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType(2 ** 31), 'INVALID', '2^31 reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType(1.1), 'INVALID', '1.1 reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType('1'), 'INVALID', '\'1\' reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType({}), 'INVALID', '{} reported to be a tty, but it is not');
+strictEqual(guessFileDescriptorType(() => {}), 'INVALID', '() => {} reported to be a tty, but it is not');