Added a check for booleans http::formatValue

Aaron Fullerton · Aaron Fullerton · commit 5b92da1c9cc9 · 2017-06-30T11:00:54.000-06:00
diff --git a/src/http.js b/src/http.js
@@ -148,7 +148,8 @@ function formatValue({value, collectionFormat, allowEmptyValue}, skipEncoding) {
   if (typeof value === 'undefined' && allowEmptyValue) {
     return ''
   }
-  if (isFile(value)) {
+
+  if (isFile(value) || typeof value === 'boolean') {
     return value
   }
 
diff --git a/test/http.js b/test/http.js
@@ -208,11 +208,14 @@ describe('http', () => {
           },
           two: {
             value: 2
+          },
+          three: {
+            value: false
           }
         }
       }
 
-      expect(encodeFormOrQuery(req.query)).toEqual('one=1&two=2')
+      expect(encodeFormOrQuery(req.query)).toEqual('one=1&two=2&three=false')
     })
 
     it('should handle arrays', function () {

Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,8 @@ function formatValue({value, collectionFormat, allowEmptyValue}, skipEncoding) {`
`148`	`148`	`if (typeof value === 'undefined' && allowEmptyValue) {`
`149`	`149`	`return ''`
`150`	`150`	`}`
`151`		`- if (isFile(value)) {`
	`151`	`+`
	`152`	`+ if (isFile(value) \|\| typeof value === 'boolean') {`
`152`	`153`	`return value`
`153`	`154`	`}`
`154`	`155`
Original file line number	Diff line number	Diff line change
`@@ -208,11 +208,14 @@ describe('http', () => {`
`208`	`208`	`},`
`209`	`209`	`two: {`
`210`	`210`	`value: 2`
	`211`	`+ },`
	`212`	`+ three: {`
	`213`	`+ value: false`
`211`	`214`	`}`
`212`	`215`	`}`
`213`	`216`	`}`
`214`	`217`
`215`		`- expect(encodeFormOrQuery(req.query)).toEqual('one=1&two=2')`
	`218`	`+ expect(encodeFormOrQuery(req.query)).toEqual('one=1&two=2&three=false')`
`216`	`219`	`})`
`217`	`220`
`218`	`221`	`it('should handle arrays', function () {`