console, doc, test: corrections following review

Nits in documentation, rework dirxml to use console.log, tests.

Fixes: #17128

doc/api/console.md

@@ -282,15 +282,13 @@ Defaults to `false`. Colors are customizable; see
 added: v8.0.0
 changes:
   - version: REPLACEME
-    pr-url: https://github.com/nodejs/node/pull/17128
-    description: "`console.dirxml` now calls `console.dir` for each argument."
+    pr-url: https://github.com/nodejs/node/pull/17152
+    description: "`console.dirxml` now calls `console.log` for its arguments."
 -->
 * `...data` {any}
 
-This method calls `console.dir()` with default options for each argument it
-receives. See [`console.dir()`][] for more details about said defaults.
-Please note that this method doesn't produce any xml formatting and uses the
-default `console.dir()` formatting resolution instead.
+This method calls `console.log()` passing it the arguments received.
+Please note that this method does not produce any XML formatting.
 
 ### console.error([data][, ...args])
 <!-- YAML
@@ -524,7 +522,6 @@ added: v8.0.0
 This method does not display anything unless used in the inspector. The
 `console.timelineEnd()` method is the deprecated form of [`console.timeEnd()`][].
 
-[`console.dir()`]: #console_console_dir_obj_options
 [`console.error()`]: #console_console_error_data_args
 [`console.group()`]: #console_console_group_label
 [`console.log()`]: #console_console_log_data_args

lib/console.js

@@ -162,11 +162,7 @@ Console.prototype.dir = function dir(object, options) {
 };
 
 
-Console.prototype.dirxml = function dirxml(...data) {
-  for (const item of data) {
-    Console.prototype.dir.call(this, item);
-  }
-};
+Console.prototype.dirxml = Console.prototype.log;
 
 
 Console.prototype.time = function time(label = 'default') {

test/parallel/test-console.js

@@ -179,13 +179,10 @@ assert.strictEqual(strings.shift(),
                    "{ foo: 'bar', inspect: [Function: inspect] }\n");
 assert.ok(strings.shift().includes('foo: [Object]'));
 assert.strictEqual(strings.shift().includes('baz'), false);
-assert.strictEqual(strings.shift(),
-                   "{ foo: 'bar', inspect: [Function: inspect] }\n");
-assert.strictEqual(strings.shift(),
-                   "{ foo: 'bar', inspect: [Function: inspect] }\n");
-assert.strictEqual(strings.shift().includes('foo: { bar: { baz:'), true);
-assert.strictEqual(strings.shift().includes('quux'), true);
-assert.strictEqual(strings.shift().includes('quux: true'), true);
+assert.strictEqual(strings.shift(), 'inspect inspect\n');
+assert.ok(strings[0].includes('foo: { bar: { baz:'));
+assert.ok(strings[0].includes('quux'));
+assert.ok(strings.shift().includes('quux: true'));
 
 assert.ok(/^label: \d+\.\d{3}ms$/.test(strings.shift().trim()));
 assert.ok(/^__proto__: \d+\.\d{3}ms$/.test(strings.shift().trim()));