[NEW] Allow user with required permissions to remove member from room

[cardoso]

@RocketChat/ios

Also renames Subscription * Request -> Room * Request. It was starting to get confusing.

Adds test to room roles fetching

Closes #1319

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (develop@b7516bb). Click here to learn what that means.
The diff coverage is 46.42%.

@@            Coverage Diff            @@
##             develop   #1637   +/-   ##
=========================================
  Coverage           ?   36.7%           
=========================================
  Files              ?     307           
  Lines              ?   14318           
  Branches           ?       0           
=========================================
  Hits               ?    5256           
  Misses             ?    9062           
  Partials           ?       0

Impacted Files	Coverage Δ
...t.Chat/API/Requests/Room/RoomMessagesRequest.swift	0% <ø> (ø)
...et.Chat/API/Requests/Room/RoomMembersRequest.swift	93.54% <ø> (ø)
Rocket.Chat/API/API.swift	72.54% <ø> (ø)
Rocket.Chat/Models/User/UserUtils.swift	100% <ø> (ø)
...ocket.Chat/API/Requests/Room/RoomInfoRequest.swift	91.66% <ø> (ø)
...t.Chat/API/Requests/Room/RoomMentionsRequest.swift	88.88% <ø> (ø)
...cket.Chat/API/Requests/Room/RoomFilesRequest.swift	92.85% <ø> (ø)
Rocket.Chat/Models/Subscription/Subscription.swift	71.42% <ø> (ø)
...llers/Chat/ChatControllerMessageCellProtocol.swift	0% <0%> (ø)
Rocket.Chat/Helpers/Alert.swift	35.18% <0%> (ø)
... and 18 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7516bb...d3a7ff3. Read the comment docs.

[cardoso]

Fixed conflicts and 2 bugs:

• Not formatting confirmation alert correctly
• Not refreshing list after removing user

[rafaelks]

@cardoso I don't think I should be able to remove myself from the room 🤔:

[cardoso]

@rafaelks do you think we should remove the option or rename it to "Exit Room" ?

[rafaelks]

@cardoso Yes, let's not open anything to "myself" for now.

[cardoso]

@rafaelks Made your changes and fixed conflicts!

[rafaelks]

This is causing real weird behaviour on the list... we should be adding new rows instead of reloading the entire tableView. But not required right now. 👍

[rafaelks]

LGTM!