fix: audio recording and playing crashes [WPB-16687] [WPB-16835] 🍒

[github-actions]

This PR was automatically cherry-picked based on the following PR:

• fix: audio recording and playing crashes [WPB-16687] [WPB-16835] #3955

Original PR description:

---

WPB-16687 [Android] App crashes when second audio is recorded after playing the first before sending

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

This PR contains fixes for two issues related to recording and playing audio messages:
https://wearezeta.atlassian.net/browse/WPB-16687
https://wearezeta.atlassian.net/browse/WPB-16835

Issues

The app crashes when attempting to send an audio recording for the second time because the user played the 1st recording before sending it.
The app crashes when attempting to play an audio message after sending it.

Causes (Optional)

• MediaPlayer being released after closing or sending a message so it cannot be reused again
• audio focus not being abandoned correctly and receiving audiofocus loss when trying to record again
• trying to pause audio when there is no audio being played and MediaPlayer is not initialised
• asset file path is being changed from temporary to proper one after uploading and player keeps the temporary path

Solutions

• do not release MediaPlayer after sending or closing, it can be reused then (the same instance is persisted as long as the view model exists)
• abandon focus when closing the recording
• fix abandonExclusive function to abandon exclusiveFocusRequest and not regular focusRequest
• check if there is any current audio set before trying to pause or resume
• implement missing "uploading" audio message state so that view model is not injected while the path can change
• execute getting asset again when the resulting file does not exist anymore

Testing

How to Test

Steps to Reproduce:

1
-record an audio message
-play the recording before sending it
-send the audio message
-attempt to make another audio recording

2
-record an audio message
-send it
-attempt to play it from the audio message cell

Attachments (Optional)

record-play-send-and-record-again.mp4

play_after_uploading.mp4

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[codecov]

Codecov Report

Attention: Patch coverage is 32.25806% with 21 lines in your changes missing coverage. Please review.

Project coverage is 45.55%. Comparing base (e15daac) to head (36dd0aa).

Files with missing lines	Patch %	Lines
...oid/media/audiomessage/RecordAudioMessagePlayer.kt	0.00%	11 Missing ⚠️
...tions/model/messagetypes/audio/AudioMessageType.kt	0.00%	8 Missing ⚠️
...ire/android/media/audiomessage/AudioFocusHelper.kt	0.00%	1 Missing ⚠️
...dia/audiomessage/ConversationAudioMessagePlayer.kt	90.90%	0 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (32.25%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3956      +/-   ##
===========================================
- Coverage    45.57%   45.55%   -0.02%     
===========================================
  Files          518      518              
  Lines        17959    17971      +12     
  Branches      3041     3044       +3     
===========================================
+ Hits          8184     8186       +2     
- Misses        8936     8946      +10     
  Partials       839      839              

Files with missing lines	Coverage Δ
...essagecomposer/recordaudio/RecordAudioViewModel.kt	65.28% <ø> (+0.15%)	⬆️
...ire/android/media/audiomessage/AudioFocusHelper.kt	0.00% <0.00%> (ø)
...dia/audiomessage/ConversationAudioMessagePlayer.kt	66.04% <90.90%> (+0.38%)	⬆️
...tions/model/messagetypes/audio/AudioMessageType.kt	0.00% <0.00%> (ø)
...oid/media/audiomessage/RecordAudioMessagePlayer.kt	1.13% <0.00%> (-0.05%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e15daac...36dd0aa. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

Built wire-android-staging-compat-pr-3956.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3956.apk is available for download

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Built wire-android-staging-compat-pr-3956.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3956.apk is available for download