fix(mls): fetch and set mls-removal keys for 1on1 conversations (WPB-10743) 🍒 🍒

[github-actions]

WPB-10743 [Android] Use proper MLS removal key with federated 1:1 conversations

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

• fix(mls): fetch and set mls-removal keys for 1on1 conversations (WPB-10743) 🍒 #3021

Original PR description:

---

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

• fix(mls): fetch and set mls-removal keys for 1on1 conversations (WPB-10743) #3020

Original PR description:

---

---

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?

Fetch and set removal-keys from the get-oneonone-conversation response for 1on1 conversations and only APIV6

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

---

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.

[sonarcloud]

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

[github-actions]

Test Results

3 212 tests  ±0   3 106 ✅ ±0   4m 11s ⏱️ -8s
  551 suites ±0     106 💤 ±0 
  551 files   ±0       0 ❌ ±0 

Results for commit 6b08709. ± Comparison against base commit c667979.

♻️ This comment has been updated with latest results.

[github-actions]

Bencher Report

Branch	fix/mls-1on1-removal-keys-cherry-pick-cherry-pick
Testbed	ubuntu-latest

⚠️ WARNING: The following Measure does not have a Threshold. Without a Threshold, no Alerts will ever be generated!

• Latency

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	698,912.63
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory	📈 view plot ⚠️ NO THRESHOLD	648,889,047.55
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark	📈 view plot ⚠️ NO THRESHOLD	951,899,951.83
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark	📈 view plot ⚠️ NO THRESHOLD	21,508,807.44

🐰 View full continuous benchmarking report in Bencher

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 39.13043% with 14 lines in your changes missing coverage. Please review.

Project coverage is 52.51%. Comparing base (c667979) to head (6b08709).

Files with missing lines	Patch %	Lines
.../network/api/v6/authenticated/ConversationApiV6.kt	0.00%	5 Missing ⚠️
...ire/kalium/logic/data/conversation/Conversation.kt	0.00%	2 Missing ⚠️
...in/com/wire/kalium/logic/data/mls/MLSPublicKeys.kt	0.00%	2 Missing ⚠️
.../logic/data/conversation/ConversationRepository.kt	0.00%	2 Missing ⚠️
...conversation/JoinExistingMLSConversationUseCase.kt	80.00%	2 Missing ⚠️
...authenticated/conversation/ConversationResponse.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3044      +/-   ##
===========================================
- Coverage    52.51%   52.51%   -0.01%     
===========================================
  Files         1303     1304       +1     
  Lines        50174    50178       +4     
  Branches      4679     4679              
===========================================
+ Hits         26349    26350       +1     
- Misses       21939    21942       +3     
  Partials      1886     1886              

Files with missing lines	Coverage Δ
...lium/logic/data/conversation/ConversationMapper.kt	58.40% <ø> (ø)
...gic/data/conversation/MLSConversationRepository.kt	83.72% <ø> (ø)
...ogic/data/mlspublickeys/MLSPublicKeysRepository.kt	22.22% <ø> (-7.78%)	⬇️
...onversation/mls/MLSOneOnOneConversationResolver.kt	100.00% <100.00%> (ø)
...authenticated/conversation/ConversationResponse.kt	0.00% <0.00%> (ø)
...ire/kalium/logic/data/conversation/Conversation.kt	0.00% <0.00%> (ø)
...in/com/wire/kalium/logic/data/mls/MLSPublicKeys.kt	0.00% <0.00%> (ø)
.../logic/data/conversation/ConversationRepository.kt	62.19% <0.00%> (-0.13%)	⬇️
...conversation/JoinExistingMLSConversationUseCase.kt	80.48% <80.00%> (-0.35%)	⬇️
.../network/api/v6/authenticated/ConversationApiV6.kt	0.00% <0.00%> (ø)

... 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 c667979...6b08709. Read the comment docs.

[datadog-wireapp]

Datadog Report

Branch report: fix/mls-1on1-removal-keys-cherry-pick-cherry-pick
Commit report: 3ca54cd
Test service: kalium-jvm

✅ 0 Failed, 3106 Passed, 106 Skipped, 14.46s Total Time