fix(utils): uses a recursive convertToPSParam to get converted types …

…inside array
rannn505 · Nov 12, 2019 · 33e7970 · 33e7970
1 parent b0a0f7f
commit 33e7970
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/utils.js b/lib/utils.js
@@ -59,7 +59,7 @@ const convertToPSParam = val => {
     case 'Number':
       return val;
     case 'Array':
-      return val;
+      return val.map(convertToPSParam);
     case 'Object':
       return `@${JSON.stringify(val).replace(/:/g, '=').replace(/,/g, ';')}`;
     case 'Boolean':

diff --git a/test/others.spec.js b/test/others.spec.js
@@ -14,6 +14,7 @@ const {
   convertToPSParam,
   convertToPSOption,
 } = require('../lib/utils');
+
 const {
   PS_PROC_ERROR,
   PS_ARG_MISS_ERROR,
@@ -44,6 +45,15 @@ describe('Utils', () => {
   });
 
   describe('convert', () => {
+    it('convert an array to a PS param', () => {
+      const param = [true, "String with spaces", { key: 123 }];
+      const convertedParam = convertToPSParam(param);
+
+      expect(convertedParam[0]).to.equal("$True");
+      expect(convertedParam[1]).to.equal('"String with spaces"');
+      expect(convertedParam[2]).to.equal('@{"key"=123}');
+    });
+
     it('convert a JS param to a PS param');
     it('convert a JS option to a PS option');
   });