Merge pull request #1093 from extempl/master

Fix boolean false parameter

Author: shockey

src/execute.js

@@ -193,6 +193,11 @@ export function pathBuilder({req, value, parameter}) {
 // Add a query to the `query` object, which will later be stringified into the URL's search
 export function queryBuilder({req, value, parameter}) {
   req.query = req.query || {}
+
+  if (value === false && parameter.type === 'boolean') {
+    value = 'false'
+  }
+
   if (value) {
     req.query[parameter.name] = {
       collectionFormat: parameter.collectionFormat,

test/execute.js

@@ -379,6 +379,39 @@ describe('execute', () => {
       })
     })
 
+    it('should correctly process boolean parameters', function () {
+      // Given
+      const spec = {
+        host: 'swagger.io',
+        basePath: '/v1',
+        consumes: ['application/json'],
+        paths: {
+          '/pets/findByStatus': {
+            get: {
+              operationId: 'getMe',
+              parameters: [{
+                in: 'query',
+                name: 'status',
+                type: 'boolean',
+                required: false
+              }]
+            }
+          }
+        }
+      }
+
+      // When
+      const req = buildRequest({spec, operationId: 'getMe', parameters: {status: false}})
+
+      // Then
+      expect(req).toEqual({
+        url: 'http://swagger.io/v1/pets/findByStatus?status=false',
+        method: 'GET',
+        credentials: 'same-origin',
+        headers: { }
+      })
+    })
+
     it('should throw error if there is no parameter value', function () {
       // Given
       const spec = {