forest_liana 5.3.3 → 6.0.0.pre.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 52b354dcc7c4a05c8303bffa7c1deca674900d7b693f37e816d2bafe665c4b12
-  data.tar.gz: 358a702306af12e805a6fe954c2e0fbb1d1ccc3ebe8fe8d8b73d6ebd8f101bb5
+  metadata.gz: 8a534b6abdb1a47d2690f785ef13dff74d97117fa7e1871b82659644ac9c1a53
+  data.tar.gz: 9d7c3281edc36a9aabf47f74851c3ba191d1e31c7bd29a0a4c7d9e8b6f3522f7
 SHA512:
-  metadata.gz: 68dac9b8f4e7f427d1c9f4bf885d2412cf740b8e6038a6e83e919c967e9fbe9369fddc753992b7d80a191a04688366c06cc028a2afe309a476686e993603e8a6
-  data.tar.gz: fcdb37f174ec29d416bfe1ce668608baa9e1924ec6ea2b51e61dcc74f1976e11fdf6eb41eb5503a22ed352d4e0ad772bc1498f2f31f958a467ac54dcf6e56a75
+  metadata.gz: c18f213548f7584a4dc7aabe121442d0e50270f29860c54164348bd45fca77810d5402284147b9a114d04f17b5314ee8d7f64daae1887e6f258cfd7663ba6c1f
+  data.tar.gz: 6edfe33f160b65de926013b45ef0ba4b7ae374a4446faf64ca6ffbd3a5e606845c89c9e30756fe8e26d4d420786ab09a36b38b0077d0c975a4d695b0aaa0c5ef

data/app/controllers/forest_liana/actions_controller.rb

@@ -1,102 +1,7 @@
 module ForestLiana
   class ActionsController < ForestLiana::BaseController
-
     def values
       render serializer: nil, json: {}, status: :ok
     end
-
-    def get_collection(collection_name)
-      ForestLiana.apimap.find { |collection| collection.name.to_s == collection_name }
-    end
-
-    def get_action(collection_name)
-      collection = get_collection(collection_name)
-      begin
-         collection.actions.find {|action| ActiveSupport::Inflector.parameterize(action.name) == params[:action_name]}
-      rescue => error
-        FOREST_LOGGER.error "Smart Action get action retrieval error: #{error}"
-        nil
-      end
-    end
-    
-    def get_record
-      model = ForestLiana::SchemaUtils.find_model_from_collection_name(params[:collectionName])
-      redord_getter = ForestLiana::ResourceGetter.new(model, {:id => params[:recordIds][0]})
-      redord_getter.perform
-      redord_getter.record
-    end
-
-    def get_smart_action_load_ctx(fields)
-      fields = fields.reduce({}) do |p, c|
-        ForestLiana::WidgetsHelper.set_field_widget(c)
-        p.update(c[:field] => c.merge!(value: nil))
-      end
-      {:record => get_record, :fields => fields}
-    end
-
-    def get_smart_action_change_ctx(fields)
-      fields = fields.reduce({}) do |p, c|
-        field = c.permit!.to_h.symbolize_keys
-        ForestLiana::WidgetsHelper.set_field_widget(field)
-        p.update(c[:field] => field)
-      end
-      {:record => get_record, :fields => fields}
-    end
-
-    def handle_result(result, formatted_fields, action)
-      if result.nil? || !result.is_a?(Hash)
-        return render status: 500, json: { error: 'Error in smart action load hook: hook must return an object' }
-      end
-      is_same_data_structure = ForestLiana::IsSameDataStructureHelper::Analyser.new(formatted_fields, result, 1)
-      unless is_same_data_structure.perform
-        return render status: 500, json: { error: 'Error in smart action hook: fields must be unchanged (no addition nor deletion allowed)' }
-      end
-
-      # Apply result on fields (transform the object back to an array), preserve order.
-      fields = action.fields.map do |field|
-        updated_field = result[field[:field]]
-        # Reset `value` when not present in `enums` (which means `enums` has changed).
-        if updated_field[:enums].is_a?(Array) && !updated_field[:enums].include?(updated_field[:value])
-          updated_field[:value] = nil
-        end
-        updated_field
-      end
-
-      render serializer: nil, json: { fields: fields}, status: :ok
-    end
-
-    def load
-      action = get_action(params[:collectionName])
-
-      if !action
-        render status: 500, json: {error: 'Error in smart action load hook: cannot retrieve action from collection'}
-      else
-        # Transform fields from array to an object to ease usage in hook, adds null value.
-        context = get_smart_action_load_ctx(action.fields)
-        formatted_fields = context[:fields].clone # clone for following test on is_same_data_structure
-
-        # Call the user-defined load hook.
-        result = action.hooks[:load].(context)
-
-        handle_result(result, formatted_fields, action)
-      end
-    end
-
-    def change
-      action = get_action(params[:collectionName])
-
-      if !action
-        render status: 500, json: {error: 'Error in smart action change hook: cannot retrieve action from collection'}
-      else
-        # Transform fields from array to an object to ease usage in hook.
-        context = get_smart_action_change_ctx(params[:fields])
-        formatted_fields = context[:fields].clone # clone for following test on is_same_data_structure
-
-        # Call the user-defined change hook.
-        result = action.hooks[:change][params[:changedField]].(context)
-
-        handle_result(result, formatted_fields, action)
-      end
-    end
   end
 end

