feat: remove polyfill for user timing api, add unit tests

Author: jmtavares

dist/statful.js

@@ -1,6 +1,6 @@
 {
   "name": "statful-client-javascript",
-  "version": "2.0.4",
+  "version": "2.1.0",
   "description": "Statful client for Javascript applications",
   "banner": "/**\n* <%= pkg.name %> <%= pkg.version %>\n* Copyright 2017 Statful <https://www.statful.com/>\n*/\n",
   "scripts": {
@@ -26,7 +26,7 @@
     "babel-plugin-external-helpers": "^6.22.0",
     "babel-polyfill": "^6.26.0",
     "babel-preset-env": "^1.6.1",
-    "eslint": "^4.9.0",
+    "eslint": "^4.11.0",
     "jasmine-ajax": "^3.3.1",
     "jasmine-core": "^2.8.0",
     "karma": "^1.7.1",

dist/statful.min.js

@@ -1,4 +1,3 @@
-import 'usertiming';
 import StatfulLogger from './logger';
 import StatfulUtil from './statful-util';
 import Metric from './metric.model';
@@ -111,17 +110,6 @@ export default class Statful {
         }
     }
 
-    /**
-     * Clear resources
-     */
-    static clearResources() {
-        try {
-            window.performance.clearResourceTimings();
-        } catch (ex) {
-            this.logger.error(ex);
-        }
-    }
-
     /**
      * Clear measures
      * @param {Array} measures - list of measures to clear (optional)

dist/statful.umd.js

@@ -12,7 +12,23 @@ describe('Statful Util Unit testing', () => {
         });
     });
 
-    it('should to send POST request', () => {
+    it('should not send POST request - Dry Run', () => {
+        statfulUtil = new StatfulUtil({
+            dryrun: true
+        });
+        jasmine.Ajax.install();
+
+        statfulUtil.sendData({
+            id: 1
+        });
+
+        const request = jasmine.Ajax.requests.mostRecent();
+        expect(request).toEqual(undefined);
+
+        jasmine.Ajax.uninstall();
+    });
+
+    it('should send POST request', () => {
         jasmine.Ajax.install();
 
         statfulUtil.sendData({

dist/statful.umd.min.js

@@ -186,20 +186,16 @@ describe('Statful Client Unit testing', () => {
     it('should clear all performance marks when calling clearMarks without arguments', () => {
         statful.initialize();
 
-        statful.registerMark('start_test');
-
         spyOn(window.performance, 'clearMarks');
 
         statful.clearMarks();
 
-        expect(window.performance.clearMarks).toHaveBeenCalled();
+        expect(window.performance.clearMarks).toHaveBeenCalledWith();
     });
 
     it('should clear performance marks when calling clearMarks with an Array of marks', () => {
         statful.initialize();
 
-        statful.registerMark('start_test');
-
         spyOn(window.performance, 'clearMarks');
 
         statful.clearMarks(['start_test']);
@@ -214,19 +210,17 @@ describe('Statful Client Unit testing', () => {
 
         statful.clearMeasures();
 
-        expect(window.performance.clearMeasures).toHaveBeenCalled();
+        expect(window.performance.clearMeasures).toHaveBeenCalledWith();
     });
 
-    it('should clear performance measures when calling clearMeasures with an Array of measures', () => {
+    it('should clear performance measures when calling clearMeasures with an Array of marks', () => {
         statful.initialize();
 
-        statful.registerMeasure('measure', 'metric');
-
-        spyOn(window.performance, 'clearMarks');
+        spyOn(window.performance, 'clearMeasures');
 
-        statful.clearMarks(['measure']);
+        statful.clearMeasures(['start_test']);
 
-        expect(window.performance.clearMarks).toHaveBeenCalledWith('measure');
+        expect(window.performance.clearMeasures).toHaveBeenCalledWith('start_test');
     });
 
     it('should add a performance mark when calling registerMark', () => {
@@ -239,6 +233,16 @@ describe('Statful Client Unit testing', () => {
         expect(window.performance.mark).toHaveBeenCalledWith('mark_test');
     });
 
+    it('should not add a performance mark when calling registerMark', () => {
+        statful.initialize();
+
+        spyOn(window.performance, 'mark');
+
+        statful.registerMark();
+
+        expect(window.performance.mark).not.toHaveBeenCalled();
+    });
+
     it('should add a performance measure when calling registerMeasure', () => {
         statful.initialize();