fix: do not remove MLS from the set of supported protocols [WPB-16254]

[MohamadJaara]

WPB-16254 [Android] Prevent removal of MLS from supported protocols if it was previously supported

---

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 can happen that a client remove MLS from the supported protocols after it is added

Solutions

try to prevent it by checking if it does exists and not remove it if it does

Dependencies (Optional)

If there are some other pull requests related to this one (e.g. new releases of frameworks), specify them here.

Needs releases with:

• GitHub link to other pull request

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

How to Test

Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.

Notes (Optional)

Specify here any other facts that you think are important for this issue.

Attachments (Optional)

Attachments like images, videos, etc. (drag and drop in the text box)

---

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

[github-actions]

Test Results

3 368 tests   3 258 ✅  5m 32s ⏱️
  572 suites    110 💤
  572 files        0 ❌

Results for commit 68ac729.

♻️ This comment has been updated with latest results.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (release/candidate@58cc378). Learn more about missing BASE report.

Additional details and impacted files

@@                 Coverage Diff                  @@
##             release/candidate    #3311   +/-   ##
====================================================
  Coverage                     ?   54.55%           
====================================================
  Files                        ?     1252           
  Lines                        ?    36695           
  Branches                     ?     3741           
====================================================
  Hits                         ?    20020           
  Misses                       ?    15252           
  Partials                     ?     1423           

Files with missing lines	Coverage Δ
...re/user/UpdateSelfUserSupportedProtocolsUseCase.kt	90.90% <100.00%> (ø)

---

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 58cc378...68ac729. Read the comment docs.

[datadog-wireapp]

Datadog Report

Branch report: fix/Prevent-removal-of-MLS-from-supported-protocols-if-it-was-previously-supported
Commit report: d932144
Test service: kalium-jvm

✅ 0 Failed, 3258 Passed, 110 Skipped, 1m 2.8s Total Time