Merge pull request #34 from ysa23/fix/console-reporter

Fix: Console reporter

package.json

@@ -10,7 +10,7 @@
     "datadog",
     "DogStatsD"
   ],
-  "version": "0.14.0",
+  "version": "0.14.1",
   "repository": {
     "type": "git",
     "url": "https://github.com/ysa23/metrics-js"

src/reporters/console-reporter.js

@@ -1,8 +1,8 @@
-const StringReporter = require('./string-reporter');
+const { StringReporter } = require('./string-reporter');
 
 function ConsoleReporter() {
   // eslint-disable-next-line no-console
-  const stringReporter = new StringReporter(console.log);
+  const stringReporter = new StringReporter({ action: console.log });
 
   function report(key, value, tags) {
     stringReporter.report(key, value, tags);

src/reporters/console-reporter.test.js

@@ -0,0 +1,45 @@
+const { Metrics, ConsoleReporter } = require('../index');
+
+describe('ConsoleReporter', () => {
+  describe('constructor', () => {
+    it('should create a new instance', () => {
+      expect(() => new ConsoleReporter()).not.toThrow();
+    });
+  });
+
+  describe('report', () => {
+    it('should not throw when called', () => {
+      const errback = jest.fn();
+      const reporter = new ConsoleReporter();
+      const metrics = new Metrics({ reporters: [reporter], errback });
+
+      metrics.space('space.meter').meter(() => console.log('do something'))();
+
+      expect(errback).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('increment', () => {
+    it('should not throw when called', () => {
+      const errback = jest.fn();
+      const reporter = new ConsoleReporter();
+      const metrics = new Metrics({ reporters: [reporter], errback });
+
+      metrics.space('space.meter').increment();
+
+      expect(errback).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('value', () => {
+    it('should not throw when called', () => {
+      const errback = jest.fn();
+      const reporter = new ConsoleReporter();
+      const metrics = new Metrics({ reporters: [reporter], errback });
+
+      metrics.space('space.meter').value(5);
+
+      expect(errback).not.toHaveBeenCalled();
+    });
+  });
+});