fix api requests with post/put

Author: jochen-testingbot

Makefile

@@ -9,4 +9,4 @@ clean:
 lint:
 		eslint lib/
 test: clean lint
-	mocha --reporter spec test/ $(MOCHAFLAGS)
+	mocha -t 6000 --reporter spec test/ $(MOCHAFLAGS)

lib/api.js

@@ -134,25 +134,33 @@ TestingBot.prototype.deleteLabTest = function(testID, callback) {
 };
 
 TestingBot.prototype.request = function(req_data, callback) {
-  var requestPath = '/v1/' + req_data.url;
+  var requestPath = '/v1' + req_data.url;
   if (req_data.method === 'GET' && req_data.data) {
     requestPath = requestPath + '?' + querystring.stringify(req_data.data);
   }
-
-  request({
+  var requestOptions = {
     method: req_data.method,
     uri: 'https://api.testingbot.com' + requestPath,
     auth: {
       user: this.options.api_key,
       pass: this.options.api_secret,
       sendImmediately: true
     }
-  }, function(error, response) {
+  };
+
+  if (req_data.method !== 'GET' && req_data.data) {
+    requestOptions.json = req_data.data;
+  }
+
+  request(requestOptions, function(error, response) {
     var responseBody = null;
+    if (typeof(response.body) === 'string') {
+      response.body = JSON.parse(response.body);
+    }
     if (response.statusCode.toString().substring(0, 1) === '2') {
-      responseBody = JSON.parse(response.body);
+      responseBody = response.body;
     } else {
-      error = JSON.parse(response.body);
+      error = response.body;
     }
     callback(error, responseBody);
   });

package.json

@@ -2,7 +2,7 @@
   "author": "TestingBot <[email protected]> (testingbot.com)",
   "name": "testingbot-api",
   "description": "A wrapper around TestingBot's REST API",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "scripts": {
     "lint": "eslint .",
     "test": "make test"

test/api_test.js

@@ -51,4 +51,18 @@ describe('Api Tests', function() {
       });
     });
   });
+
+  it('should update a user object', function(done) {
+    var that = this;
+    var newName = "name_" + Math.round(Math.random()*10000);
+    this.api.updateUserInfo({ user: { first_name : newName } }, function(err, response) {
+      assert.equal(err, null);
+      that.api.getUserInfo(function(err, response) {
+        assert.notEqual(response, null);
+        assert.equal(err, null);
+        assert.equal(response.first_name, newName);
+        done();
+      });
+    });
+  });
 });