fix(mls): set removal-keys for 1on1 calls from conversation-response (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): set removal-keys for 1on1 calls from conversation-response (WPB-10743) 🍒 #3019

Original PR description:

---

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

• fix(mls): set removal-keys for 1on1 calls from conversation-response (WPB-10743) #3009

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 create-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.

[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

[github-actions]

Test Results

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

Results for commit c5b3f2c. ± Comparison against base commit 27fc353.

♻️ This comment has been updated with latest results.

[github-actions]

Bencher Report

Branch	fix/federated-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	700,971.81
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory	📈 view plot ⚠️ NO THRESHOLD	341,961,072.89
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark	📈 view plot ⚠️ NO THRESHOLD	946,587,869.87
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark	📈 view plot ⚠️ NO THRESHOLD	21,344,391.79

🐰 View full continuous benchmarking report in Bencher

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 15.29412% with 72 lines in your changes missing coverage. Please review.

Project coverage is 52.51%. Comparing base (27fc353) to head (c5b3f2c).

Files with missing lines	Patch %	Lines
...lium/logic/data/conversation/ConversationMapper.kt	3.33%	28 Missing and 1 partial ⚠️
...om/wire/kalium/network/api/model/ApiModelMapper.kt	0.00%	18 Missing ⚠️
.../network/api/v6/authenticated/ConversationApiV6.kt	0.00%	11 Missing ⚠️
...authenticated/conversation/ConversationResponse.kt	0.00%	10 Missing ⚠️
...ogic/data/mlspublickeys/MLSPublicKeysRepository.kt	57.14%	2 Missing and 1 partial ⚠️
...gic/data/conversation/MLSConversationRepository.kt	83.33%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3041      +/-   ##
===========================================
- Coverage    52.56%   52.51%   -0.06%     
===========================================
  Files         1303     1303              
  Lines        50104    50174      +70     
  Branches      4671     4679       +8     
===========================================
+ Hits         26338    26349      +11     
- Misses       21884    21939      +55     
- Partials      1882     1886       +4     

Files with missing lines	Coverage Δ
...c/data/conversation/ConversationGroupRepository.kt	91.09% <100.00%> (+0.05%)	⬆️
...gic/data/conversation/MLSConversationRepository.kt	83.72% <83.33%> (-0.18%)	⬇️
...ogic/data/mlspublickeys/MLSPublicKeysRepository.kt	30.00% <57.14%> (+20.00%)	⬆️
...authenticated/conversation/ConversationResponse.kt	0.00% <0.00%> (ø)
.../network/api/v6/authenticated/ConversationApiV6.kt	0.00% <0.00%> (ø)
...om/wire/kalium/network/api/model/ApiModelMapper.kt	0.00% <0.00%> (ø)
...lium/logic/data/conversation/ConversationMapper.kt	58.40% <3.33%> (-4.03%)	⬇️

... and 4 files 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 27fc353...c5b3f2c. Read the comment docs.

[datadog-wireapp]

Datadog Report

Branch report: fix/federated-1on1-removal-keys-cherry-pick-cherry-pick
Commit report: 105efef
Test service: kalium-jvm

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