fix(telemetry): ⬇️ Reduce event syncs (#1296)

* fix(telemetry): ⬇️ Reduce event syncs Signed-off-by: Nishant Arora <[email protected]> * fix(telemetry): 🧪 Fix tests Signed-off-by: Nishant Arora <[email protected]> --------- Signed-off-by: Nishant Arora <[email protected]>
ipfs · Sep 25, 2023 · 6326cdb · 6326cdb
1 parent 9f852ef
commit 6326cdb
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/add-on/src/lib/trackers/requestTracker.ts b/add-on/src/lib/trackers/requestTracker.ts
@@ -2,14 +2,16 @@ import debug from 'debug'
 import type browser from 'webextension-polyfill'
 import { trackEvent } from '../telemetry.js'
 
+export const DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL = 1000 * 60 * 60
+
 export class RequestTracker {
   private readonly eventKey: 'url-observed' | 'url-resolved'
   private readonly flushInterval: number
   private readonly log: debug.Debugger & { error?: debug.Debugger }
   private lastSync: number = Date.now()
   private requestTypeStore: { [key in browser.WebRequest.ResourceType]?: number } = {}
 
-  constructor (eventKey: 'url-observed' | 'url-resolved', flushInterval = 1000 * 60 * 5) {
+  constructor (eventKey: 'url-observed' | 'url-resolved', flushInterval = DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL) {
     this.eventKey = eventKey
     this.log = debug(`ipfs-companion:request-tracker:${eventKey}`)
     this.log.error = debug(`ipfs-companion:request-tracker:${eventKey}:error`)

diff --git a/test/functional/lib/trackers/requestTrackers.test.ts b/test/functional/lib/trackers/requestTrackers.test.ts
@@ -2,7 +2,7 @@ import { expect } from 'chai';
 import sinon from 'sinon';
 import browser from 'sinon-chrome';
 import PatchedCountly from 'countly-sdk-web'
-import { RequestTracker } from './../../../../add-on/src/lib/trackers/requestTracker.js'
+import { DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL, RequestTracker } from './../../../../add-on/src/lib/trackers/requestTracker.js'
 
 const sinonSandBox = sinon.createSandbox()
 describe('lib/trackers/requestTracker', () => {
@@ -32,11 +32,11 @@ describe('lib/trackers/requestTracker', () => {
 
     it('should track a request', async () => {
       await requestTracker.track({ type: 'main_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
       sinon.assert.calledWith(countlySDKStub.add_event, {
         key: 'url-observed',
         count: 1,
-        dur: 300000,
+        dur: 3600000,
         segmentation: {
           main_frame: 1
         }
@@ -47,11 +47,11 @@ describe('lib/trackers/requestTracker', () => {
       await requestTracker.track({ type: 'main_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
       await requestTracker.track({ type: 'sub_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
       await requestTracker.track({ type: 'xmlHTTPRequest' } as browser.WebRequest.OnBeforeRequestDetailsType)
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
       sinon.assert.calledWith(countlySDKStub.add_event, {
         key: 'url-observed',
         count: 3,
-        dur: 300000,
+        dur: 3600000,
         segmentation: {
           main_frame: 1,
           sub_frame: 1,
@@ -61,7 +61,7 @@ describe('lib/trackers/requestTracker', () => {
     })
 
     it('should not send event if count is 0', async () => {
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
 
       sinon.assert.notCalled(countlySDKStub.add_event)
     })
@@ -79,11 +79,11 @@ describe('lib/trackers/requestTracker', () => {
 
     it('should track a request', async () => {
       await requestTracker.track({ type: 'main_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
       sinon.assert.calledWith(countlySDKStub.add_event, {
         key: 'url-resolved',
         count: 1,
-        dur: 300000,
+        dur: 3600000,
         segmentation: {
           main_frame: 1
         }
@@ -94,11 +94,11 @@ describe('lib/trackers/requestTracker', () => {
       await requestTracker.track({ type: 'main_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
       await requestTracker.track({ type: 'sub_frame' } as browser.WebRequest.OnBeforeRequestDetailsType)
       await requestTracker.track({ type: 'xmlHTTPRequest' } as browser.WebRequest.OnBeforeRequestDetailsType)
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
       sinon.assert.calledWith(countlySDKStub.add_event, {
         key: 'url-resolved',
         count: 3,
-        dur: 300000,
+        dur: 3600000,
         segmentation: {
           main_frame: 1,
           sub_frame: 1,
@@ -108,7 +108,7 @@ describe('lib/trackers/requestTracker', () => {
     })
 
     it('should not send event if count is 0', async () => {
-      clock.tick(1000 * 60 * 6)
+      clock.tick(DEFAULT_REQUEST_TRACKER_FLUSH_INTERVAL)
 
       sinon.assert.notCalled(countlySDKStub.add_event)
     })