Fix broken add/remove permissions method

Author: tsmith023

src/roles/index.ts

@@ -106,9 +106,13 @@ const roles = (connection: ConnectionREST): Roles => {
         .then(() => true)
         .catch(() => false),
     addPermissions: (roleName: string, permissions: PermissionsInput) =>
-      connection.postEmpty(`/authz/roles/${roleName}/add-permissions`, { permissions }),
+      connection.postEmpty(`/authz/roles/${roleName}/add-permissions`, {
+        permissions: Map.flattenPermissions(permissions).flatMap(Map.permissionToWeaviate),
+      }),
     removePermissions: (roleName: string, permissions: PermissionsInput) =>
-      connection.postEmpty(`/authz/roles/${roleName}/remove-permissions`, { permissions }),
+      connection.postEmpty(`/authz/roles/${roleName}/remove-permissions`, {
+        permissions: Map.flattenPermissions(permissions).flatMap(Map.permissionToWeaviate),
+      }),
     hasPermissions: (roleName: string, permission: Permission | Permission[]) =>
       Promise.all(
         (Array.isArray(permission) ? permission : [permission])

src/roles/integration.test.ts

@@ -326,6 +326,35 @@ maybe('Integration testing of the roles namespace', () => {
     });
   });
 
+  it('should be able to add permissions to one of the created roles', async () => {
+    await client.roles.addPermissions(
+      'backups',
+      weaviate.permissions.backup({ collection: 'Another-collection', manage: true })
+    );
+    const role = await client.roles.byName('backups');
+    expect(role).toEqual({
+      name: 'backups',
+      ...emptyPermissions,
+      backupsPermissions: [
+        { collection: 'Some-collection', actions: ['manage_backups'] },
+        { collection: 'Another-collection', actions: ['manage_backups'] },
+      ],
+    });
+  });
+
+  it('should be able to remove permissions from one of the created roles', async () => {
+    await client.roles.removePermissions(
+      'backups',
+      weaviate.permissions.backup({ collection: 'Another-collection', manage: true })
+    );
+    const role = await client.roles.byName('backups');
+    expect(role).toEqual({
+      name: 'backups',
+      ...emptyPermissions,
+      backupsPermissions: [{ collection: 'Some-collection', actions: ['manage_backups'] }],
+    });
+  });
+
   it('should delete one of the created roles', async () => {
     await client.roles.delete('backups');
     await expect(client.roles.byName('backups')).rejects.toThrowError(WeaviateUnexpectedStatusCodeError);