[STREAM-850] - Suppressing benign error so it does not show up on UI and cause confusion (#940)

Author: maxwellmooney13

changelog.md

@@ -4,6 +4,8 @@ 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.0...HEAD)
+# Changed
+* [STREAM-850](https://inindca.atlassian.net/browse/STREAM-850) - Suppress `error.ininedgecontrol.session.inactive` errors to prevent consuming UIs from presenting benign errors
 
 # [v11.3.0](https://github.com/MyPureCloud/genesys-cloud-webrtc-sdk/compare/v11.2.3...v11.3.0)
 # Added

src/sessions/softphone-session-handler.ts

@@ -392,6 +392,9 @@ export class SoftphoneSessionHandler extends BaseSessionHandler {
     callState: ICallStateFromParticipant
   ): void {
     if (callState.errorInfo) {
+      if (callState.errorInfo.code === 'error.ininedgecontrol.session.inactive') {
+        return;
+      }
       this.debouncedEmitCallError(update, participant, callState);
     }
   }

test/unit/sessions/softphone-session-handler.test.ts

@@ -2550,6 +2550,22 @@ describe('checkForCallErrors', () => {
 
     expect(spy).not.toHaveBeenCalled();
   });
+
+  it('should return out if call error is inactive with "is inactive for" message', () => {
+    const update = { id: 'convoUpdate' };
+    const participant = { id: 'participantId' };
+    const callState = {
+      errorInfo: {
+        code: 'error.ininedgecontrol.session.inactive',
+        message: 'is inactive for 10 seconds'
+      }
+    };
+
+    const spy = jest.spyOn(handler, 'debouncedEmitCallError');
+    handler.checkForCallErrors(update as any, participant as any, callState as any);
+
+    expect(spy).not.toHaveBeenCalled();
+  });
 });
 
 describe('getActiveConversations', () => {