Merge pull request #41 from VisualTesting/fix/query-params

Using query params instead of body for getBuild

Author: elicwhite

server/controllers/api.js

@@ -172,8 +172,8 @@ Api.prototype = {
   },
 
   getBuild: function(req, res) {
-    var project = req.body.project;
-    var buildId = req.body.id;
+    var project = req.query.project;
+    var buildId = req.query.id;
 
     if (!project || !buildId) {
       res.status(400).json({

test/api-test.js

@@ -381,7 +381,7 @@ describe('module/api', function() {
           build: id
         })).resolves(false);
 
-        instance = instance.send({
+        instance = instance.query({
           project: project,
           id: id
         })
@@ -397,7 +397,7 @@ describe('module/api', function() {
 
           storageStub.hasBuild = this.sinon.stub().resolves(false);
 
-          instance = instance.send({
+          instance = instance.query({
             project: project,
             id: id
           });
@@ -429,7 +429,7 @@ describe('module/api', function() {
         storageStub.hasBuild = this.sinon.stub().resolves(true);
 
         instance = api.get('/api/getBuild')
-        .send({
+        .query({
           project: project,
           id: buildId
         });
@@ -442,7 +442,7 @@ describe('module/api', function() {
           build: buildId
         })).resolves();
 
-        return instance.send({
+        return instance.query({
           project: project,
           id: buildId
         })