rollbar 2.26.2 → 2.26.4

package/test/server.transport.test.js

@@ -1,4 +1,4 @@
-"use strict";
+'use strict';
 
 var assert = require('assert');
 var util = require('util');
@@ -6,101 +6,156 @@ var vows = require('vows');
 var Transport = require('../src/server/transport');
 var t = new Transport();
 
-vows.describe('transport')
+vows
+  .describe('transport')
   .addBatch({
-    'post': {
+    post: {
       'base data': {
-        topic: function() {
+        topic: function () {
           return {
             accessToken: 'abc123',
             options: {},
             payload: {
               access_token: 'abc123',
-              data: {a: 1}
-            }
+              data: { a: 1 },
+            },
           };
         },
         'with no payload': {
-          topic: function(data) {
-            var factory = transportFactory(null, '{"err": null, "result":"all good"}');
-            t.post(data.accessToken, data.options, null, this.callback, factory);
+          topic: function (data) {
+            var factory = transportFactory(
+              null,
+              '{"err": null, "result":"all good"}',
+            );
+            t.post(
+              data.accessToken,
+              data.options,
+              null,
+              this.callback,
+              factory,
+            );
           },
-          'should have an error': function(err, resp) {
+          'should have an error': function (err, resp) {
             assert.ok(err);
-          }
+          },
         },
         'with a payload and no error': {
-          topic: function(data) {
-            var factory = transportFactory(null, '{"err": null, "result":{"uuid":"42def", "message":"all good"}}',
-                function() {
-                  assert.equal(this.options.headers['Content-Type'], 'application/json');
-                  assert.isNumber(this.options.headers['Content-Length']);
-                  assert(this.options.headers['Content-Length'] > 0);
-                  assert.equal(this.options.headers['X-Rollbar-Access-Token'], data.accessToken);
-                });
-            t.post(data.accessToken, data.options, data.payload, this.callback, factory);
+          topic: function (data) {
+            var factory = transportFactory(
+              null,
+              '{"err": null, "result":{"uuid":"42def", "message":"all good"}}',
+              function () {
+                assert.equal(
+                  this.options.headers['Content-Type'],
+                  'application/json',
+                );
+                assert.isNumber(this.options.headers['Content-Length']);
+                assert(this.options.headers['Content-Length'] > 0);
+                assert.equal(
+                  this.options.headers['X-Rollbar-Access-Token'],
+                  data.accessToken,
+                );
+              },
+            );
+            t.post(
+              data.accessToken,
+              data.options,
+              data.payload,
+              this.callback,
+              factory,
+            );
           },
-          'should not error': function(err, resp) {
+          'should not error': function (err, resp) {
             assert.ifError(err);
           },
-          'should have the right response data': function(err, resp) {
+          'should have the right response data': function (err, resp) {
             assert.equal(resp.message, 'all good');
-          }
+          },
         },
         'with a payload and an error in the response': {
-          topic: function(data) {
-            var factory = transportFactory(null, '{"err": "bork", "message":"things broke"}',
-                function() {
-                  assert.equal(this.options.headers['Content-Type'], 'application/json');
-                  assert.isNumber(this.options.headers['Content-Length']);
-                  assert(this.options.headers['Content-Length'] > 0);
-                  assert.equal(this.options.headers['X-Rollbar-Access-Token'], data.accessToken);
-                });
-            t.post(data.accessToken, data.options, data.payload, this.callback, factory);
+          topic: function (data) {
+            var factory = transportFactory(
+              null,
+              '{"err": "bork", "message":"things broke"}',
+              function () {
+                assert.equal(
+                  this.options.headers['Content-Type'],
+                  'application/json',
+                );
+                assert.isNumber(this.options.headers['Content-Length']);
+                assert(this.options.headers['Content-Length'] > 0);
+                assert.equal(
+                  this.options.headers['X-Rollbar-Access-Token'],
+                  data.accessToken,
+                );
+              },
+            );
+            t.post(
+              data.accessToken,
+              data.options,
+              data.payload,
+              this.callback,
+              factory,
+            );
           },
-          'should error': function(err, resp) {
+          'should error': function (err, resp) {
             assert.ok(err);
           },
-          'should have the message somewhere': function(err, resp) {
+          'should have the message somewhere': function (err, resp) {
             assert.match(err.message, /things broke/);
           },
-          'should not have a response': function(err, resp) {
+          'should not have a response': function (err, resp) {
             assert.ifError(resp);
-          }
+          },
         },
         'with a payload and an error during sending': {
-          topic: function(data) {
-            var factory = transportFactory(new Error('bork'), null,
-                function() {
-                  assert.equal(this.options.headers['Content-Type'], 'application/json');
-                  assert.isNumber(this.options.headers['Content-Length']);
-                  assert(this.options.headers['Content-Length'] > 0);
-                  assert.equal(this.options.headers['X-Rollbar-Access-Token'], data.accessToken);
-                });
-            t.post(data.accessToken, data.options, data.payload, this.callback, factory);
+          topic: function (data) {
+            var factory = transportFactory(
+              new Error('bork'),
+              null,
+              function () {
+                assert.equal(
+                  this.options.headers['Content-Type'],
+                  'application/json',
+                );
+                assert.isNumber(this.options.headers['Content-Length']);
+                assert(this.options.headers['Content-Length'] > 0);
+                assert.equal(
+                  this.options.headers['X-Rollbar-Access-Token'],
+                  data.accessToken,
+                );
+              },
+            );
+            t.post(
+              data.accessToken,
+              data.options,
+              data.payload,
+              this.callback,
+              factory,
+            );
           },
-          'should error': function(err, resp) {
+          'should error': function (err, resp) {
             assert.ok(err);
           },
-          'should have the message somewhere': function(err, resp) {
+          'should have the message somewhere': function (err, resp) {
             assert.match(err.message, /bork/);
           },
-          'should not have a response': function(err, resp) {
+          'should not have a response': function (err, resp) {
             assert.ifError(resp);
-          }
-        }
+          },
+        },
       },
       'with rate limiting': {
-        topic: function() {
+        topic: function () {
           return new Transport();
         },
-        'should transmit non-rate limited requests': function(t) {
+        'should transmit non-rate limited requests': function (t) {
           var response = new TestResponse({
             statusCode: 200,
             headers: {
               'x-rate-limit-remaining': '1',
-              'x-rate-limit-remaining-seconds': '100'
-            }
+              'x-rate-limit-remaining-seconds': '100',
+            },
           });
           var error;
 
@@ -108,35 +163,48 @@ vows.describe('transport')
 
           t.handleResponse(response);
 
-          var factory = transportFactory(null, '{"err": null, "result": "all good"}');
-          t.post('token', {}, 'payload', function(err){ error = err; }, factory);
+          var factory = transportFactory(
+            null,
+            '{"err": null, "result": "all good"}',
+          );
+          t.post(
+            'token',
+            {},
+            'payload',
+            function (err) {
+              error = err;
+            },
+            factory,
+          );
 
           assert.equal(error, null);
           assert.isTrue(Math.floor(Date.now() / 1000) >= t.rateLimitExpires);
         },
-        'should drop rate limited requests and set timeout': function(t) {
+        'should drop rate limited requests and set timeout': function (t) {
           var response = new TestResponse({
             statusCode: 429,
             headers: {
               'x-rate-limit-remaining': '0',
-              'x-rate-limit-remaining-seconds': '100'
-            }
+              'x-rate-limit-remaining-seconds': '100',
+            },
           });
           var error;
 
           t.handleResponse(response);
 
-          t.post('token', {}, 'payload', function(err){ error = err; });
+          t.post('token', {}, 'payload', function (err) {
+            error = err;
+          });
 
           assert.match(error.message, /Exceeded rate limit/);
           assert.isTrue(Math.floor(Date.now() / 1000) < t.rateLimitExpires);
-        }
-      }
-    }
+        },
+      },
+    },
   })
-  .export(module, {error: false});
+  .export(module, { error: false });
 
-var TestTransport = function(options, error, response, assertions) {
+var TestTransport = function (options, error, response, assertions) {
   this.options = options;
   this.error = error;
   this.response = response;
@@ -144,7 +212,7 @@ var TestTransport = function(options, error, response, assertions) {
   this.requestCallback = null;
   this.assertions = assertions;
 };
-var TestRequest = function(error, response, transport) {
+var TestRequest = function (error, response, transport) {
   this.error = error;
   this.responseData = response;
   this.data = [];
@@ -152,18 +220,18 @@ var TestRequest = function(error, response, transport) {
   this.transport = transport;
   this.response = null;
 };
-TestTransport.prototype.request = function(opts, cb) {
+TestTransport.prototype.request = function (opts, cb) {
   this.requestOpts = opts;
   this.requestCallback = cb;
   return new TestRequest(this.error, this.response, this);
 };
-TestRequest.prototype.write = function(data) {
+TestRequest.prototype.write = function (data) {
   this.data.push(data);
 };
-TestRequest.prototype.on = function(event, cb) {
+TestRequest.prototype.on = function (event, cb) {
   this.events[event] = cb;
 };
-TestRequest.prototype.end = function() {
+TestRequest.prototype.end = function () {
   if (this.transport.assertions) {
     this.transport.assertions();
   }
@@ -182,20 +250,20 @@ TestRequest.prototype.end = function() {
     }
   }
 };
-var TestResponse = function(options = {}) {
+var TestResponse = function (options = {}) {
   this.encoding = null;
   this.events = {};
   this.headers = options.headers || {};
   this.statusCode = options.statusCode || 200;
 };
-TestResponse.prototype.setEncoding = function(s) {
+TestResponse.prototype.setEncoding = function (s) {
   this.encoding = s;
-}
-TestResponse.prototype.on = function(event, cb) {
+};
+TestResponse.prototype.on = function (event, cb) {
   this.events[event] = cb;
 };
-var transportFactory = function(error, response, assertions) {
-  return function(options) {
+var transportFactory = function (error, response, assertions) {
+  return function (options) {
     return new TestTransport(options, error, response, assertions);
   };
 };

package/test/telemetry.test.js

@@ -5,8 +5,8 @@
 
 var Telemeter = require('../src/telemetry');
 
-describe('Telemetry()', function() {
-  it('should have all of the expected methods', function(done) {
+describe('Telemetry()', function () {
+  it('should have all of the expected methods', function (done) {
     var options = {};
     var t = new Telemeter(options);
     expect(t).to.have.property('copyEvents');
@@ -20,12 +20,12 @@ describe('Telemetry()', function() {
   });
 });
 
-describe('capture', function() {
-  it('should return a valid telemetry event', function(done) {
+describe('capture', function () {
+  it('should return a valid telemetry event', function (done) {
     var options = {};
     var t = new Telemeter(options);
     var now = +new Date();
-    var event = t.capture('network', {url: 'a.com'}, 'debug');
+    var event = t.capture('network', { url: 'a.com' }, 'debug');
     expect(event.timestamp_ms - now).to.be.below(500);
     expect(event.type).to.equal('network');
     expect(event.level).to.equal('debug');
@@ -35,10 +35,10 @@ describe('capture', function() {
   });
 });
 
-describe('captureEvent', function() {
-  it('should return a valid telemetry event', function(done) {
+describe('captureEvent', function () {
+  it('should return a valid telemetry event', function (done) {
     var t = new Telemeter();
-    var event = t.captureEvent('log', {message: 'bar'}, 'info');
+    var event = t.captureEvent('log', { message: 'bar' }, 'info');
     expect(event.type).to.equal('log');
     expect(event.level).to.equal('info');
     expect(event.body.message).to.equal('bar');
@@ -47,32 +47,41 @@ describe('captureEvent', function() {
   });
 });
 
-describe('filterTelemetry', function() {
-  it('should filter out events that don\'t match the test', function(done) {
+describe('filterTelemetry', function () {
+  it("should filter out events that don't match the test", function (done) {
     var options = {
-      filterTelemetry: function(e) {
-        return e.type === 'network'
-          && (e.body.subtype === 'xhr' || e.body.subtype === 'fetch')
-          && e.body.url.indexOf('https://spammer.com') === 0;
-      }
+      filterTelemetry: function (e) {
+        return (
+          e.type === 'network' &&
+          (e.body.subtype === 'xhr' || e.body.subtype === 'fetch') &&
+          e.body.url.indexOf('https://spammer.com') === 0
+        );
+      },
     };
     var t = new Telemeter(options);
-    var evt = t.capture('network', {url: 'https://spammer.com', subtype: 'xhr'}, 'debug');
+    var evt = t.capture(
+      'network',
+      { url: 'https://spammer.com', subtype: 'xhr' },
+      'debug',
+    );
     expect(evt).to.be(false);
 
     done();
   });
 
-  it('should filter out events in copy even if they are modified after capture', function(done) {
+  it('should filter out events in copy even if they are modified after capture', function (done) {
     var options = {
-      filterTelemetry: function(e) {
-        return e.type === 'network'
-          && e.body.statusCode === 200;
-      }
+      filterTelemetry: function (e) {
+        return e.type === 'network' && e.body.statusCode === 200;
+      },
     };
     var t = new Telemeter(options);
-    var evt = t.capture('network', {url: 'https://spammer.com'}, 'debug');
-    var evt2 = t.capture('network', {url: 'https://spammer.com', statusCode: 404}, 'debug');
+    var evt = t.capture('network', { url: 'https://spammer.com' }, 'debug');
+    var evt2 = t.capture(
+      'network',
+      { url: 'https://spammer.com', statusCode: 404 },
+      'debug',
+    );
     expect(evt).not.to.be(false);
     expect(evt2).not.to.be(false);
     var events = t.copyEvents();
@@ -88,50 +97,50 @@ describe('filterTelemetry', function() {
   });
 });
 
-describe('configure', function() {
-  it('should truncate events to new max', function(done) {
-    var options = {maxTelemetryEvents: 5};
+describe('configure', function () {
+  it('should truncate events to new max', function (done) {
+    var options = { maxTelemetryEvents: 5 };
     var t = new Telemeter(options);
 
     for (var i = 0; i < 7; i++) {
-      t.capture('network', {url: 'a.com'}, 'debug');
+      t.capture('network', { url: 'a.com' }, 'debug');
     }
 
     expect(t.queue.length).to.equal(5);
-    t.configure({maxTelemetryEvents: 3});
+    t.configure({ maxTelemetryEvents: 3 });
     expect(t.queue.length).to.equal(3);
     done();
   });
-  it('should lengthen events to allow new max', function(done) {
-    var options = {maxTelemetryEvents: 3};
+  it('should lengthen events to allow new max', function (done) {
+    var options = { maxTelemetryEvents: 3 };
     var t = new Telemeter(options);
 
     for (var i = 0; i < 7; i++) {
-      t.capture('network', {url: 'a.com'}, 'debug');
+      t.capture('network', { url: 'a.com' }, 'debug');
     }
 
     expect(t.queue.length).to.equal(3);
-    t.configure({maxTelemetryEvents: 5});
+    t.configure({ maxTelemetryEvents: 5 });
     expect(t.queue.length).to.equal(3);
     for (var i = 0; i < 7; i++) {
-      t.capture('network', {url: 'a.com'}, 'debug');
+      t.capture('network', { url: 'a.com' }, 'debug');
     }
     expect(t.queue.length).to.equal(5);
     done();
   });
-  it('does not drop existing options that are not passed to configure', function(done) {
-    var options = {maxTelemetryEvents: 3};
+  it('does not drop existing options that are not passed to configure', function (done) {
+    var options = { maxTelemetryEvents: 3 };
     var t = new Telemeter(options);
 
     for (var i = 0; i < 7; i++) {
-      t.capture('network', {url: 'a.com'}, 'debug');
+      t.capture('network', { url: 'a.com' }, 'debug');
     }
 
     expect(t.queue.length).to.equal(3);
     t.configure({});
     expect(t.queue.length).to.equal(3);
     for (var i = 0; i < 7; i++) {
-      t.capture('network', {url: 'a.com'}, 'debug');
+      t.capture('network', { url: 'a.com' }, 'debug');
     }
     expect(t.queue.length).to.equal(3);
     done();