Merge pull request #6 from phillip-hall/master

Fix issue coercing boolean value
kogosoftwarellc · Mar 9, 2016 · 9464880 · 9464880
2 parents f932414 + 33a4bdb
commit 9464880
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -55,6 +55,10 @@ var COERCION_STRATEGIES = {
   },
 
   boolean: function(input) {
+    if (typeof input === 'boolean') {
+      return input;
+    }
+
     if (input === 'false') {
       return false;
     } else {

diff --git a/test/data-driven/coerce-boolean-params.js b/test/data-driven/coerce-boolean-params.js
@@ -11,11 +11,23 @@ module.exports = {
         in: 'path',
         name: 'path2',
         type: 'boolean'
+      },
+
+      {
+        in: 'query',
+        name: 'query1',
+        type: 'boolean'
+      },
+
+      {
+        in: 'query',
+        name: 'query2',
+        type: 'boolean'
       }
     ]
   },
 
-  requestPath: '/true/false',
+  requestPath: '/true/false?query1=true&query2=false',
 
   requestHeaders: null,
 
@@ -26,5 +38,8 @@ module.exports = {
     path2: false
   },
 
-  query: null
+  query: {
+    query1: true,
+    query2: false
+  }
 };