Merge pull request #202 from Caascad/permissions_endpoints

Add permissions update endpoint

Author: jadolg

rocketchat_API/APISections/permissions.py

@@ -5,3 +5,9 @@ class RocketChatPermissions(RocketChatBase):
     def permissions_list_all(self, **kwargs):
         """Returns all permissions from the server."""
         return self.call_api_get("permissions.listAll", kwargs=kwargs)
+
+    def permissions_update(self, permissions, **kwargs):
+        """Edits permissions on the server."""
+        return self.call_api_post(
+            "permissions.update", permissions=permissions, kwargs=kwargs
+        )

tests/test_permissions.py

@@ -5,10 +5,59 @@ def test_permissions_list_all(logged_rocket):
     assert "remove" in permissions_list_all
 
 
+# pylint: disable=invalid-name
 def test_permissions_list_all_with_updatedSince(logged_rocket):
     permissions_list_all = logged_rocket.permissions_list_all(
         updatedSince="2017-11-25T15:08:17.248Z"
     ).json()
     assert permissions_list_all.get("success")
     assert "update" in permissions_list_all
     assert "remove" in permissions_list_all
+
+
+def test_permissions_update(logged_rocket):
+    def get_updated_roles(permission, permissions_update):
+        for permission_update in permissions_update:
+            if permission.get("_id", "A") == permission_update.get("_id", "B"):
+                return permission_update.get("roles")
+
+        # Permission not found
+        raise ValueError
+
+    def check_update(permissions, permissions_update):
+        for permission in permissions:
+            try:
+                if permission.get("roles") != get_updated_roles(
+                    permission, permissions_update
+                ):
+                    return False
+            except ValueError:
+                return False
+
+        return True
+
+    permissions = [
+        {"_id": "access-permissions", "roles": ["admin", "bot"]},
+        {"_id": "add-user-to-any-c-room", "roles": ["admin"]},
+    ]
+    permissions_update = logged_rocket.permissions_update(
+        permissions=permissions
+    ).json()
+    assert permissions_update.get("success")
+    assert check_update(permissions, permissions_update.get("permissions", []))
+
+    permissions_wrong_id = [
+        {"_id": "wrong-id", "roles": ["admin", "bot"]},
+    ]
+    assert (
+        check_update(permissions_wrong_id, permissions_update.get("permissions", []))
+        is False
+    )
+
+    permissions_wrong_roles = [
+        {"_id": "access-permissions", "roles": ["bot"]},
+    ]
+    assert (
+        check_update(permissions_wrong_roles, permissions_update.get("permissions", []))
+        is False
+    )

tests/test_user.py

@@ -181,7 +181,7 @@ def test_users_set_avatar_from_file(logged_rocket):
 def test_users_set_avatar_from_url(logged_rocket):
     # ToDo: Modify this test so it can run while offline
     users_set_avatar = logged_rocket.users_set_avatar(
-        avatar_url="https://avatars.dicebear.com/api/bottts/rocket.svg"
+        avatar_url="https://upload.wikimedia.org/wikipedia/commons/7/77/Wikipedia_svg_logo.svg"
     ).json()
     assert users_set_avatar.get("success"), users_set_avatar.get("error")