fix: add user inside context for smart actions hooks (#532)

Author: arnaud-moncel

app/controllers/forest_liana/actions_controller.rb

@@ -32,7 +32,7 @@ def get_smart_action_load_ctx(fields)
         field[:value] = nil unless field[:value]
         field
       end
-      {:fields => fields, :params => params}
+      {:fields => fields, :params => params, :user => forest_user}
     end
 
     def get_smart_action_change_ctx(fields, field_changed)
@@ -42,7 +42,7 @@ def get_smart_action_change_ctx(fields, field_changed)
         ForestLiana::WidgetsHelper.set_field_widget(field)
         field
       end
-      {:field_changed => found_field_changed, :fields => fields, :params => params}
+      {:field_changed => found_field_changed, :fields => fields, :params => params, :user => forest_user}
     end
 
     def handle_result(result, action)

spec/dummy/tmp/cache/11D/A92/http%3A%2F%2Flocalhost%3A3000%2Fforest%2Fauthentication%2Fcallback-env_secret_test-client-data

@@ -0,0 +1 @@
+o: ActiveSupport::Cache::Entry	:@value{:client_idI"random_id:ET:issuerI"api.forestadmin.com;T:@version0:@created_atf1639392521.7375941:@expires_in0

spec/requests/actions_controller_spec.rb

@@ -136,13 +136,26 @@
         }
     }
 
+    use_user_context_action_definition = {
+      name: 'use_user_context',
+      fields: [foo],
+      hooks: {
+        :load => -> (context) {
+          foo = context[:fields].find{|field| field[:field] == 'foo'}
+          foo[:value] = context[:user]['first_name']
+          context[:fields]
+        }
+      }
+    }
+
     action = ForestLiana::Model::Action.new(action_definition)
     fail_action = ForestLiana::Model::Action.new(fail_action_definition)
     cheat_action = ForestLiana::Model::Action.new(cheat_action_definition)
     enums_action = ForestLiana::Model::Action.new(enums_action_definition)
     multiple_enums_action = ForestLiana::Model::Action.new(multiple_enums_action_definition)
+    use_user_context_action = ForestLiana::Model::Action.new(use_user_context_action_definition)
     island = ForestLiana.apimap.find {|collection| collection.name.to_s == ForestLiana.name_for(Island)}
-    island.actions = [action, fail_action, cheat_action, enums_action, multiple_enums_action]
+    island.actions = [action, fail_action, cheat_action, enums_action, multiple_enums_action, use_user_context_action]
 
     describe 'call /load' do
       params = {
@@ -174,6 +187,12 @@
         expect(response.status).to eq(500)
         expect(JSON.parse(response.body)).to eq({'error' => 'Error in smart action load hook: hook must return an array of fields'})
       end
+
+      it 'should return the first_name of the user who call the action' do
+        post '/forest/actions/use_user_context/hooks/load', params: JSON.dump(params), headers: headers
+        expect(response.status).to eq(200)
+        expect(JSON.parse(response.body)).to eq({'fields' => [foo.merge({:value => 'Michael'}).transform_keys { |key| key.to_s.camelize(:lower) }.stringify_keys]})
+      end
     end
 
     describe 'call /change' do