Merge pull request #152 from fschmtt/update-roles-endpoint

feat: add Roles::update endpoint

Author: fschmtt

src/Resource/Roles.php

@@ -68,4 +68,19 @@ public function delete(string $realm, string $roleName): void
             ),
         );
     }
+
+    public function update(string $realm, Role $role): void
+    {
+        $this->commandExecutor->executeCommand(
+            new Command(
+                '/admin/realms/{realm}/roles/{roleName}',
+                Method::PUT,
+                [
+                    'realm' => $realm,
+                    'roleName' => $role->getName(),
+                ],
+                $role,
+            ),
+        );
+    }
 }

tests/Integration/Resource/RolesTest.php

@@ -14,7 +14,7 @@ class RolesTest extends TestCase
 {
     use IntegrationTestBehaviour;
 
-    public function testCreateRetrieveDeleteRole(): void
+    public function testCreateRetrieveUpdateDeleteRole(): void
     {
         $resource = $this->getKeycloak()->roles();
 
@@ -41,6 +41,9 @@ public function testCreateRetrieveDeleteRole(): void
         $role = $resource->get('master', 'test-role');
         static::assertSame('test-role', $role->getName());
 
+        // Update (created) role
+        $resource->update('master', $role->withDescription('updated-test-role-description'));
+
         // Delete (created) role
         $resource->delete('master', 'test-role');
 

tests/Unit/Resource/RolesTest.php

@@ -134,4 +134,34 @@ public function testDeleteRole(): void
 
         $roles->delete('test-realm', $deletedRoleName);
     }
+
+    public function testUpdateRole(): void
+    {
+        $updatedRole = new Role(name: 'updated-role');
+        $updatedRoleName = $updatedRole->getName();
+
+        static::assertIsString($updatedRoleName);
+
+        $command = new Command(
+            '/admin/realms/{realm}/roles/{roleName}',
+            Method::PUT,
+            [
+                'realm' => 'test-realm',
+                'roleName' => $updatedRoleName,
+            ],
+            $updatedRole,
+        );
+
+        $commandExecutor = $this->createMock(CommandExecutor::class);
+        $commandExecutor->expects(static::once())
+            ->method('executeCommand')
+            ->with($command);
+
+        $roles = new Roles(
+            $commandExecutor,
+            $this->createMock(QueryExecutor::class),
+        );
+
+        $roles->update('test-realm', $updatedRole);
+    }
 }