From 7d4759bc3a691431bd40d475845c87344f06852d Mon Sep 17 00:00:00 2001
From: Luigi Pinca <luigipinca@gmail.com>
Date: Thu, 6 May 2021 20:06:14 +0200
Subject: [PATCH] os: add os.devnull()

Returns the platform-specific file path of the null device.
---
 doc/api/os.md              | 12 ++++++++++++
 doc/api/path.md            | 12 ------------
 lib/os.js                  |  9 +++++++++
 lib/path.js                |  2 --
 test/parallel/test-os.js   | 10 ++++++++++
 test/parallel/test-path.js |  6 ------
 6 files changed, 31 insertions(+), 20 deletions(-)

diff --git a/doc/api/os.md b/doc/api/os.md
index f3d3f2d65249a4..b4fbde786fd2a6 100644
--- a/doc/api/os.md
+++ b/doc/api/os.md
@@ -122,6 +122,18 @@ The properties included on each object include:
 `nice` values are POSIX-only. On Windows, the `nice` values of all processors
 are always 0.
 
+## `os.devnull()`
+<!-- YAML
+added: REPLACEME
+-->
+
+* Returns: {string}
+
+Returns the platform-specific file path of the null device:
+
+* `\\.\nul` on Windows
+* `/dev/null` on POSIX
+
 ## `os.endianness()`
 <!-- YAML
 added: v0.9.4
diff --git a/doc/api/path.md b/doc/api/path.md
index 800d7a175b11ef..839c45da906677 100644
--- a/doc/api/path.md
+++ b/doc/api/path.md
@@ -135,18 +135,6 @@ process.env.PATH.split(path.delimiter);
 // Returns ['C:\\Windows\\system32', 'C:\\Windows', 'C:\\Program Files\\node\\']
 ```
 
-## `path.devnull`
-<!-- YAML
-added: REPLACEME
--->
-
-* {string}
-
-Provides the platform-specific file path of the null device:
-
-* `\\.\nul` on Windows
-* `/dev/null` on POSIX
-
 ## `path.dirname(path)`
 <!-- YAML
 added: v0.1.16
diff --git a/lib/os.js b/lib/os.js
index 4396fcefd533c4..9ac75d97c507e5 100644
--- a/lib/os.js
+++ b/lib/os.js
@@ -349,9 +349,18 @@ function userInfo(options) {
   return user;
 }
 
+/**
+ * @returns {string}
+ */
+function devnull() {
+  return isWindows ? '\\\\.\\nul' : '/dev/null';
+}
+devnull[SymbolToPrimitive] = () => devnull();
+
 module.exports = {
   arch,
   cpus,
+  devnull,
   endianness,
   freemem: getFreeMem,
   getPriority,
diff --git a/lib/path.js b/lib/path.js
index 7d15d69a259324..f3e5a3b3416c4b 100644
--- a/lib/path.js
+++ b/lib/path.js
@@ -1065,7 +1065,6 @@ const win32 = {
     return ret;
   },
 
-  devnull: '\\\\.\\nul',
   sep: '\\',
   delimiter: ';',
   win32: null,
@@ -1532,7 +1531,6 @@ const posix = {
     return ret;
   },
 
-  devnull: '/dev/null',
   sep: '/',
   delimiter: ':',
   win32: null,
diff --git a/test/parallel/test-os.js b/test/parallel/test-os.js
index 5ee0fb9ca8dadb..4b3f0f5fccb668 100644
--- a/test/parallel/test-os.js
+++ b/test/parallel/test-os.js
@@ -259,3 +259,13 @@ if (!common.isIBMi) {
 
 is.number(+os.freemem, 'freemem');
 is.number(os.freemem(), 'freemem');
+
+const devnull = os.devnull();
+
+if (common.isWindows) {
+  assert.strictEqual(devnull, '\\\\.\\nul');
+} else {
+  assert.strictEqual(devnull, '/dev/null');
+}
+
+assert.strictEqual(`${os.devnull}`, devnull);
diff --git a/test/parallel/test-path.js b/test/parallel/test-path.js
index 5139d5c18802ad..7a5f3d4715a7f2 100644
--- a/test/parallel/test-path.js
+++ b/test/parallel/test-path.js
@@ -67,12 +67,6 @@ assert.strictEqual(path.win32.delimiter, ';');
 // posix
 assert.strictEqual(path.posix.delimiter, ':');
 
-// path.devnull tests
-// windows
-assert.strictEqual(path.win32.devnull, '\\\\.\\nul');
-// posix
-assert.strictEqual(path.posix.devnull, '/dev/null');
-
 if (common.isWindows)
   assert.strictEqual(path, path.win32);
 else