Support for saving members attribute upon group create (and similar use cases) #98

Author: arietimmerman

src/Attribute/MutableCollection.php

@@ -58,7 +58,11 @@ public function replace($value, Model &$object, ?Path $path = null)
             );
         }
 
-        $object->{$this->attribute}()->sync($existingObjects->all());
+        $object->saved(function (Model $model) use ($existingObjects)  {
+            $model->{$this->attribute}()->sync($existingObjects->all());
+            $model->load($this->attribute);
+        });
+        
     }
 
     public function patch($operation, $value, Model &$object, ?Path $path = null)

tests/GroupsTest.php

@@ -3,6 +3,7 @@
 namespace ArieTimmerman\Laravel\SCIMServer\Tests;
 
 use ArieTimmerman\Laravel\SCIMServer\Tests\Model\Group;
+use ArieTimmerman\Laravel\SCIMServer\Tests\Model\User;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
@@ -69,6 +70,37 @@ public function testCreate(){
 
     }
 
+    public function testCreateWithMembers(){
+        $response = $this->post('/scim/v2/Groups', [
+            'schemas' => ['urn:ietf:params:scim:schemas:core:2.0:Group'], // Required
+            'urn:ietf:params:scim:schemas:core:2.0:Group' => [
+                'displayName' => 'TestGroup',
+                'members' => [
+                    [
+                        'value' => User::first()->id,
+                    ]
+                ]
+            ]
+        ]);
+
+        $response->assertJsonStructure([
+            'id',
+            'urn:ietf:params:scim:schemas:core:2.0:Group' => [
+                'displayName',
+                'members' => [
+                    0 => [
+                        'value',
+                        'display'
+                    ]
+                ]
+            ]
+        ]);
+
+        $this->assertEquals(User::first()->id, $response->json(['urn:ietf:params:scim:schemas:core:2.0:Group'])['members'][0]['value']);
+        $this->assertNotNull(Group::find($response->json('id')));
+        $this->assertNotNull(Group::where('displayName', 'TestGroup')->first());
+    }
+
     public function testBulk(){
         $response = $this->post('/scim/v2/Bulk', [
             'schemas' => ['urn:ietf:params:scim:api:messages:2.0:BulkRequest'], // Required