feat(calling): fixed-double-media-request

[adhmenon]

COMPLETES #< https://jira-eng-sjc12.cisco.com/jira/browse/CAI-7272 >

This pull request addresses

• During the mobius chaos testing, a small issue was found with the SDK during the answerOffer phase.
• The SDK was sending a media request ot mobius even after delete.
• The reason was because of a check we were not doing in our handleRoapError

by making the following changes

• Added a minor if check in handleRoapError to only post actual ERRORS from media core to mobius

Vidcast - https://app.vidcast.io/share/d7377177-c7aa-4e64-a44a-694e71daf0d3

Change Type

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Tooling change
• Internal code refactor

The following scenarios were tested

• Tested the same TC as mobius team did.
• Tested basic outgoing call.

The GAI Coding Policy And Copyright Annotation Best Practices

• GAI was not used (or, no additional notation is required)
• Code was generated entirely by GAI
• GAI was used to create a draft that was subsequently customized or modified
• Coder created a draft manually that was non-substantively modified by GAI (e.g., refactoring was performed by GAI on manually written code)
• Tool used for AI assistance (GitHub Copilot / Other - specify)
  • Github Copilot
  • Other - Please Specify
• This PR is related to
  • Feature
  • Defect fix
  • Tech Debt
  • Automation

I certified that

• I have read and followed contributing guidelines
• I discussed changes with code owners prior to submitting this pull request
• I have not skipped any automated checks
• All existing and new tests passed
• I have updated the documentation accordingly

---

Make sure to have followed the contributing guidelines before submitting.

[aws-amplify-us-east-2]

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-4556.d3m3l2kee0btzx.amplifyapp.com

[Kesari3008]

Why was this change needed ?

[adhmenon]

It was failing... and I removed it because it was not testing the correct flow.
It was tryng to test media error without mocking it... but can check this further.

[rarajes2]

Can we add a test to cover the case where it covers the if condition

[rarajes2]

LGTM. Existing test has the message.messageType as ERROR which covers the case