util: format() now formats bigint and booleans

This is necessary to distinguish them from other data types.

PR-URL: nodejs#25046
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>

Author: BridgeAR

doc/api/util.md

@@ -245,8 +245,7 @@ util.format('%s:%s', 'foo');
 ```
 
 Values that are not part of the format string are formatted using
-`util.inspect()` if their type is either `'object'`, `'symbol'`, `'function'`
-or `'number'` and using `String()` in all other cases.
+`util.inspect()` if their type is not `string`.
 
 If there are more arguments passed to the `util.format()` method than the
 number of specifiers, the extra arguments are concatenated to the returned

lib/util.js

@@ -174,17 +174,8 @@ function formatWithOptions(inspectOptions, ...args) {
 
   while (a < args.length) {
     const value = args[a];
-    // TODO(BridgeAR): This should apply for all besides strings. Especially
-    // BigInt should be properly inspected.
     str += join;
-    if (typeof value !== 'string' &&
-        typeof value !== 'boolean' &&
-        // eslint-disable-next-line valid-typeof
-        typeof value !== 'bigint') {
-      str += inspect(value, inspectOptions);
-    } else {
-      str += value;
-    }
+    str += typeof value !== 'string' ? inspect(value, inspectOptions) : value;
     join = ' ';
     a++;
   }

test/parallel/test-util-format.js

@@ -316,10 +316,27 @@ assert.strictEqual(util.format(new BadCustomError('foo')),
 assert.strictEqual(util.format('1', '1'), '1 1');
 assert.strictEqual(util.format(1, '1'), '1 1');
 assert.strictEqual(util.format('1', 1), '1 1');
-assert.strictEqual(util.format(1, 1), '1 1');
+assert.strictEqual(util.format(1, -0), '1 -0');
 assert.strictEqual(util.format('1', () => {}), '1 [Function]');
 assert.strictEqual(util.format(1, () => {}), '1 [Function]');
 assert.strictEqual(util.format('1', "'"), "1 '");
 assert.strictEqual(util.format(1, "'"), "1 '");
 assert.strictEqual(util.format('1', 'number'), '1 number');
 assert.strictEqual(util.format(1, 'number'), '1 number');
+assert.strictEqual(util.format(5n), '5n');
+assert.strictEqual(util.format(5n, 5n), '5n 5n');
+
+// Check `formatWithOptions`.
+assert.strictEqual(
+  util.formatWithOptions(
+    { colors: true },
+    true, undefined, Symbol(), 1, 5n, null, 'foobar'
+  ),
+  '\u001b[33mtrue\u001b[39m ' +
+    '\u001b[90mundefined\u001b[39m ' +
+    '\u001b[32mSymbol()\u001b[39m ' +
+    '\u001b[33m1\u001b[39m ' +
+    '\u001b[33m5n\u001b[39m ' +
+    '\u001b[1mnull\u001b[22m ' +
+    'foobar'
+);