Exclude nested properties in md output mode. Closes #4241

pnp · Apr 5, 2023 · 600b4c0 · 600b4c0
1 parent 20be5df
commit 600b4c0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/src/Command.spec.ts b/src/Command.spec.ts
@@ -509,7 +509,7 @@ describe('Command', () => {
     assert(actual.indexOf('\\_\\*\\~\\`\\|') > -1);
   });
 
-  it('serializes objects that are values to JSON', () => {
+  it('excludes objects in md output', () => {
     const command = new MockCommand1();
     const commandOutput = [
       {
@@ -519,7 +519,7 @@ describe('Command', () => {
       }
     ];
     const actual = command.getMdOutput(commandOutput, command, { options: { output: 'md' } });
-    assert(actual.indexOf(JSON.stringify(commandOutput[0].property)) > -1);
+    assert(actual.indexOf(JSON.stringify(commandOutput[0].property)) === -1);
   });
 
   it('excludes objects that are values to JSON', () => {

diff --git a/src/Command.ts b/src/Command.ts
@@ -631,14 +631,16 @@ export default abstract class Command {
           `Property | Value`, os.EOL,
           `---------|-------`, os.EOL
         );
-        output.push(Object.keys(l).map(k => {
-          const value = l[k];
-          let stringValue = value;
-          if (typeof value === 'object') {
-            stringValue = JSON.stringify(value);
+        output.push(Object.keys(l).filter(x => {
+          if (!options.query && typeof l[x] === 'object') {
+            return;
           }
 
-          return `${md.escapeMd(k)} | ${md.escapeMd(stringValue)}`;
+          return x;
+        }).map(k => {
+          const value = l[k];
+
+          return `${md.escapeMd(k)} | ${md.escapeMd(value)}`;
         }).join(os.EOL), os.EOL);
         output.push(os.EOL);
       });