Merge pull request #14 from crossfilter/fix-filtering-test-array-reduce

Filtering tests and initial 0 for reduce

package.json

@@ -1,6 +1,6 @@
 {
   "name": "universe",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "The fastest way to query and explore multivariate datasets",
   "main": "src/universe.js",
   "directories": {

src/aggregation.js

@@ -189,13 +189,13 @@ function convertAggregatorString(keyString) {
 function $sum(children) {
   return children.reduce(function(a, b) {
     return a + b
-  })
+  }, 0)
 }
 
 function $avg(children) {
   return children.reduce(function(a, b) {
     return a + b
-  }) / children.length
+  }, 0) / children.length
 }
 
 function $max(children) {

test/postAggregation.spec.js

@@ -82,6 +82,28 @@ describe('universe postAggregation', function() {
       })
   })
 
+  it('works after filtering', function() {
+    var query
+    return u.then(function(u) {
+        return u.query({
+          groupBy: 'total',
+        })
+      })
+      .then(function(res) {
+        return res.changeMap({
+          count: true
+        })
+      })
+      .then(function(res) {
+        query = res
+        expect(res.data[0].value.countChangeFromEnd).to.equal(5)
+        return res.universe.filter('type', 'cash')
+      })
+      .then(function(){
+        expect(query.data[0].value.countChangeFromEnd).to.equal(0)
+      })
+  })
+
   it('can sortByKey ascending and descending', function() {
     return u.then(function(u) {
         return u.query({