data/app/models/forest_liana/model/action.rb

@@ -5,7 +5,7 @@ class ForestLiana::Model::Action
   extend ActiveModel::Naming
 
   attr_accessor :id, :name, :base_url, :endpoint, :http_method, :fields, :redirect,
-    :type, :download, :hooks
+    :type, :download
 
   def initialize(attributes = {})
     if attributes.key?(:global)
@@ -66,7 +66,6 @@ class ForestLiana::Model::Action
     @base_url ||= nil
     @type ||= "bulk"
     @download ||= false
-    @hooks = !@hooks.nil? ? @hooks.symbolize_keys : nil
   end
 
   def persisted?

data/app/services/forest_liana/apimap_sorter.rb

@@ -39,7 +39,6 @@ module ForestLiana
       'redirect',
       'download',
       'fields',
-      'hooks',
     ]
     KEYS_ACTION_FIELD = [
       'field',

data/app/services/forest_liana/resources_getter.rb

@@ -12,11 +12,11 @@ module ForestLiana
       @collection = get_collection(@collection_name)
       @fields_to_serialize = get_fields_to_serialize
       @field_names_requested = field_names_requested
-      get_segment
-      compute_includes
+      get_segment()
+      compute_includes()
       @search_query_builder = SearchQueryBuilder.new(@params, @includes, @collection)
 
-      prepare_query
+      prepare_query()
     end
 
     def self.get_ids_from_request(params)

data/config/routes.rb

@@ -57,6 +57,4 @@ ForestLiana::Engine.routes.draw do
 
   # Smart Actions forms value
   post 'actions/:action_name/values' => 'actions#values'
-  post 'actions/:action_name/hooks/load' => 'actions#load'
-  post 'actions/:action_name/hooks/change' => 'actions#change'
 end

data/lib/forest_liana/bootstrapper.rb

@@ -38,14 +38,6 @@ module ForestLiana
 
     private
 
-    def get_collection(collection_name)
-      ForestLiana.apimap.find { |collection| collection.name.to_s == collection_name }
-    end
-
-    def get_action(collection, action_name)
-      collection.actions.find {|action| action.name == action_name}
-    end
-
     def generate_apimap
       create_apimap
       require_lib_forest_liana
@@ -53,17 +45,6 @@ module ForestLiana
 
       if Rails.env.development?
         @collections_sent = ForestLiana.apimap.as_json
-
-        @collections_sent.each do |collection|
-          collection['actions'].each do |action|
-            c = get_collection(collection['name'])
-            a = get_action(c, action['name'])
-            load = !a.hooks.nil? && a.hooks.key?(:load) && a.hooks[:load].is_a?(Proc)
-            change = !a.hooks.nil? && a.hooks.key?(:change) && a.hooks[:change].is_a?(Hash) ? a.hooks[:change].keys : []
-            action['hooks'] = {:load => load, :change => change}
-          end
-        end
-
         @meta_sent = ForestLiana.meta
         SchemaFileUpdater.new(SCHEMA_FILENAME, @collections_sent, @meta_sent).perform()
       else

data/lib/forest_liana/schema_file_updater.rb

@@ -50,7 +50,6 @@ module ForestLiana
       'redirect',
       'download',
       'fields',
-      'hooks',
     ]
     KEYS_ACTION_FIELD = [
       'field',

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "5.3.3"
+  VERSION = "6.0.0-beta.1"
 end

data/spec/services/forest_liana/apimap_sorter_spec.rb

@@ -75,8 +75,7 @@ module ForestLiana
                 type: 'File',
                 field: 'File'
               }],
-              http_method: nil,
-              hooks: nil,
+              http_method: nil
             }
           }, {
             attributes: {
@@ -96,8 +95,7 @@ module ForestLiana
               download: nil,
               endpoint: nil,
               redirect: nil,
-              'http_method': nil,
-              hooks: nil,
+              'http_method': nil
             }
           }]
         }
