allow 0 in pushUnique

so we can use it to make index arrays

Author: alexcjohnson

src/lib/push_unique.js

@@ -11,6 +11,8 @@
 /**
  * Push array with unique items
  *
+ * Ignores falsy items, except 0 so we can use it to construct arrays of indices.
+ *
  * @param {array} array
  *  array to be filled
  * @param {any} item
@@ -30,7 +32,7 @@ module.exports = function pushUnique(array, item) {
         }
         array.push(item);
     }
-    else if(item && array.indexOf(item) === -1) array.push(item);
+    else if((item || item === 0) && array.indexOf(item) === -1) array.push(item);
 
     return array;
 };

test/jasmine/tests/lib_test.js

@@ -1292,21 +1292,21 @@ describe('Test lib.js:', function() {
             expect(this.array).toBe(out);
         });
 
-        it('should ignore falsy items', function() {
+        it('should ignore falsy items except 0', function() {
             Lib.pushUnique(this.array, false);
             expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }]);
 
             Lib.pushUnique(this.array, undefined);
             expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }]);
 
-            Lib.pushUnique(this.array, 0);
-            expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }]);
-
             Lib.pushUnique(this.array, null);
             expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }]);
 
             Lib.pushUnique(this.array, '');
             expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }]);
+
+            Lib.pushUnique(this.array, 0);
+            expect(this.array).toEqual(['a', 'b', 'c', { a: 'A' }, 0]);
         });
 
         it('should ignore item already in array', function() {