Merge pull request #6 from richmolj/master

Ensure `false` can be passed in `where` clause

Author: richmolj

src/util/parameterize.ts

@@ -4,7 +4,7 @@ export default function parameterize(obj : any, prefix? : string) : string {
   for (let key in obj) {
     let value = obj[key];
 
-    if (!!value) {
+    if (value !== undefined && value !== null && value !== '') {
       if (prefix) {
         key = `${prefix}[${key}]`;
       }

test/integration/finders-test.ts

@@ -260,6 +260,26 @@ describe('Model finders', function() {
       });
     });
 
+    describe('when value is false', function() {
+      before(function () {
+        fetchMock.reset();
+        fetchMock.get('http://example.com/api/v1/people?filter[id]=2&filter[a]=false', {
+          data: [
+            { id: '2', type: 'people' }
+          ]
+        });
+      });
+
+      it('still queries correctly', function(done) {
+        resultData(Person.where({ id: 2 }).where({ a: false }).all()).then((data) => {
+          expect(data.length).to.eq(1);
+          expect(data[0]).to.be.instanceof(Person);
+          expect(data[0]).to.have.property('id', '2');
+          done();
+        });
+      });
+    });
+
     describe('when merging association #where', function() {
       before(function () {
         fetchMock.reset();