feat: Handle needToRemoveLocally flag [WPB-14603]

[m-zagorski]

WPB-14603 [Android] handle needToRemoveLocally flag when reciving a cleared message

https://wearezeta.atlassian.net/browse/WPB-14603

What's new in this PR?

Issues

Handle needToRemoveLocally flag for Cleared messages

Solutions

In this PR we only define it and handle its behavior, we do NOT send it or handle sending right now

1. If needToRemoveLocally is false or null, then only the conversation content is cleared, including any assets that are stored locally, but the conversation is not removed from the list.
2. needToRemoveLocally is true and the user is not a conversation member: same as option 1, but the conversation is removed.
3. needToRemoveLocally is true and the user is a conversation member: same as option 1

---

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.

[github-actions]

Test Results

3 346 tests  +5   3 239 ✅ +5   5m 28s ⏱️ -15s
  573 suites +1     107 💤 ±0 
  573 files   +1       0 ❌ ±0 

Results for commit 375400a. ± Comparison against base commit 5396afa.

♻️ This comment has been updated with latest results.

[github-actions]

Bencher Report

Branch	feat/need-to-remove-locally-flag
Testbed	ubuntu-latest

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

• Latency (nanoseconds (ns))

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds CLI flag.

Click to view all benchmark results

Benchmark	Latency	nanoseconds (ns)
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInFiles	📈 view plot ⚠️ NO THRESHOLD	664,827.35
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory	📈 view plot ⚠️ NO THRESHOLD	327,715,593.14
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark	📈 view plot ⚠️ NO THRESHOLD	1,311,282,950.29
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark	📈 view plot ⚠️ NO THRESHOLD	21,659,425.20

🐰 View full continuous benchmarking report in Bencher

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 54.13%. Comparing base (5396afa) to head (375400a).

Files with missing lines	Patch %	Lines
...re/kalium/logic/data/message/ProtoContentMapper.kt	0.00%	4 Missing ⚠️
...m/wire/kalium/logic/data/message/MessageContent.kt	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3158      +/-   ##
===========================================
+ Coverage    54.11%   54.13%   +0.02%     
===========================================
  Files         1252     1252              
  Lines        36519    36527       +8     
  Branches      3699     3703       +4     
===========================================
+ Hits         19763    19775      +12     
+ Misses       15331    15327       -4     
  Partials      1425     1425              

Files with missing lines	Coverage Δ
...re/conversation/ClearConversationContentUseCase.kt	100.00% <100.00%> (ø)
...eceiver/handler/ClearConversationContentHandler.kt	100.00% <100.00%> (+100.00%)	⬆️
...m/wire/kalium/logic/data/message/MessageContent.kt	0.00% <0.00%> (ø)
...re/kalium/logic/data/message/ProtoContentMapper.kt	65.04% <0.00%> (-0.32%)	⬇️

... and 1 file with indirect coverage changes

---

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 5396afa...375400a. Read the comment docs.

[datadog-wireapp]

Datadog Report

Branch report: feat/need-to-remove-locally-flag
Commit report: 065eb66
Test service: kalium-jvm

✅ 0 Failed, 3239 Passed, 107 Skipped, 1m 0.09s Total Time

[vitorhugods]

Well-done, just one small suggestion to rename a variable that doesn't reflect its value/usage properly.

[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 SonarQube Cloud