Merge pull request #9 from statful/counter_default_aggr

fix: counter default aggregations (sum, count)

Author: jmtavares

README.md

@@ -88,7 +88,7 @@ Read the methods options reference bellow to get more information about the defa
 
 | Option | Description | Type | Default for Counter | Default for Gauge | Default for Timer |
 |:---|:---|:---|:---|:---|:---|
-| agg | Defines the aggregations to be executed. These aggregations are merged with the ones configured globally, including method defaults.<br><br> **Valid Aggregations:** `avg, count, sum, first, last, p90, p95, min, max` | `array` | `['avg', 'p90']` | `[last]` | `['avg', 'p90', 'count']` |
+| agg | Defines the aggregations to be executed. These aggregations are merged with the ones configured globally, including method defaults.<br><br> **Valid Aggregations:** `avg, count, sum, first, last, p90, p95, min, max` | `array` | `['sum', 'count']` | `[last]` | `['avg', 'p90', 'count']` |
 | aggFreq | Defines the aggregation frequency in **seconds**. It overrides the global aggregation frequency configuration.<br><br> **Valid Aggregation Frequencies:** `10, 30, 60, 120, 180, 300` | `number` | `10` | `10` | `10` |
 | sampleRate | Defines the rate sampling. It overrides the global namespace configuration. | `number` | `100` | `100` | `100` |
 | namespace | Defines the namespace of the metric. It overrides the global namespace configuration. | `string` | `web` | `web` | `web` |

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "statful-client-javascript",
-  "version": "1.1.9",
+  "version": "2.0.0",
   "description": "Statful client for Javascript applications",
   "main": "dist/statful.js",
   "authors": [

dist/statful.js

@@ -1,5 +1,5 @@
 /**
-* statful-client-javascript 1.1.9
+* statful-client-javascript 2.0.0
 * Copyright 2017 Statful <https://www.statful.com/>
 */
 
@@ -247,7 +247,7 @@
         },
         counter: {
             tags: {},
-            aggregations: [ "avg", "p90" ]
+            aggregations: [ "sum", "count" ]
         },
         gauge: {
             tags: {},

dist/statful.min.js

@@ -1,6 +1,6 @@
 {
   "name": "statful-client-javascript",
-  "version": "1.1.9",
+  "version": "2.0.0",
   "description": "Statful client for Javascript applications",
   "banner": "/**\n* <%= pkg.name %> <%= pkg.version %>\n* Copyright 2017 Statful <https://www.statful.com/>\n*/\n",
   "scripts": {

package.json

@@ -33,7 +33,7 @@
         },
         counter: {
             tags: {},
-            aggregations: ['avg', 'p90']
+            aggregations: ['sum', 'count']
         },
         gauge: {
             tags: {},

src/statful.js

@@ -28,7 +28,7 @@ describe('Statful Client Unit testing', function () {
         expect(statful.config.timer.aggregations).toEqual(['avg', 'p90', 'count']);
 
         expect(statful.config.counter.tags).toEqual({});
-        expect(statful.config.counter.aggregations).toEqual(['avg', 'p90']);
+        expect(statful.config.counter.aggregations).toEqual(['sum', 'count']);
 
         expect(statful.config.gauge.tags).toEqual({});
         expect(statful.config.gauge.aggregations).toEqual(['last']);