Merge pull request #84 from caseywebdev/patch-1

Fix `t` query parameter for `build` command
apocas · Aug 28, 2017 · 94161fb · 94161fb
2 parents 2d0d03f + 8565c3f
commit 94161fb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/modem.js b/lib/modem.js
@@ -341,7 +341,7 @@ Modem.prototype.buildQuerystring = function(opts) {
 
   // serialize map values as JSON strings, else querystring truncates.
   Object.keys(opts).map(function(key, i) {
-    clone[key] = (opts[key] && typeof opts[key] === 'object') ?
+    clone[key] = (opts[key] && typeof opts[key] === 'object' && key !== 't') ?
       JSON.stringify(opts[key]) : opts[key];
   });
 

diff --git a/test/modem_test.js b/test/modem_test.js
@@ -84,9 +84,10 @@ describe('Modem', function () {
       "limit": 12,
       "filters": {
         "label": ["staging", "env=green"]
-      }
+      },
+      "t": ["repo:latest", "repo:1.0.0"]
     }
-    var control = 'limit=12&filters={"label"%3A["staging"%2C"env%3Dgreen"]}'
+    var control = 'limit=12&filters={"label"%3A["staging"%2C"env%3Dgreen"]}&t=repo%3Alatest&t=repo%3A1.0.0'
     var qs = modem.buildQuerystring(opts);
     assert.strictEqual(decodeURI(qs), control);
   });