Merge branch 'topic/complex-values-replace'

app/models/scimitar/resources/mixin.rb

@@ -952,7 +952,10 @@ def from_patch_backend_apply!(nature:, path:, value:, altering_hash:)
 
                 when 'replace'
                   if path_component == 'root'
-                    altering_hash[path_component].merge!(value)
+                    dot_pathed_value = value.inject({}) do |hsh, (k, v)|
+                      hsh.deep_merge!(dot_path(k.split('.'), v))
+                    end
+                    altering_hash[path_component].deep_merge!(dot_pathed_value)
                   else
                     altering_hash[path_component] = value
                   end
@@ -1105,6 +1108,13 @@ def all_matching_filter(filter:, within_array:, &block)
             end
           end
 
+          def dot_path(arr, value)
+            return value if arr.empty?
+
+            {}.tap do |hash|
+              hash[arr.shift] = dot_path(arr, value)
+            end
+          end
       end # "included do"
     end # "module Mixin"
   end # "module Resources"

spec/models/scimitar/resources/mixin_spec.rb

@@ -2714,6 +2714,28 @@ def self.scim_queryable_attributes
             expect(@instance.username).to eql('1234')
           end
 
+          it 'which updates nested values using root syntax' do
+            @instance.update!(first_name: 'Foo', last_name: 'Bar')
+
+            path = 'name.givenName'
+            path = path.upcase if force_upper_case
+
+            patch = {
+              'schemas'    => ['urn:ietf:params:scim:api:messages:2.0:PatchOp'],
+              'Operations' => [
+                {
+                  'op'    => 'replace',
+                  'value' => {
+                    path => 'Baz'
+                  }
+                }
+              ]
+            }
+
+            @instance.from_scim_patch!(patch_hash: patch)
+            expect(@instance.first_name).to eql('Baz')
+          end
+
           it 'which updates nested values' do
             @instance.update!(first_name: 'Foo', last_name: 'Bar')