feat: discover legal hold when sending message [WPB-5999]

[saleniuk]

WPB-5999 Discover legal hold when sending messages

---

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?

Issues

We need to discover missing/deleted legal hold clients and changes in legal hold for conversation on Android when sending messages and handle this change properly by showing system messages. If the legal hold status for conversation changed when sending a message, the user needs to be informed about that and decide whether he/she still wants to resend the message.

Solutions

Handle new type of failure LegalHoldEnabledForConversationFailure when sending a message and show a dialog informing the user that this conversation is under legal hold and give him/her option to decide whether he/she still wants to resend that message.

Dependencies (Optional)

Needs releases with:

• feat: discover legal hold when sending message [WPB-5999] kalium#2333

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

How to Test

Enable legal hold for some other member before logging in and syncing, then login - at this point the app doesn't yet know that a conversation or member is under legal hold, but after sending first message, it should be handled properly.

Attachments (Optional)

legal_hold_discovery_send_message.mov

---

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: 6 lines in your changes are missing coverage. Please review.

Comparison is base (d141853) 41.16% compared to head (0d453cb) 41.25%.

Additional details and impacted files

@@              Coverage Diff              @@
##             develop    #2558      +/-   ##
=============================================
+ Coverage      41.16%   41.25%   +0.08%     
- Complexity      1134     1138       +4     
=============================================
  Files            361      361              
  Lines          13160    13183      +23     
  Branches        1733     1738       +5     
=============================================
+ Hits            5417     5438      +21     
+ Misses          7216     7214       -2     
- Partials         527      531       +4     

Files	Coverage Δ
.../ui/home/conversations/MessageComposerViewState.kt	88.88% <100.00%> (+1.38%)	⬆️
.../ui/home/conversations/MessageComposerViewModel.kt	81.66% <72.72%> (+0.66%)	⬆️

---

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 d141853...0d453cb. Read the comment docs.

[github-actions]

Test Results

741 tests  +3   741 ✅ +3   9m 22s ⏱️ +46s
102 suites ±0     0 💤 ±0 
102 files   ±0     0 ❌ ±0 

Results for commit 0d453cb. ± Comparison against base commit d141853.

This pull request removes 3 and adds 6 tests. Note that renamed tests count towards both.

com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to dismiss when enabled legal hold, when invoked sending, then message is not sent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to send anyway when enabled legal hold, when invoked sending, then message is sent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user needs to be informed about enabled legal hold, when invoked sending, then message is not sent and dialog shown()

com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to dismiss when enabled legal hold before sending, then message is not sent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to dismiss when enabled legal hold when sending fails, then message is not resent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to send anyway when enabled legal hold before sending, then message is sent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user chose to send anyway when enabled legal hold when sending fails, then message is resent and dialog hidden()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user needs to be informed about enabled legal hold when sending fails, then message is not resent and dialog shown()
com.wire.android.ui.home.conversations.MessageComposerViewModelTest ‑ given that user needs to be informed about enabled legal hold when sending, then message is not sent and dialog shown()

♻️ This comment has been updated with latest results.

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!