@@ -150,8 +148,8 @@ module ForestLiana
         end
 
         it 'should sort the included actions and segments objects attributes values' do
-          expect(apimap_sorted['included'][0]['attributes'].keys).to eq(['name', 'endpoint', 'http_method', 'redirect', 'download', 'hooks'])
-          expect(apimap_sorted['included'][1]['attributes'].keys).to eq(['name', 'http_method', 'fields', 'hooks'])
+          expect(apimap_sorted['included'][0]['attributes'].keys).to eq(['name', 'endpoint', 'http_method', 'redirect', 'download'])
+          expect(apimap_sorted['included'][1]['attributes'].keys).to eq(['name', 'http_method', 'fields'])
           expect(apimap_sorted['included'][2]['attributes'].keys).to eq(['name'])
           expect(apimap_sorted['included'][3]['attributes'].keys).to eq(['name'])
         end

data/spec/services/forest_liana/schema_adapter_spec.rb

@@ -9,7 +9,7 @@ module ForestLiana
 
           expect(collection.fields.map { |field| field[:field] }).to eq(
             ["id", "name", "created_at", "updated_at", "trees"]
-          )
+          );
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 5.3.3
+  version: 6.0.0.pre.beta.1
 platform: ruby
 authors:
 - Sandro Munda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-08 00:00:00.000000000 Z
+date: 2020-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -211,10 +211,8 @@ files:
 - app/helpers/forest_liana/adapter_helper.rb
 - app/helpers/forest_liana/application_helper.rb
 - app/helpers/forest_liana/decoration_helper.rb
-- app/helpers/forest_liana/is_same_data_structure_helper.rb
 - app/helpers/forest_liana/query_helper.rb
 - app/helpers/forest_liana/schema_helper.rb
-- app/helpers/forest_liana/widgets_helper.rb
 - app/models/forest_liana/model/action.rb
 - app/models/forest_liana/model/collection.rb
 - app/models/forest_liana/model/segment.rb
@@ -338,11 +336,9 @@ files:
 - spec/dummy/db/migrate/20190716130830_add_age_to_tree.rb
 - spec/dummy/db/migrate/20190716135241_add_type_to_user.rb
 - spec/dummy/db/schema.rb
-- spec/helpers/forest_liana/is_same_data_structure_helper_spec.rb
 - spec/helpers/forest_liana/query_helper_spec.rb
 - spec/helpers/forest_liana/schema_helper_spec.rb
 - spec/rails_helper.rb
-- spec/requests/actions_controller_spec.rb
 - spec/requests/resources_spec.rb
 - spec/services/forest_liana/apimap_sorter_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
@@ -455,9 +451,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.0.8
 signing_key: 
@@ -560,7 +556,6 @@ test_files:
 - spec/services/forest_liana/apimap_sorter_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
 - spec/spec_helper.rb
-- spec/requests/actions_controller_spec.rb
 - spec/requests/resources_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/app/views/layouts/application.html.erb
@@ -603,6 +598,5 @@ test_files:
 - spec/dummy/config/initializers/backtrace_silencers.rb
 - spec/dummy/config/database.yml
 - spec/helpers/forest_liana/schema_helper_spec.rb
-- spec/helpers/forest_liana/is_same_data_structure_helper_spec.rb
 - spec/helpers/forest_liana/query_helper_spec.rb
 - spec/rails_helper.rb

data/app/helpers/forest_liana/is_same_data_structure_helper.rb

@@ -1,44 +0,0 @@
-require 'set'
-
-module ForestLiana
-  module IsSameDataStructureHelper
-    class Analyser
-      def initialize(object, other, deep = 0)
-        @object = object
-        @other = other
-        @deep = deep
-      end
-
-      def are_objects(object, other)
-        object && other && object.is_a?(Hash) && other.is_a?(Hash)
-      end
-
-      def check_keys(object, other, step = 0)
-        unless are_objects(object, other)
-          return false
-        end
-
-        object_keys = object.keys
-        other_keys = other.keys
-
-        if object_keys.length != other_keys.length
-          return false
-        end
-
-        object_keys_set = object_keys.to_set
-        other_keys.each { |key|
-          if !object_keys_set.member?(key) || (step + 1 <= @deep && !check_keys(object[key], other[key], step + 1))
-            return false
-          end
-        }
-
-        return true
-      end
-
-      def perform
-        check_keys(@object, @other)
-      end
-    end
-  end
-end
-

data/app/helpers/forest_liana/widgets_helper.rb

