Merge pull request #12324 from bbc/WSTEAMA-1237-release-reverb-pidgin…

…-TEST

WSTEAMA-1237: Launch Reverb tracking on TEST for Pidgin

src/app/hooks/useClickTrackerHandler/index.test.jsx

@@ -1,6 +1,6 @@
 /* eslint-disable jsx-a11y/click-events-have-key-events */
 /* eslint-disable jsx-a11y/no-static-element-interactions */
-import React from 'react';
+import React, { createContext } from 'react';
 import userEvent from '@testing-library/user-event';
 import { waitFor } from '@testing-library/dom';
 import { STORY_PAGE } from '#app/routes/utils/pageTypes';
@@ -13,6 +13,8 @@ import {
   act,
   fireEvent,
 } from '../../components/react-testing-library-with-providers';
+import * as serviceContextModule from '../../contexts/ServiceContext';
+
 import pidginData from './fixtureData/tori-51745682.json';
 import useClickTrackerHandler from '.';
 
@@ -89,6 +91,17 @@ beforeEach(() => {
     assign: jest.fn(),
     ...rest,
   };
+
+  jest.replaceProperty(
+    serviceContextModule,
+    'ServiceContext',
+    createContext({
+      atiAnalyticsProducerId: '70',
+      atiAnalyticsProducerName: 'PIDGIN',
+      service: 'pidgin',
+      useReverb: false,
+    }),
+  );
 });
 
 afterEach(() => {

src/app/hooks/useViewTracker/index.test.jsx

@@ -1,6 +1,6 @@
 /* eslint-disable no-console */
 
-import React from 'react';
+import React, { createContext } from 'react';
 import {
   renderHook,
   act,
@@ -11,7 +11,8 @@ import { RequestContextProvider } from '#contexts/RequestContext';
 import { ToggleContextProvider } from '#contexts/ToggleContext';
 import { STORY_PAGE } from '#app/routes/utils/pageTypes';
 import OPTIMIZELY_CONFIG from '#lib/config/optimizely';
-import { ServiceContextProvider } from '../../contexts/ServiceContext';
+import * as serviceContextModule from '../../contexts/ServiceContext';
+
 import useViewTracker from '.';
 
 import fixtureData from './fixtureData.json';
@@ -65,6 +66,17 @@ beforeEach(() => {
   jest.useFakeTimers();
   console.error = jest.fn();
   global.IntersectionObserver = IntersectionObserver;
+
+  jest.replaceProperty(
+    serviceContextModule,
+    'ServiceContext',
+    createContext({
+      atiAnalyticsProducerId: '70',
+      atiAnalyticsProducerName: 'PIDGIN',
+      service: 'pidgin',
+      useReverb: false,
+    }),
+  );
 });
 
 afterEach(() => {
@@ -98,13 +110,13 @@ const wrapper = ({ pageData, atiData, children, toggles = defaultToggles }) => (
     service="pidgin"
     pathname="/pidgin/tori-51745682"
   >
-    <ServiceContextProvider service="pidgin">
+    <serviceContextModule.ServiceContextProvider service="pidgin">
       <ToggleContextProvider toggles={toggles}>
         <EventTrackingContextProvider data={pageData} atiData={atiData}>
           {children}
         </EventTrackingContextProvider>
       </ToggleContextProvider>
-    </ServiceContextProvider>
+    </serviceContextModule.ServiceContextProvider>
   </RequestContextProvider>
 );
 

src/app/lib/analyticsUtils/sendBeacon/index.js

@@ -1,3 +1,4 @@
+import isLive from '../../utilities/isLive';
 import onClient from '../../utilities/onClient';
 import nodeLogger from '../../logger.node';
 import { ATI_LOGGING_ERROR } from '../../logger.const';
@@ -106,7 +107,7 @@ const callReverb = async eventDetails => {
 const sendBeacon = async (url, reverbBeaconConfig) => {
   if (onClient()) {
     try {
-      if (reverbBeaconConfig) {
+      if (!isLive() && reverbBeaconConfig) {
         const {
           params: { page, user },
           eventDetails,

src/app/lib/analyticsUtils/sendBeacon/index.test.js

@@ -27,7 +27,7 @@ describe('sendBeacon', () => {
   });
 
   afterEach(() => {
-    fetch.resetMocks();
+    jest.clearAllMocks();
   });
 
   it(`should fetch`, () => {
@@ -61,20 +61,76 @@ describe('sendBeacon', () => {
       },
     };
 
-    it('should call Reverb viewEvent if Reverb config is passed', async () => {
-      const sendBeacon = require('./index').default;
+    const originalProcessEnv = process.env;
 
-      await sendBeacon('https://foobar.com', reverbConfig);
+    afterEach(() => {
+      process.env = originalProcessEnv;
+    });
+
+    describe('LOCAL', () => {
+      beforeEach(() => {
+        process.env.SIMORGH_APP_ENV = 'local';
+      });
+
+      it('should call Reverb viewEvent if Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
+
+        await sendBeacon('https://foobar.com', reverbConfig);
+
+        expect(reverbMock.viewEvent).toHaveBeenCalledTimes(1);
+      });
 
-      expect(reverbMock.viewEvent).toHaveBeenCalledTimes(1);
+      it('should not call "fetch" if Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
+
+        await sendBeacon('https://foobar.com', reverbConfig);
+
+        expect(fetch).not.toHaveBeenCalled();
+      });
     });
 
-    it('should not call "fetch" if Reverb config is passed', async () => {
-      const sendBeacon = require('./index').default;
+    describe('TEST', () => {
+      beforeEach(() => {
+        process.env.SIMORGH_APP_ENV = 'test';
+      });
+
+      it('should call Reverb viewEvent if Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
+
+        await sendBeacon('https://foobar.com', reverbConfig);
+
+        expect(reverbMock.viewEvent).toHaveBeenCalledTimes(1);
+      });
+
+      it('should not call "fetch" if Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
 
-      await sendBeacon('https://foobar.com', reverbConfig);
+        await sendBeacon('https://foobar.com', reverbConfig);
 
-      expect(fetch).not.toHaveBeenCalled();
+        expect(fetch).not.toHaveBeenCalled();
+      });
+    });
+
+    describe('LIVE', () => {
+      beforeEach(() => {
+        process.env.SIMORGH_APP_ENV = 'live';
+      });
+
+      it('should not call Reverb viewEvent if Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
+
+        await sendBeacon('https://foobar.com', reverbConfig);
+
+        expect(reverbMock.viewEvent).not.toHaveBeenCalled();
+      });
+
+      it('should call "fetch" when Reverb config is passed', async () => {
+        const sendBeacon = require('./index').default;
+
+        await sendBeacon('https://foobar.com', reverbConfig);
+
+        expect(fetch).toHaveBeenCalledTimes(1);
+      });
     });
   });
 
@@ -86,10 +142,6 @@ describe('sendBeacon', () => {
       fetchResponse = Promise.reject(error);
     });
 
-    afterEach(() => {
-      jest.resetAllMocks();
-    });
-
     it(`should send error to logger`, async () => {
       const sendBeacon = require('./index').default;
 

src/app/lib/config/services/pidgin.ts

@@ -13,6 +13,7 @@ export const service: DefaultServiceConfig = {
     atiAnalyticsAppName: 'news-pidgin',
     atiAnalyticsProducerId: '70',
     atiAnalyticsProducerName: 'PIDGIN',
+    useReverb: true,
     chartbeatDomain: 'pidgin.bbc.co.uk',
     brandName: 'BBC News Pidgin',
     product: 'BBC News',

src/server/Document/component.test.jsx

@@ -15,6 +15,10 @@ describe('Document Component', () => {
   // uses values set in the .env file in lieu of mocked values
   dotenv.config();
 
+  beforeEach(() => {
+    process.env.SIMORGH_APP_ENV = 'local';
+  });
+
   afterEach(() => {
     process.env = originalProcessEnv;
   });