Add http source "formData"

Add "formData" as an alias for "form", to make remoting metadata close to Swagger.
agriwebb · Jun 13, 2016 · 2e4a500 · 2e4a500
1 parent 6cf759a
commit 2e4a500
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 0 deletions.
diff --git a/lib/http-context.js b/lib/http-context.js
@@ -140,6 +140,7 @@ HttpContext.prototype.buildArgs = function(method) {
               val = ctx.req.body;
               break;
             case 'form':
+            case 'formData':
               // From the form (body)
               val = ctx.req.body && ctx.req.body[name];
               break;

diff --git a/lib/http-invocation.js b/lib/http-invocation.js
@@ -118,6 +118,7 @@ HttpInvocation.prototype._processArg = function(req, verb, query, accept) {
             req.body = val;
             break;
           case 'form':
+          case 'formData':
             // From the form (body)
             req.body = req.body || {};
             req.body[name] = val;

diff --git a/test/rest.browser.test.js b/test/rest.browser.test.js
@@ -204,6 +204,27 @@ describe('strong-remoting-rest', function() {
         });
       });
 
+      it('should allow arguments in the formData', function(done) {
+        var method = givenSharedStaticMethod(
+          function bar(a, b, cb) {
+            cb(null, a + b);
+          },
+          {
+            accepts: [
+              { arg: 'b', type: 'number', http: {source: 'formData' }  },
+              { arg: 'a', type: 'number', http: {source: 'formData' } }
+            ],
+            returns: { arg: 'n', type: 'number' },
+            http: { path: '/' }
+          }
+        );
+
+        objects.invoke(method.name, [1, 2], function(err, n) {
+          assert.equal(n, 3);
+          done();
+        });
+      });
+
       it('should respond with correct args if returns has multiple args', function(done) {
         var method = givenSharedStaticMethod(
           function(a, b, cb) {