@webex/internal-plugin-metrics 3.12.0-next.30 → 3.12.0-next.32

package/test/unit/spec/call-diagnostic/call-diagnostic-metrics.ts

@@ -868,6 +868,7 @@ describe('internal-plugin-metrics', () => {
             identifiers: {correlationId: 'test-id'},
             name: 'client.alert.displayed',
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -889,6 +890,7 @@ describe('internal-plugin-metrics', () => {
             identifiers: {correlationId: 'test-id'},
             name: 'client.alert.displayed',
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -928,6 +930,42 @@ describe('internal-plugin-metrics', () => {
       });
     });
 
+    describe('#getUserActivation', () => {
+      let originalDescriptor;
+
+      beforeEach(() => {
+        originalDescriptor = Object.getOwnPropertyDescriptor(global, 'navigator');
+      });
+
+      afterEach(() => {
+        if (originalDescriptor) {
+          Object.defineProperty(global, 'navigator', originalDescriptor);
+        } else {
+          delete (global as any).navigator;
+        }
+      });
+
+      it('should return the userActivation state when navigator.userActivation is available', () => {
+        Object.defineProperty(global, 'navigator', {
+          value: {userActivation: {hasBeenActive: true, isActive: false}},
+          configurable: true,
+          writable: true,
+        });
+
+        assert.deepEqual(cd.getUserActivation(), {hasBeenActive: true, isActive: false});
+      });
+
+      it('should return undefined when navigator.userActivation is not available', () => {
+        Object.defineProperty(global, 'navigator', {
+          value: {},
+          configurable: true,
+          writable: true,
+        });
+
+        assert.isUndefined(cd.getUserActivation());
+      });
+    });
+
     describe('#submitClientEvent', () => {
       it('should submit client event successfully with meetingId', () => {
         const prepareDiagnosticEventSpy = sinon.spy(cd, 'prepareDiagnosticEvent');
@@ -986,6 +1024,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1017,6 +1056,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1059,6 +1099,7 @@ describe('internal-plugin-metrics', () => {
               webClientPreload: undefined,
               isVipMeeting: false,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
             eventId: 'my-fake-id',
             origin: {
@@ -1137,6 +1178,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1168,6 +1210,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1210,6 +1253,7 @@ describe('internal-plugin-metrics', () => {
               webClientPreload: undefined,
               isVipMeeting: false,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
             eventId: 'my-fake-id',
             origin: {
@@ -1289,6 +1333,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1321,6 +1366,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1364,6 +1410,7 @@ describe('internal-plugin-metrics', () => {
               webClientPreload: undefined,
               isVipMeeting: false,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
             eventId: 'my-fake-id',
             origin: {
@@ -1442,6 +1489,7 @@ describe('internal-plugin-metrics', () => {
             webexSubServiceType: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1474,6 +1522,7 @@ describe('internal-plugin-metrics', () => {
             webexSubServiceType: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1517,6 +1566,7 @@ describe('internal-plugin-metrics', () => {
               webexSubServiceType: undefined,
               isVipMeeting: false,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
             eventId: 'my-fake-id',
             origin: {
@@ -1597,6 +1647,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1631,6 +1682,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1676,6 +1728,7 @@ describe('internal-plugin-metrics', () => {
               webClientPreload: undefined,
               isVipMeeting: false,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
             eventId: 'my-fake-id',
             origin: {
@@ -1804,6 +1857,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1829,6 +1883,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -1906,6 +1961,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -1936,6 +1992,7 @@ describe('internal-plugin-metrics', () => {
               telemetryOptOut: undefined,
               webClientPreload: undefined,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
           },
           options.preLoginId
@@ -2001,6 +2058,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -2033,6 +2091,7 @@ describe('internal-plugin-metrics', () => {
               emailInput: 'current',
               webClientPreload: undefined,
               isAutomatedUser: false,
+              userActivation: undefined,
             },
           },
           options.preLoginId
@@ -2081,6 +2140,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2139,6 +2199,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2197,6 +2258,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: true,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -2222,6 +2284,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: true,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2293,6 +2356,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -2325,6 +2389,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2396,6 +2461,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: true,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           options
         );
@@ -2428,6 +2494,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: true,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2505,6 +2572,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2590,6 +2658,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2665,6 +2734,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2742,6 +2812,7 @@ describe('internal-plugin-metrics', () => {
             name: 'client.alert.displayed',
             webClientPreload: undefined,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -2830,6 +2901,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
           },
           eventId: 'my-fake-id',
           origin: {
@@ -3122,7 +3194,6 @@ describe('internal-plugin-metrics', () => {
         assert.calledWith(requestStub, {
           eventPayload: {event: 'test'},
           type: ['diagnostic-event'],
-          markTelemetryOptOutOnResponse: true,
         });
       });
     });
@@ -4252,6 +4323,7 @@ describe('internal-plugin-metrics', () => {
                     webClientPreload: undefined,
                     isVipMeeting: false,
                     isAutomatedUser: false,
+                    userActivation: undefined,
                   },
                   eventId: 'my-fake-id',
                   origin: {
@@ -4422,7 +4494,6 @@ describe('internal-plugin-metrics', () => {
         assert.calledWith(requestStub, {
           eventPayload: {event: 'test'},
           type: ['diagnostic-event'],
-          markTelemetryOptOutOnResponse: true,
         });
       });
     });
@@ -4661,6 +4732,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
             meetingSummaryInfo: {
               featureName: 'syncSystemMuteStatus',
               featureActions: [
@@ -4705,6 +4777,7 @@ describe('internal-plugin-metrics', () => {
             webClientPreload: undefined,
             isVipMeeting: false,
             isAutomatedUser: false,
+            userActivation: undefined,
             meetingSummaryInfo: {
               featureName: 'syncSystemMuteStatus',
               featureActions: [

package/test/unit/spec/prelogin-metrics-batcher.ts

@@ -94,7 +94,6 @@ describe('internal-plugin-metrics', () => {
           },
         });
         assert.deepEqual(webexRequestArgs.body.metrics[0].type, ['diagnostic-event']);
-        assert.equal(webexRequestArgs.body.metrics[0].markTelemetryOptOutOnResponse, true);
         assert.lengthOf(
           webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.queue,
           0
@@ -314,7 +313,7 @@ describe('internal-plugin-metrics', () => {
     });
 
     describe('#submitHttpRequest', () => {
-      it('calls webex.request with the correct parameters and then it calls handleHttpResponseStatus on success', async () => {
+      it('calls webex.request with the correct parameters', async () => {
         const payload = [
           {
             eventPayload: {event: 'my.event'},
@@ -327,18 +326,11 @@ describe('internal-plugin-metrics', () => {
         );
         webex.request = sinon.stub().resolves({statusCode: 200});
 
-        const handleHttpResponseStatusSpy = sinon.spy(
-          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher,
-          'handleHttpResponseStatus'
-        );
-
         const promise =
           webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.submitHttpRequest(
             payload
           );
 
-        assert.deepEqual(handleHttpResponseStatusSpy.getCalls().length, 0);
-
         await flushPromises();
 
         clock.tick(config.metrics.batcherWait);
@@ -361,117 +353,6 @@ describe('internal-plugin-metrics', () => {
           service: 'metrics',
           waitForServiceTimeout: 30,
         });
-
-        assert.deepEqual(handleHttpResponseStatusSpy.getCalls().length, 1);
-
-        assert.deepEqual(handleHttpResponseStatusSpy.args[0][0], 200);
-        assert.deepEqual(handleHttpResponseStatusSpy.args[0][1], payload);
-      });
-
-      it('it calls handleHttpResponseStatus on failure', async () => {
-        const payload = [
-          {
-            eventPayload: {event: 'my.event'},
-            type: ['diagnostic-event'],
-          },
-        ];
-
-        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.savePreLoginId(
-          preLoginId
-        );
-        webex.request = sinon.stub().rejects({statusCode: 503});
-
-        const handleHttpResponseStatusSpy = sinon.spy(
-          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher,
-          'handleHttpResponseStatus'
-        );
-
-        const promise =
-          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.submitHttpRequest(
-            payload
-          );
-
-        assert.deepEqual(handleHttpResponseStatusSpy.getCalls().length, 0);
-
-        await flushPromises();
-
-        clock.tick(config.metrics.batcherWait);
-
-        let error;
-
-        try {
-          await promise;
-        } catch (err) {
-          error = err;
-        }
-
-        assert.deepEqual(error.statusCode, 503);
-
-        assert.deepEqual(handleHttpResponseStatusSpy.getCalls().length, 1);
-
-        assert.deepEqual(handleHttpResponseStatusSpy.args[0][0], 503);
-        assert.deepEqual(handleHttpResponseStatusSpy.args[0][1], payload);
-      });
-    })
-
-    describe('#handleHttpResponseStatus', () => {
-      let setIsTelemetryOptOutAutomaticStub;
-
-      beforeEach(() => {
-        setIsTelemetryOptOutAutomaticStub = sinon.stub(
-          webex.internal.newMetrics.callDiagnosticMetrics,
-          'setIsTelemetryOptOutAutomatic'
-        );
-      });
-
-      [201, 400, 503, undefined].forEach((statusCode) => {
-        it(`does not call setIsTelemetryOptOutAutomatic when shouldMark is true and statusCode is ${statusCode}`, () => {
-          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.handleHttpResponseStatus(
-            statusCode,
-            [{markTelemetryOptOutOnResponse: true}]
-          );
-
-          assert.notCalled(setIsTelemetryOptOutAutomaticStub);
-        });
-      });
-
-      it('calls setIsTelemetryOptOutAutomatic(true) when statusCode is 200 and markTelemetryOptOutOnResponse is true', () => {
-        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.handleHttpResponseStatus(
-          200,
-          [{markTelemetryOptOutOnResponse: true}]
-        );
-
-        assert.calledOnce(setIsTelemetryOptOutAutomaticStub);
-        assert.calledWithExactly(setIsTelemetryOptOutAutomaticStub, true);
-      });
-
-      [200, 201, 400, 503, undefined].forEach((statusCode) => {
-        it(`does not call setIsTelemetryOptOutAutomatic when shouldMark is false (statusCode: ${statusCode})`, () => {
-          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.handleHttpResponseStatus(
-            statusCode,
-            [{markTelemetryOptOutOnResponse: false}]
-          );
-
-          assert.notCalled(setIsTelemetryOptOutAutomaticStub);
-        });
-      });
-
-      it('does not call setIsTelemetryOptOutAutomatic when payload is empty', () => {
-        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.handleHttpResponseStatus(
-          200,
-          []
-        );
-
-        assert.notCalled(setIsTelemetryOptOutAutomaticStub);
-      });
-
-      it('does not call setIsTelemetryOptOutAutomatic when payload is not an array', () => {
-        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.handleHttpResponseStatus(
-          200,
-          null
-        );
-
-        assert.notCalled(setIsTelemetryOptOutAutomaticStub);
       });
     });
   });