feat: add and remove conversation favorite [WPB-11639]

[Garzas]

WPB-11639 [Android] User Story for Add to favourites/Remove from favourites

---

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?

• implemented adding conversation to favorites folder
• implemented removing conversation from favorite folder
• extracted favorite add/remove management to smaller view model to not duplicate it in 3 places (OtherUserScreen, ConversationDetailsScreen, ConversationListScreen)

add_to_favorites.mov

[saleniuk]

suggestion: move the // TODO(profile): enable when implemented line below this one so that it's right next to commented line // moveConversationToFolder: () -> Unit,

[borichellow]

in fact there are only 2 differences in this if else: title of item and boolean parameter for changeFavoriteState.
So it will save a lot of lines to have something like:

add {
    MenuBottomSheetItem(
        title = stringResource(if (isFavorite) R.string.label_remove_from_favourites else R.string.label_add_to_favourites),
        leading = {
            MenuItemIcon(
                id = R.drawable.ic_favourite,
                contentDescription = null,
            )
        },
        onItemClick = {
            changeFavoriteState(
                GroupDialogState(
                    conversationSheetContent.conversationId,
                    conversationSheetContent.title
                ),
                !isFavorite
            )
        }
    )
}      

[borichellow]

please, use contentDescription = null if icon is not clickable and it's just duplicating title that goes after the icon.
Otherwise Talkback will have a bit broken behaviour ;P

[sonarqubecloud]

Quality Gate passed

Issues
2 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

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.12%. Comparing base (c911271) to head (3d797f4).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3653      +/-   ##
===========================================
+ Coverage    46.04%   46.12%   +0.07%     
===========================================
  Files          472      472              
  Lines        16108    16110       +2     
  Branches      2666     2666              
===========================================
+ Hits          7417     7430      +13     
+ Misses        7917     7906      -11     
  Partials       774      774              

Files with missing lines	Coverage Δ
...tlin/com/wire/android/mapper/ConversationMapper.kt	68.04% <100.00%> (+0.67%)	⬆️
...n/kotlin/com/wire/android/model/SnackBarMessage.kt	75.00% <100.00%> (+75.00%)	⬆️
...tions/details/GroupConversationDetailsViewModel.kt	67.21% <100.00%> (+0.54%)	⬆️
...roupConversationDetailsBottomSheetEventsHandler.kt	0.00% <ø> (ø)
...ome/conversationslist/ConversationListViewModel.kt	32.35% <ø> (+0.26%)	⬆️
...i/home/conversationslist/model/ConversationItem.kt	77.63% <100.00%> (+0.91%)	⬆️
...serprofile/other/OtherUserProfileEventsHandlers.kt	0.00% <ø> (ø)
...erprofile/other/OtherUserProfileScreenViewModel.kt	59.09% <100.00%> (+0.50%)	⬆️

... 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 c911271...3d797f4. Read the comment docs.

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests