Removed workaround for mouseDown event in jest tests (#3673)

Author: vhuseinova-msft

change-beta/@azure-communication-react-d0492f53-bae0-4425-bbad-a83d039c3ad2.json

@@ -0,0 +1,9 @@
+{
+  "type": "none",
+  "area": "fix",
+  "workstream": "",
+  "comment": "Removed workaround for mouseDown event in jest tests. Removed duplicated strict mode from Calling sample app",
+  "packageName": "@azure/communication-react",
+  "email": "98852890+vhuseinova-msft@users.noreply.github.com",
+  "dependentChangeType": "none"
+}

change/@azure-communication-react-d0492f53-bae0-4425-bbad-a83d039c3ad2.json

@@ -0,0 +1,9 @@
+{
+  "type": "none",
+  "area": "fix",
+  "workstream": "",
+  "comment": "Removed workaround for mouseDown event in jest tests. Removed duplicated strict mode from Calling sample app",
+  "packageName": "@azure/communication-react",
+  "email": "98852890+vhuseinova-msft@users.noreply.github.com",
+  "dependentChangeType": "none"
+}

packages/react-components/src/components/SendBox.test.tsx

@@ -12,8 +12,6 @@ import { render, waitFor, act, fireEvent } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 /* @conditional-compile-remove(mention) */
 import { Mention } from './MentionPopover';
-/* @conditional-compile-remove(mention) */
-import { triggerMouseEvent } from './utils/testUtils';
 
 describe('SendBox strings should be localizable and overridable', () => {
   beforeAll(() => {
@@ -206,10 +204,6 @@ describe('Clicks/Touch should select mention', () => {
     // Select the suggestion
     await selectFirstMention();
     expect(input.value).toBe(value + suggestions[0].displayText);
-    // Fix for mousedown issue in userEvent when `document` become null unexpectedly
-    await act(async () => {
-      triggerMouseEvent(input, 'mousedown');
-    });
   };
 
   test('Mouse click on first word in mention should select mention', async () => {
@@ -292,10 +286,6 @@ describe('Clicks/Touch should select mention', () => {
       await userEvent.keyboard(' and');
     });
     const typedValue = trigger + suggestions[0].displayText + ' and';
-    // Fix for mousedown issue in userEvent when `document` become null unexpectedly
-    await act(async () => {
-      triggerMouseEvent(input, 'mousedown');
-    });
     // Triple click a letter at 14-th index
     await userEvent.pointer({
       keys: '[MouseLeft][MouseLeft][MouseLeft]',
@@ -446,10 +436,6 @@ describe('Keyboard events should be handled for mentions', () => {
     // Select the suggestion
     await selectMention(0);
     expect(input.value).toBe(value + suggestions[0].displayText);
-    // Fix for mousedown issue in userEvent when `document` become null unexpectedly
-    await act(async () => {
-      triggerMouseEvent(input, 'mousedown');
-    });
   };
 
   test('Keyboard navigation with arrows should navigate mention by words', async () => {

packages/react-components/src/components/utils/testUtils.tsx

@@ -56,14 +56,3 @@ export const createTestLocale = (testStrings: PartialDeep<ComponentStrings>): Co
   });
   return { strings };
 };
-
-/** @private */
-// Trigger a mouse event manually to fix mouseDown event in userEvent
-// when `document` become null unexpectedly
-// https://github.com/jestjs/jest/issues/12670
-export const triggerMouseEvent = (node: HTMLElement, eventType: string): void => {
-  const clickEvent = new MouseEvent(eventType, {
-    view: window
-  });
-  node.dispatchEvent(clickEvent);
-};

samples/Calling/src/app/App.tsx

@@ -204,18 +204,16 @@ const App = (): JSX.Element => {
         return <Spinner label={'Getting user credentials from server'} ariaLive="assertive" labelPosition="top" />;
       }
       return (
-        <React.StrictMode>
-          <CallScreen
-            token={token}
-            userId={userId}
-            displayName={displayName}
-            callLocator={callLocator}
-            /* @conditional-compile-remove(PSTN-calls) */
-            alternateCallerId={alternateCallerId}
-            /* @conditional-compile-remove(teams-identity-support) */
-            isTeamsIdentityCall={isTeamsCall}
-          />
-        </React.StrictMode>
+        <CallScreen
+          token={token}
+          userId={userId}
+          displayName={displayName}
+          callLocator={callLocator}
+          /* @conditional-compile-remove(PSTN-calls) */
+          alternateCallerId={alternateCallerId}
+          /* @conditional-compile-remove(teams-identity-support) */
+          isTeamsIdentityCall={isTeamsCall}
+        />
       );
     }
     default: