STREAM-777: try cleanup

Author: Aidan Zimmermann

.gitignore

@@ -14,15 +14,12 @@ test/test-pages/*/index.html
 
 *.tgz
 .ignore
-.DS_Store
 
 ## vscode specific files
 /.vscode
 
-## webstorm specific files
-/.idea
-
 ## jetbrains specific files
+/.idea
 genesys-cloud-webrtc-sdk.iml
 
 # tap specific files
@@ -32,3 +29,5 @@ genesys-cloud-webrtc-sdk.iml
 /.npmrc
 
 /test-results
+
+.DS_Store

changelog.md

@@ -4,7 +4,6 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 # [Unreleased](https://github.com/MyPureCloud/genesys-cloud-webrtc-sdk/compare/v11.3.3...HEAD)
-
 ### Fixed
 * [STREAM-978](https://inindca.atlassian.net/browse/STREAM-978) - When the RTCPeerConnection is in a 'failed' state, clean up the session. Filter out voicemail participants from conversation update events. Fix potential TypeError when fetching conversations.
 * [STREAM-987](https://inindca.atlassian.net/browse/STREAM-987) - Demo app: Memoize conversations so they don't change every render when nothing has changed

test/unit/client.test.ts

@@ -329,80 +329,6 @@ describe('Client', () => {
         expect(sessionManagerMock.startSession).not.toHaveBeenCalled();
       }
     });
-
-    it('should allow video conference with JWT authentication', async () => {
-      const testJwt = 'test.jwt.token';
-      sdk = constructSdk({ jwt: testJwt });
-
-      sdk._personDetails = {
-        id: 'test-user-id',
-        name: 'Test User',
-        chat: {
-          jabberId: 'test-user@test.com'
-        }
-      };
-
-      sessionManagerMock.startSession.mockResolvedValue({});
-      await sdk.startVideoConference('test-room@conference.com');
-
-      expect(sessionManagerMock.startSession).toBeCalledWith({
-        jid: 'test-room@conference.com',
-        sessionType: SessionTypes.collaborateVideo
-      });
-    });
-
-    it('should include JWT in video session request', async () => {
-      const testJwt = 'test.jwt.token';
-      const mockDecodedJwt = {
-        data: {
-          jid: 'test-user@test.com',
-          conversationId: 'test-conversation-id',
-          sourceCommunicationId: 'test-source-comm-id'
-        }
-      };
-
-      jwtDecodeSpy.mockReturnValue(mockDecodedJwt);
-
-      sdk = constructSdk({ jwt: testJwt });
-      const handler = new VideoSessionHandler(sdk, sessionManagerMock);
-
-      sdk._personDetails = {
-        id: 'test-user-id',
-        name: 'Test User',
-        chat: {
-          jabberId: 'test-user@test.com'
-        }
-      };
-
-      const mockInitiateRtcSession = jest.fn().mockResolvedValue(undefined);
-      sdk._streamingConnection = {
-        webrtcSessions: {
-          initiateRtcSession: mockInitiateRtcSession
-        },
-        disconnect: jest.fn().mockResolvedValue(undefined)
-      } as any;
-
-      const requestApiSpy = jest.spyOn(utils, 'requestApi');
-
-      const result = await handler.startSession({
-        jid: 'test-room@conference.com',
-        sessionType: SessionTypes.collaborateVideo
-      });
-
-      expect(mockInitiateRtcSession).toHaveBeenCalledWith({
-        jid: mockDecodedJwt.data.jid,
-        conversationId: mockDecodedJwt.data.conversationId,
-        sourceCommunicationId: mockDecodedJwt.data.sourceCommunicationId,
-        mediaPurpose: SessionTypes.collaborateVideo,
-        sessionType: SessionTypes.collaborateVideo
-      });
-
-      expect(result).toEqual({
-        conversationId: mockDecodedJwt.data.conversationId
-      });
-
-      expect(requestApiSpy).not.toHaveBeenCalled();
-    });
   });
 
   describe('startLiveMonitoringSession()', () => {
@@ -424,6 +350,7 @@ describe('Client', () => {
     });
   });
 
+
   describe('startScreenShare()', () => {
     it('should reject if authenticated user', async () => {
       sdk = constructSdk();