Add type check for upload operation's 'isPublic' parameter

Author: bgrana

tasks/lib/operations.js

@@ -31,6 +31,10 @@ module.exports.upload_mac = function upload_mac(grunt, instance_name, file, isPu
                 'Authorization': 'Bearer ' + instance_info.token_info.access_token
             };
 
+            if (typeof isPublic !== 'boolean') {
+                reject('Error: isPublic parameter must be a boolean.');
+            }
+
             try {
                 headers['Content-Length'] = fs.statSync(file)['size'];
             } catch (e) {

test/operations.spec.js

@@ -220,4 +220,11 @@ describe('Upload', function () {
             expect(url).to.include('public=true');
         });
     });
+
+    it('should fail to upload a MAC if the isPublic parameter is not a boolean', function () {
+        var publicErrorMsg = 'Error: isPublic parameter must be a boolean.';
+        common.stubOperation('post', {statusCode: 200});
+        var promise = ops.upload_mac(grunt, "some_instance", "File", "Not a boolean");
+        expect(promise).to.be.rejectedWith(publicErrorMsg);
+    });
 });