@@ -1,59 +0,0 @@
-require 'set'
-
-module ForestLiana
-  module WidgetsHelper
-
-    @widget_edit_list = [
-      'address editor',
-      'belongsto typeahead',
-      'belongsto dropdown',
-      'boolean editor',
-      'checkboxes',
-      'color editor',
-      'date editor',
-      'dropdown',
-      'embedded document editor',
-      'file picker',
-      'json code editor',
-      'input array',
-      'multiple select',
-      'number input',
-      'point editor',
-      'price editor',
-      'radio button',
-      'rich text',
-      'text area editor',
-      'text editor',
-      'time input',
-    ]
-
-    @v1_to_v2_edit_widgets_mapping = {
-      address: 'address editor',
-      'belongsto select': 'belongsto dropdown',
-      'color picker': 'color editor',
-      'date picker': 'date editor',
-      price: 'price editor',
-      'JSON editor': 'json code editor',
-      'rich text editor': 'rich text',
-      'text area': 'text area editor',
-      'text input': 'text editor',
-    }
-
-    def self.set_field_widget(field)
-
-      if field[:widget]
-        if @v1_to_v2_edit_widgets_mapping[field[:widget].to_sym]
-          field[:widgetEdit] = {name: @v1_to_v2_edit_widgets_mapping[field[:widget].to_sym], parameters: {}}
-        elsif @widget_edit_list.include?(field[:widget])
-          field[:widgetEdit] = {name: field[:widget], parameters: {}}
-        end
-      end
-
-      if !field.key?(:widgetEdit)
-        field[:widgetEdit] = nil
-      end
-
-      field.delete(:widget)
-    end
-  end
-end

data/spec/helpers/forest_liana/is_same_data_structure_helper_spec.rb

