console: fix console.table() display edge case

Trott · MylesBorins · commit 29bc735d42e7 · 2018-05-04T11:54:32.000-04:00
If the properties are not specified in `console.table()`, then we should make a best effort to determine them rather than put all values into a "Values" column. PR-URL: #20323 Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/console.js b/lib/console.js
@@ -363,9 +363,7 @@ Console.prototype.table = function(tabularData, properties) {
     tabularData = previewSetIterator(tabularData);
 
   const setlike = setIter || isSet(tabularData);
-  if (setlike ||
-      (properties === undefined &&
-       (isArray(tabularData) || isTypedArray(tabularData)))) {
+  if (setlike) {
     const values = [];
     let length = 0;
     for (const v of tabularData) {
diff --git a/test/parallel/test-console-table.js b/test/parallel/test-console-table.js
@@ -41,13 +41,13 @@ test([1, 2, 3], `
 `);
 
 test([Symbol(), 5, [10]], `
-&#9484;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9488;
-&#9474; (index) &#9474;  Values  &#9474;
-&#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
-&#9474;    0    &#9474; Symbol() &#9474;
-&#9474;    1    &#9474;    5     &#9474;
-&#9474;    2    &#9474;  [ 10 ]  &#9474;
-&#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
+&#9484;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9488;
+&#9474; (index) &#9474; 0  &#9474;  Values  &#9474;
+&#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
+&#9474;    0    &#9474;    &#9474; Symbol() &#9474;
+&#9474;    1    &#9474;    &#9474;    5     &#9474;
+&#9474;    2    &#9474; 10 &#9474;          &#9474;
+&#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
 `);
 
 test([undefined, 5], `
@@ -182,10 +182,10 @@ test({ a: undefined }, ['x'], `
 `);
 
 test([], `
-&#9484;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9488;
-&#9474; (index) &#9474; Values &#9474;
-&#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
-&#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
+&#9484;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9488;
+&#9474; (index) &#9474;
+&#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
+&#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
 `);
 
 test(new Map(), `
@@ -194,3 +194,12 @@ test(new Map(), `
 &#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
 &#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
 `);
+
+test([{ a: 1, b: 'Y' }, { a: 'Z', b: 2 }], `
+&#9484;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9472;&#9516;&#9472;&#9472;&#9472;&#9472;&#9472;&#9488;
+&#9474; (index) &#9474;  a  &#9474;  b  &#9474;
+&#9500;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9532;&#9472;&#9472;&#9472;&#9472;&#9472;&#9508;
+&#9474;    0    &#9474;  1  &#9474; 'Y' &#9474;
+&#9474;    1    &#9474; 'Z' &#9474;  2  &#9474;
+&#9492;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9524;&#9472;&#9472;&#9472;&#9472;&#9472;&#9496;
+`);

-Original file line number
+Diff line change
 `);
 test([Symbol(), 5, [10]], `
 -┌─────────┬──────────┐
 -│ (index) │  Values  │
 -├─────────┼──────────┤
 -│    0    │ Symbol() │
 -│    1    │    5     │
 -│    2    │  [ 10 ]  │
 -└─────────┴──────────┘
 +┌─────────┬────┬──────────┐
 +│ (index) │ 0  │  Values  │
 +├─────────┼────┼──────────┤
 +│    0    │    │ Symbol() │
 +│    1    │    │    5     │
 +│    2    │ 10 │          │
 +└─────────┴────┴──────────┘
 `);
 test([undefined, 5], `
 `);
 test([], `
 -┌─────────┬────────┐
 -│ (index) │ Values │
 -├─────────┼────────┤
 -└─────────┴────────┘
 +┌─────────┐
 +│ (index) │
 +├─────────┤
 +└─────────┘
 `);
 test(new Map(), `
 ├───────────────────┼─────┼────────┤
 └───────────────────┴─────┴────────┘
 `);
++
 +test([{ a: 1, b: 'Y' }, { a: 'Z', b: 2 }], `
 +┌─────────┬─────┬─────┐
 +│ (index) │  a  │  b  │
 +├─────────┼─────┼─────┤
 +│    0    │  1  │ 'Y' │
 +│    1    │ 'Z' │  2  │
 +└─────────┴─────┴─────┘
 +`);