VIDEO-11239 Get current unit tests to pass.

Author: manjeshbhargav

package-lock.json

@@ -20,7 +20,10 @@ import useLocalVideoToggle from '../../../hooks/useLocalVideoToggle/useLocalVide
 jest.mock('../../../hooks/useFlipCameraToggle/useFlipCameraToggle');
 jest.mock('@material-ui/core/useMediaQuery');
 jest.mock('../../../state');
-jest.mock('../../../hooks/useVideoContext/useVideoContext', () => () => ({ room: { sid: 'mockRoomSid' } }));
+jest.mock('../../../hooks/useVideoContext/useVideoContext', () => () => ({
+  localTracks: [],
+  room: { sid: 'mockRoomSid' },
+}));
 jest.mock('../../../hooks/useIsRecording/useIsRecording');
 jest.mock('../../../hooks/useChatContext/useChatContext');
 jest.mock('../../../hooks/useLocalVideoToggle/useLocalVideoToggle');

src/components/MenuBar/Menu/Menu.test.tsx

@@ -25,6 +25,7 @@ mockUseVideoContext.mockImplementation(() => ({
   connect: mockConnect,
   isAcquiringLocalTracks: false,
   isConnecting: false,
+  localTracks: [],
 }));
 
 describe('the DeviceSelectionScreen component', () => {
@@ -38,6 +39,7 @@ describe('the DeviceSelectionScreen component', () => {
       connect: mockConnect,
       isAcquiringLocalTracks: false,
       isConnecting: true,
+      localTracks: [],
     }));
 
     const wrapper = shallow(<DeviceSelectionScreen name="test name" roomName="test room name" setStep={() => {}} />);
@@ -60,6 +62,7 @@ describe('the DeviceSelectionScreen component', () => {
       connect: mockConnect,
       isAcquiringLocalTracks: true,
       isConnecting: false,
+      localTracks: [],
     }));
 
     const wrapper = shallow(<DeviceSelectionScreen name="test name" roomName="test room name" setStep={() => {}} />);
@@ -82,6 +85,7 @@ describe('the DeviceSelectionScreen component', () => {
       connect: mockConnect,
       isAcquiringLocalTracks: false,
       isConnecting: false,
+      localTracks: [],
     }));
     mockUseAppState.mockImplementationOnce(() => ({ getToken: mockGetToken, isFetching: true }));
     const wrapper = shallow(<DeviceSelectionScreen name="test name" roomName="test room name" setStep={() => {}} />);

src/components/PreJoinScreens/DeviceSelectionScreen/DeviceSelectionScreen.test.tsx

@@ -61,7 +61,6 @@ describe('the VideoProvider component', () => {
       onError: expect.any(Function),
       connect: expect.any(Function),
       getLocalVideoTrack: expect.any(Function),
-      getLocalAudioTrack: expect.any(Function),
       removeLocalVideoTrack: expect.any(Function),
       isAcquiringLocalTracks: true,
       toggleScreenShare: expect.any(Function),

src/components/VideoProvider/index.test.tsx

@@ -1,12 +1,18 @@
 import { act, renderHook } from '@testing-library/react-hooks';
 import { getDeviceInfo, isPermissionDenied } from '../../../utils';
 import { SELECTED_AUDIO_INPUT_KEY, SELECTED_VIDEO_INPUT_KEY, DEFAULT_VIDEO_CONSTRAINTS } from '../../../constants';
+import { useAppState } from '../../../state';
 import useLocalTracks from './useLocalTracks';
 import Video from 'twilio-video';
 
+jest.mock('../../../state');
 jest.mock('../../../utils');
+
 const mockGetDeviceInfo = getDeviceInfo as jest.Mock<any>;
 const mockIsPermissionDenied = isPermissionDenied as jest.Mock<Promise<boolean>>;
+const mockUseAppState = useAppState as jest.Mock<any>;
+
+mockUseAppState.mockImplementation(() => ({ setIsKrispEnabled: false }));
 
 describe('the useLocalTracks hook', () => {
   beforeEach(() => {

src/components/VideoProvider/useLocalTracks/useLocalTracks.test.tsx

@@ -103,9 +103,11 @@ export default function useLocalTracks() {
         (hasSelectedAudioDevice
           ? {
               deviceId: { exact: selectedAudioDeviceId! },
-              noiseCancellationOptions: isKrispInstalled ? noiseCancellationOptions : {},
+              ...(isKrispInstalled ? { noiseCancellationOptions } : {}),
             }
-          : { noiseCancellationOptions: isKrispInstalled ? noiseCancellationOptions : {} }),
+          : isKrispInstalled
+          ? { noiseCancellationOptions }
+          : true),
     };
 
     return Video.createLocalTracks(localTrackConstraints)