Merge pull request #657 from swagger-api/issue-656

fix, tests for shared params

Author: fehguy

browser/swagger-client.js

@@ -95,6 +95,9 @@ Resolver.prototype.resolve = function (spec, arg1, arg2, arg3) {
           var parameter = sharedParameters[i];
           parameters.unshift(parameter);
         }
+        if(method !== 'parameters' && _.isObject(operation)) {
+          operation.parameters = operation.parameters || parameters;
+        }
 
         for (i in parameters) {
           var parameter = parameters[i];

browser/swagger-client.min.js

@@ -943,6 +943,13 @@ describe('swagger resolver', function () {
                 description: 'ok'
               }
             }
+          },
+          post: {
+            responses: {
+              200: {
+                description: 'ok'
+              }
+            }
           }
         }
       }
@@ -952,6 +959,10 @@ describe('swagger resolver', function () {
       expect(parameters[0].name).toEqual('skip');
       expect(parameters[1].name).toEqual('limit');
       expect(spec.paths['/foo'].parameters.length).toBe(0);
+
+
+      parameters = spec.paths['/foo'].post.parameters;
+      console.log(parameters);
       done();
     });
   });
@@ -1032,6 +1043,13 @@ describe('swagger resolver', function () {
                 description: 'ok'
               }
             }
+          },
+          post: {
+            responses: {
+              200: {
+                description: 'ok'
+              }
+            }
           }
         }
       }
@@ -1041,6 +1059,10 @@ describe('swagger resolver', function () {
       expect(parameters[0].name).toEqual('skip');
       expect(parameters[1].name).toEqual('limit');
       expect(spec.paths['/foo'].parameters.length).toBe(0);
+
+      parameters = spec.paths['/foo'].post.parameters;
+      expect(parameters[0].name).toEqual('skip');
+      expect(parameters.length).toEqual(1);
       done();
     });
   });