@@ -1,87 +0,0 @@
-module ForestLiana
-  context 'IsSameDataStructure class' do
-    it 'should: be valid with simple data' do
-      object = {:a => 'a', :b => 'b'}
-      other = {:a => 'a', :b => 'b'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be true
-    end
-
-    it 'should: be invalid with simple data' do
-      object = {:a => 'a', :b => 'b'}
-      other = {:a => 'a', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with not same hash' do
-      object = {:a => 'a', :b => 'b'}
-      other = {:a => 'a', :b => 'b', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with nil' do
-      object = nil
-      other = {:a => 'a', :b => 'b', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with not hash' do
-      object = nil
-      other = {:a => 'a', :b => 'b', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with integer' do
-      object = 1
-      other = {:a => 'a', :b => 'b', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with string' do
-      object = 'a'
-      other = {:a => 'a', :b => 'b', :c => 'c'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other).perform
-      expect(result).to be false
-    end
-
-    it 'should: be valid with depth 1' do
-      object = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      other = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      result = IsSameDataStructureHelper::Analyser.new(object, other, 1).perform
-      expect(result).to be true
-    end
-
-    it 'should: be invalid with depth 1' do
-      object = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      other = {:a => {:c => 'c'}, :b => {:e => 'e'}}
-      result = IsSameDataStructureHelper::Analyser.new(object, other, 1).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with depth 1 and nil' do
-      object = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      other = {:a => {:c => 'c'}, :b => nil}
-      result = IsSameDataStructureHelper::Analyser.new(object, other, 1).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with depth 1 and integer' do
-      object = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      other = {:a => {:c => 'c'}, :b => 1}
-      result = IsSameDataStructureHelper::Analyser.new(object, other, 1).perform
-      expect(result).to be false
-    end
-
-    it 'should: be invalid with depth 1 and string' do
-      object = {:a => {:c => 'c'}, :b => {:d => 'd'}}
-      other = {:a => {:c => 'c'}, :b => 'b'}
-      result = IsSameDataStructureHelper::Analyser.new(object, other, 1).perform
-      expect(result).to be false
-    end
-  end
-end

data/spec/requests/actions_controller_spec.rb

@@ -1,174 +0,0 @@
-require 'rails_helper'
-
-describe 'Requesting Actions routes', :type => :request  do
-  before(:each) do
-    allow(ForestLiana::IpWhitelist).to receive(:is_ip_whitelist_retrieved) { true }
-    allow(ForestLiana::IpWhitelist).to receive(:is_ip_valid) { true }
-    Island.create(name: 'Corsica')
-  end
-
-  after(:each) do
-    Island.destroy_all
-  end
-
-  describe 'call /values' do
-    it 'should respond 200' do
-      post '/forest/actions/foo/values', {}
-      expect(response.status).to eq(200)
-      expect(response.body).to be {}
-    end
-  end
-
-  describe 'hooks' do
-    foo = {
-        field: 'foo',
-        type: 'String',
-        default_value: nil,
-        enums: nil,
-        is_required: false,
-        reference: nil,
-        description: nil,
-        widget: nil,
-    }
-    enum = {
-        field: 'enum',
-        type: 'Enum',
-        enums: %w[a b c],
-    }
-
-    action_definition = {
-        name: 'my_action',
-        fields: [foo],
-        hooks: {
-            :load => -> (context) {
-              context[:fields]
-            },
-            :change => {
-              'foo' => -> (context) {
-                fields = context[:fields]
-                fields['foo'][:value] = 'baz'
-                return fields
-              }
-            }
-        }
-    }
-    fail_action_definition = {
-        name: 'fail_action',
-        fields: [foo],
-        hooks: {
-            :load => -> (context) {
-              1
-            },
-            :change => {
-                'foo' => -> (context) {
-                  1
-                }
-            }
-        }
-    }
-    cheat_action_definition = {
-        name: 'cheat_action',
-        fields: [foo],
-        hooks: {
-            :load => -> (context) {
-              context[:fields]['baz'] = foo.clone.update({field: 'baz'})
-              context[:fields]
-            },
-            :change => {
-                'foo' => -> (context) {
-                  context[:fields]['baz'] = foo.clone.update({field: 'baz'})
-                  context[:fields]
-                }
-            }
-        }
-    }
-    enums_action_definition = {
-      name: 'enums_action',
-      fields: [foo, enum],
-      hooks: {
-        :change => {
-          'foo' => -> (context) {
-            fields = context[:fields]
-            fields['enum'][:enums] = %w[c d e]
-            return 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)
-    island = ForestLiana.apimap.find {|collection| collection.name.to_s == ForestLiana.name_for(Island)}
-    island.actions = [action, fail_action, cheat_action, enums_action]
-
-    describe 'call /load' do
-      params = {recordIds: [1], collectionName: 'Island'}
-
-      it 'should respond 200' do
-        post '/forest/actions/my_action/hooks/load', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(200)
-        expect(JSON.parse(response.body)).to eq({'fields' => [foo.merge({:value => nil}).stringify_keys]})
-      end
-
-      it 'should respond 500 with bad params' do
-        post '/forest/actions/my_action/hooks/load', {}
-        expect(response.status).to eq(500)
-      end
-
-      it 'should respond 500 with bad hook result type' do
-        post '/forest/actions/fail_action/hooks/load', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(500)
-      end
-
-      it 'should respond 500 with bad hook result data structure' do
-        post '/forest/actions/cheat_action/hooks/load', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(500)
-      end
-    end
-
-    describe 'call /change' do
-      updated_foo = foo.clone.merge({:previousValue => nil, :value => 'bar'})
-      params = {recordIds: [1], fields: [updated_foo], collectionName: 'Island', changedField: 'foo'}
-
-      it 'should respond 200' do
-        post '/forest/actions/my_action/hooks/change', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(200)
-        expected = updated_foo.clone.merge({:value => 'baz'})
-        expected[:widgetEdit] = nil
-        expected.delete(:widget)
-        expect(JSON.parse(response.body)).to eq({'fields' => [expected.stringify_keys]})
-      end
-
-      it 'should respond 500 with bad params' do
-        post '/forest/actions/my_action/hooks/change', {}
-        expect(response.status).to eq(500)
-      end
-
-      it 'should respond 500 with bad hook result type' do
-        post '/forest/actions/fail_action/hooks/change', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(500)
-      end
-
-      it 'should respond 500 with bad hook result data structure' do
-        post '/forest/actions/cheat_action/hooks/change', JSON.dump(params), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(500)
-      end
-
-      it 'should reset value when enums has changed' do
-        updated_enum = enum.clone.merge({:previousValue => nil, :value => 'a'}) # set value to a
-        p = {recordIds: [1], fields: [updated_foo, updated_enum], collectionName: 'Island', changedField: 'foo'}
-        post '/forest/actions/enums_action/hooks/change', JSON.dump(p), 'CONTENT_TYPE' => 'application/json'
-        expect(response.status).to eq(200)
-
-        expected_enum = updated_enum.clone.merge({ :enums => %w[c d e], :value => nil, :widgetEdit => nil})
-        expected_enum.delete(:widget)
-        expected_foo = updated_foo.clone.merge({ :widgetEdit => nil})
-        expected_foo.delete(:widget)
-
-        expect(JSON.parse(response.body)).to eq({'fields' => [expected_foo.stringify_keys, expected_enum.stringify_keys]})
-      end
-
-    end
-  end
-end