fix: prevent cancellation midway through event processing [WPB-7070]

[vitorhugods]

WPB-7070 [Android] Missing message on flaky internet

---

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

It's possible to have event processing being cancelled midway through and we would still mark them as processed.

Causes

Although unlikely, if we cancel the event processing midway through due to connectivity changing or websocket closing, we may swallow cancellation exceptions in wrapStorageRequest, and we do not return success/failure to the EventProcessor itself, as it expects exceptions to be thrown.

Solutions

The idea is to completely forbid cancellation midway through event processing, by:

1. Using the UserSessionScope as the CoroutineScope for event processing. Making it so it will only be cancelled when the user logs out.
2. Wrap the actual event processing with withContext(NonCancellable) {}

Testing

Test Coverage

• I have added automated test to this contribution

---

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.

[github-actions]

Test Results

3 171 tests  +146   3 066 ✔️ +59   3m 20s ⏱️ - 2m 3s
   543 suites +536      105 💤 +87 
   543 files   +536          0 ❌ ±  0 

Results for commit 216237a. ± Comparison against base commit faff60a.

♻️ This comment has been updated with latest results.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

[datadog-wireapp]

Datadog Report

Branch report: fix/prevent-missing-messages-on-flaky-internet
Commit report: 9417151
Test service: kalium-jvm

✅ 0 Failed, 3066 Passed, 105 Skipped, 10.3s Total Time