Tee video effects errors to state (#2729)

Author: JamesBurnside

change-beta/@azure-communication-react-000c0658-f64a-48ed-a318-8c7a7547fa39.json

@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "tee video effects errors to state",
+  "packageName": "@azure/communication-react",
+  "email": "2684369+JamesBurnside@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

change/@azure-communication-react-000c0658-f64a-48ed-a318-8c7a7547fa39.json

@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "tee video effects errors to state",
+  "packageName": "@azure/communication-react",
+  "email": "2684369+JamesBurnside@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

packages/calling-stateful-client/src/CallContext.ts

@@ -744,6 +744,21 @@ export class CallContext {
     };
   }
 
+  /**
+   * Tees direct errors to state.
+   * @remarks
+   * This is typically used for errors that are caught and intended to be shown to the user.
+   *
+   * @param error The raw error to report.
+   * @param target The error target to tee error to.
+   *
+   * @private
+   */
+  public teeErrorToState = (error: Error, target: CallErrorTarget): void => {
+    const callError = toCallError(target, error);
+    this.setLatestError(target, callError);
+  };
+
   private setLatestError(target: CallErrorTarget, error: CallError): void {
     this.modifyState((draft: CallClientState) => {
       draft.latestErrors[target] = error;

packages/calling-stateful-client/src/LocalVideoStreamVideoEffectsSubscriber.ts

@@ -67,10 +67,7 @@ export class LocalVideoStreamVideoEffectsSubscriber {
     this.updateEffectsState({
       isActive: false
     });
-    // TODO [video-background-effects]: future PR will tee error to state
-    console.error(
-      `LocalVideoStreamVideoEffectsSubscriber effectError: Method not fully implemented. Error needs teed to state. ${error}`
-    );
+    this._context.teeErrorToState(new Error(error.message), 'VideoEffectsFeature.startEffects');
   };
 
   private updateEffectsState = (newEffectsState: LocalVideoStreamVideoEffectsState): void => {