fix post parameters

Author: jochen-testingbot

lib/api.js

@@ -148,8 +148,8 @@ TestingBot.prototype.request = function(req_data, callback) {
     }
   };
 
-  if (req_data.method !== 'GET' && req_data.data) {
-    requestOptions.json = req_data.data;
+  if (req_data.method !== 'GET') {
+    requestOptions.form = req_data.data;
   }
 
   request(requestOptions, function(error, response) {

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.3",
+  "version": "1.0.4",
   "scripts": {
     "lint": "eslint .",
     "test": "make test"

test/api_test.js

@@ -65,4 +65,55 @@ describe('Api Tests', function() {
       });
     });
   });
+
+  it('should update a test by specifying legacy data', function(done) {
+    var that = this;
+    this.api.getTests(function(err, response) {
+      assert.equal(response.data && response.data.length > 0, true);
+      var singleTest = response.data[0];
+      that.api.getTestDetails(singleTest.id, function(err, response) {
+        assert.notEqual(response, null);
+        var payload = { "test[success]" : "1", "test[status_message]" : "test" };
+        that.api.updateTest(payload, singleTest.id, function(err) {
+          that.api.getTestDetails(singleTest.id, function(err, response) {
+            assert.equal(response.status_message, "test");
+            done();
+          });
+        });
+      });
+    });
+  });
+
+  it('should update a test by specifying object data', function(done) {
+    var that = this;
+    this.api.getTests(function(err, response) {
+      assert.equal(response.data && response.data.length > 0, true);
+      var singleTest = response.data[0];
+      that.api.getTestDetails(singleTest.id, function(err, response) {
+        assert.notEqual(response, null);
+        var payload = { test: { success: 1, status_message: "test2" } };
+        that.api.updateTest(payload, singleTest.id, function(err) {
+          that.api.getTestDetails(singleTest.id, function(err, response) {
+            assert.equal(response.status_message, "test2");
+            done();
+          });
+        });
+      });
+    });
+  });
+
+  it('should be possible to delete a test', function(done) {
+    var that = this;
+    this.api.getTests(function(err, response) {
+      assert.equal(response.data && response.data.length > 0, true);
+      var singleTest = response.data[0];
+      that.api.deleteTest(singleTest.id, function(err, response) {
+        that.api.getTestDetails(singleTest.id, function(err, response) {
+          assert.equal(response, null);
+          assert.notEqual(err, null);
+          done();
+        });
+      });
+    });
+  });
 });