add more tests

Author: shaed-parkar

Diff for: VideoWeb/VideoWeb/ClientApp/src/app/services/conference/heartbeat.service.spec.ts

@@ -224,5 +224,16 @@ describe('HeartbeatService', () => {
             expect(heartbeatSpy.kill).toHaveBeenCalledTimes(1);
             expect(sut.heartbeat).toBeFalsy();
         });
+
+        it('should do nothing if the heartbeat is null', () => {
+            // Arrange
+            sut.heartbeat = null;
+
+            // Act
+            sut.stopHeartbeat();
+
+            // Assert
+            expect(loggerSpy.debug).toHaveBeenCalledWith(jasmine.stringMatching(/Couldn't stop the heartbeat as it didn't exist/));
+        });
     });
 });

Diff for: VideoWeb/VideoWeb/ClientApp/src/app/shared/self-test/self-test.component.spec.ts

@@ -795,4 +795,31 @@ describe('SelfTestComponent', () => {
             expect(retrieveSelfTestScoreSpy).toHaveBeenCalledTimes(1);
         }));
     });
+
+    describe('setupPexipClient', () => {
+        it('should subscribe to video call events and set up the pexip client', async () => {
+            // Arrange
+            videoCallServiceSpy.onCallSetup.and.returnValue(of(new CallSetup(new MediaStream())));
+            videoCallServiceSpy.onCallConnected.and.returnValue(of(new ConnectedCall(new MediaStream())));
+            videoCallServiceSpy.onError.and.returnValue(of(new CallError('error')));
+            videoCallServiceSpy.onCallDisconnected.and.returnValue(of(new DisconnectedCall('reason')));
+
+            const handleCallSetupSpy = spyOn(component, 'handleCallSetup');
+            const handleCallConnectedSpy = spyOn(component, 'handleCallConnected');
+            const handleCallErrorSpy = spyOn(component, 'handleCallError');
+            const handleCallDisconnectSpy = spyOn(component, 'handleCallDisconnect');
+
+            videoCallServiceSpy.setupClient.and.resolveTo();
+
+            // Act
+            await component.setupPexipClient();
+
+            // Assert
+            expect(handleCallSetupSpy).toHaveBeenCalledTimes(1);
+            expect(handleCallConnectedSpy).toHaveBeenCalledTimes(1);
+            expect(handleCallErrorSpy).toHaveBeenCalledTimes(1);
+            expect(handleCallDisconnectSpy).toHaveBeenCalledTimes(1);
+            expect(videoCallServiceSpy.setupClient).toHaveBeenCalledTimes(1);
+        });
+    });
 });