iknow_view_models 3.6.6 → 3.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb322edf3edec85adb744dba24cab64e15914716f16cfa5ec6bb75de5fcb8e33
-  data.tar.gz: 4b41f0f4ba1fa9a3d7568ee8b0005c4139e8186c91244c7aa04ff8b3bae19fce
+  metadata.gz: 4b0920a06f9988d3cc846a4bf1fb592525891adb09ba44e3e7ccded40ed18c2b
+  data.tar.gz: 70267048408dbef5eaeb162465ffb2583e93fcd562f5d75d38c0ea7c7dca096d
 SHA512:
-  metadata.gz: a51e6a510110c90e3b4a77d0f3da2df2191e68d12137cd9ab040a97125f799e8292ae86c9b53f39b85ebb589d785fa791e8bf56883b122d6b08bfc15b2a0f702
-  data.tar.gz: 03c790566b0304ab4333e632fd81ba8886425ee8a45f1edb2fc5c777af7e2bcbfccecaa947c8a3001d47711ed8dee4132dcb4b52284cd799b915e147de9daf35
+  metadata.gz: a47fe00e8aaac20252a8508438adcf916946f4f5e4e34beabf342d530a30e5b9dbd1837c3422cd38fec8c60ecb0c0e95d1aae76a45b47fde8a66340b5346c3b4
+  data.tar.gz: d327bfab6f0c9a985aa4b44995deb51879b9e854d3bec028ebe48dc3eedd4d14158a20e960f16ab7dfcffb80d3ce4d11979f2fb46833e91a07be3f23d056fbc3

data/iknow_view_models.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.required_ruby_version = '>= 2.7'
+  spec.required_ruby_version = '>= 2.7.3'
 
   spec.add_dependency 'actionpack', '>= 5.0'
   spec.add_dependency 'activerecord', '>= 5.0'

data/lib/iknow_view_models/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module IknowViewModels
-  VERSION = '3.6.6'
+  VERSION = '3.7.0'
 end

data/lib/view_model/callbacks.rb

@@ -11,9 +11,9 @@ module ViewModel::Callbacks
   # callbacks instance with additional instance method access to the view,
   # context and extra context-dependent parameters.
   module CallbackEnvContext
-    def method_missing(method, *args, &block)
+    def method_missing(method, ...)
       if _callbacks.respond_to?(method, true)
-        _callbacks.send(method, *args, &block)
+        _callbacks.send(method, ...)
       else
         super
       end

data/lib/view_model/migrator.rb

@@ -37,8 +37,15 @@ class ViewModel
     end
 
     def migrate!(serialization)
-      migrate_tree!(serialization, references: serialization['references'] || {})
+      references = (serialization['references'] ||= {})
+
+      migrate_tree!(serialization, references: references)
+
       GarbageCollection.garbage_collect_references!(serialization)
+
+      if references.empty?
+        serialization.delete('references')
+      end
     end
 
     private

data/lib/view_model.rb

@@ -303,12 +303,12 @@ class ViewModel
     end
   end
 
-  def to_hash(serialize_context: self.class.new_serialize_context)
+  def serialize_to_hash(serialize_context: self.class.new_serialize_context)
     Jbuilder.new { |json| serialize(json, serialize_context: serialize_context) }.attributes!
   end
 
   def to_json(serialize_context: self.class.new_serialize_context)
-    ViewModel.encode_json(self.to_hash(serialize_context: serialize_context))
+    ViewModel.encode_json(self.serialize_to_hash(serialize_context: serialize_context))
   end
 
   # Render this viewmodel to a jBuilder. Usually overridden in subclasses.

data/test/helpers/arvm_test_utilities.rb

@@ -98,7 +98,7 @@ module ARVMTestUtilities
   end
 
   def assert_serializes(vm, model, serialize_context: vm.new_serialize_context)
-    h = vm.new(model).to_hash(serialize_context: serialize_context)
+    h = vm.new(model).serialize_to_hash(serialize_context: serialize_context)
     assert_kind_of(Hash, h)
     refs = serialize_context.serialize_references_to_hash
     assert_kind_of(Hash, refs)
@@ -106,7 +106,7 @@ module ARVMTestUtilities
 
   def refute_serializes(vm, model, message = nil, serialize_context: vm.new_serialize_context)
     ex = assert_raises(ViewModel::AccessControlError) do
-      vm.new(model).to_hash(serialize_context: serialize_context)
+      vm.new(model).serialize_to_hash(serialize_context: serialize_context)
       serialize_context.serialize_references_to_hash
     end
     assert_match(message, ex.message) if message

data/test/helpers/viewmodel_spec_helpers.rb

@@ -143,8 +143,8 @@ module ViewModelSpecHelpers
 
   module ParentAndChildMigrations
     extend ActiveSupport::Concern
-  
-  def model_attributes
+
+    def model_attributes
       super.merge(
         schema: ->(t) { t.integer :new_field, default: 1, null: false },
         viewmodel: ->(_v) {

data/test/unit/view_model/active_record/controller_nested_test.rb

@@ -44,7 +44,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     assert_equal(200, childcontroller.status)
 
     expected_children = @parent.children
-    assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).to_hash } },
+    assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).serialize_to_hash } },
       childcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(childcontroller.hook_trace)
@@ -59,7 +59,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     assert_equal(200, childcontroller.status)
 
     expected_children = @parent.children + distractor.children
-    assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).to_hash } },
+    assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).serialize_to_hash } },
       childcontroller.hash_response)
   end
 
@@ -79,7 +79,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     @parent.reload
 
     assert_equal(%w[c1 c2 c3], @parent.children.order(:position).pluck(:name))
-    assert_equal({ 'data' => ChildView.new(@parent.children.last).to_hash },
+    assert_equal({ 'data' => ChildView.new(@parent.children.last).serialize_to_hash },
       childcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(childcontroller.hook_trace)
@@ -102,7 +102,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     @parent.reload
 
     assert_equal(%w[c1 c2 c3 c4], @parent.children.order(:position).pluck(:name))
-    new_children_hashes = @parent.children.last(2).map { |c| ChildView.new(c).to_hash }
+    new_children_hashes = @parent.children.last(2).map { |c| ChildView.new(c).serialize_to_hash }
     assert_equal({ 'data' => new_children_hashes },
       childcontroller.hash_response)
 
@@ -265,7 +265,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     old_child.reload
 
     assert_equal('new_name', old_child.name)
-    assert_equal({ 'data' => ChildView.new(old_child).to_hash },
+    assert_equal({ 'data' => ChildView.new(old_child).serialize_to_hash },
       childcontroller.hash_response)
   end
 
@@ -275,7 +275,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     childcontroller = ChildController.new(params: { id: old_child.id })
     childcontroller.invoke(:show)
 
-    assert_equal({ 'data' => ChildView.new(old_child).to_hash },
+    assert_equal({ 'data' => ChildView.new(old_child).serialize_to_hash },
       childcontroller.hash_response)
 
     assert_equal(200, childcontroller.status)
@@ -323,7 +323,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
 
     assert_equal(200, labelcontroller.status, labelcontroller.hash_response)
 
-    assert_equal({ 'data' => LabelView.new(old_label).to_hash },
+    assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
       labelcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(labelcontroller.hook_trace)
@@ -367,7 +367,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     old_label.reload
 
     assert_equal('new label', old_label.text)
-    assert_equal({ 'data' => LabelView.new(old_label).to_hash },
+    assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
       labelcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(labelcontroller.hook_trace)
@@ -381,7 +381,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
 
     assert_equal(200, labelcontroller.status, labelcontroller.hash_response)
 
-    assert_equal({ 'data' => LabelView.new(old_label).to_hash },
+    assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
       labelcontroller.hash_response)
   end
 
@@ -414,7 +414,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
     old_label.reload
 
     assert_equal('new label', old_label.text)
-    assert_equal({ 'data' => LabelView.new(old_label).to_hash },
+    assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
       labelcontroller.hash_response)
   end
 
@@ -472,11 +472,11 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
           'updates' => [
             {
               'id'     => @parent.id,
-              'update' => TargetView.new(target).to_hash,
+              'update' => TargetView.new(target).serialize_to_hash,
             },
             {
               'id'     => other_parent.id,
-              'update' => TargetView.new(other_target).to_hash,
+              'update' => TargetView.new(other_target).serialize_to_hash,
             },
           ].sort_by { |x| x.fetch('id') }
         }
@@ -535,7 +535,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
 
     assert_equal(
       {
-        ref_key => CategoryView.new(@parent.category).to_hash,
+        ref_key => CategoryView.new(@parent.category).serialize_to_hash,
       },
       references,
     )
@@ -591,7 +591,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
 
     assert_equal(
       {
-        ref_key => TagView.new(@parent.parent_tags.first.tag).to_hash,
+        ref_key => TagView.new(@parent.parent_tags.first.tag).serialize_to_hash,
       },
       references,
     )

data/test/unit/view_model/active_record/controller_test.rb

@@ -37,7 +37,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
     parentcontroller = ParentController.new(params: { id: @parent.id })
     parentcontroller.invoke(:show)
 
-    assert_equal({ 'data' => @parent_view.to_hash },
+    assert_equal({ 'data' => @parent_view.serialize_to_hash },
                  parentcontroller.hash_response)
 
     assert_equal(200, parentcontroller.status)
@@ -52,7 +52,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
 
     parentcontroller.invoke(:show)
 
-    expected_view = @parent_view.to_hash
+    expected_view = @parent_view.serialize_to_hash
                       .except('name')
                       .merge('old_name' => @parent.name,
                              ViewModel::VERSION_ATTRIBUTE => 1,
@@ -88,7 +88,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
     assert_equal(200, parentcontroller.status)
 
     assert_equal(parentcontroller.hash_response,
-                 { 'data' => [@parent_view.to_hash, p2_view.to_hash] })
+                 { 'data' => [@parent_view.serialize_to_hash, p2_view.serialize_to_hash] })
 
     assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)
   end
@@ -112,7 +112,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
     p2_view = ParentView.new(p2)
     assert(p2.present?, 'p2 created')
 
-    assert_equal({ 'data' => p2_view.to_hash }, parentcontroller.hash_response)
+    assert_equal({ 'data' => p2_view.serialize_to_hash }, parentcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)
   end
@@ -160,7 +160,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
     @parent.reload
 
     assert_equal('new', @parent.name)
-    assert_equal({ 'data' => @parent_view.to_hash },
+    assert_equal({ 'data' => @parent_view.serialize_to_hash },
                  parentcontroller.hash_response)
 
     assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)

data/test/unit/view_model/active_record/customization_test.rb

@@ -283,13 +283,13 @@ class ViewModel::ActiveRecord::FlattenAssociationTest < ActiveSupport::TestCase
 
   def test_serialize
     v = SectionView.new(@simplesection)
-    assert_equal(@simplesection_view, v.to_hash)
+    assert_equal(@simplesection_view, v.serialize_to_hash)
 
     v = SectionView.new(@quizsection)
-    assert_equal(@quizsection_view, v.to_hash)
+    assert_equal(@quizsection_view, v.serialize_to_hash)
 
     v = SectionView.new(@vocabsection)
-    assert_equal(@vocabsection_view, v.to_hash)
+    assert_equal(@vocabsection_view, v.serialize_to_hash)
   end
 
   def new_view_like(view)

data/test/unit/view_model/active_record/has_many_test.rb

@@ -469,8 +469,8 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
     old_children = @model1.children.order(:position)
     moved_child = old_children[1]
 
-    view = viewmodel_class.new(@model2).to_hash
-    view['children'] << child_viewmodel_class.new(moved_child).to_hash
+    view = viewmodel_class.new(@model2).serialize_to_hash
+    view['children'] << child_viewmodel_class.new(moved_child).serialize_to_hash
 
     retained_children = old_children - [moved_child]
     release_view = { '_type' => 'Model', 'id' => @model1.id,
@@ -503,7 +503,7 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
   def test_has_many_append_and_update_existing_association
     child = @model1.children[1]
 
-    cv = child_viewmodel_class.new(child).to_hash
+    cv = child_viewmodel_class.new(child).serialize_to_hash
     cv['name'] = 'newname'
 
     viewmodel_class.new(@model1).append_associated(:children, cv)
@@ -558,7 +558,7 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
   def test_move_and_edit_child_to_new
     child = @model1.children[1]
 
-    child_view = child_viewmodel_class.new(child).to_hash
+    child_view = child_viewmodel_class.new(child).serialize_to_hash
     child_view['name'] = 'changed'
 
     view = { '_type' => 'Model',
@@ -592,9 +592,9 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
   def test_move_and_edit_child_to_existing
     old_child = @model1.children[1]
 
-    old_child_view = child_viewmodel_class.new(old_child).to_hash
+    old_child_view = child_viewmodel_class.new(old_child).serialize_to_hash
     old_child_view['name'] = 'changed'
-    view = viewmodel_class.new(@model2).to_hash
+    view = viewmodel_class.new(@model2).serialize_to_hash
     view['children'] << old_child_view
 
     release_view = { '_type' => 'Model', 'id' => @model1.id,

data/test/unit/view_model/active_record/migration_test.rb

@@ -30,8 +30,16 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
   let(:current_serialization) do
     ctx = viewmodel_class.new_serialize_context
     view = ViewModel.serialize_to_hash(viewmodel, serialize_context: ctx)
-    refs = ctx.serialize_references_to_hash
-    { 'data' => view, 'references' => refs }
+
+    serialization = { 'data' => view }
+
+    # Only include 'references' key if there are actually references to
+    # serialize. This matches prerender_viewmodel.
+    if ctx.has_references?
+      serialization['references'] = ctx.serialize_references_to_hash
+    end
+
+    serialization
   end
 
   let(:v2_serialization_data) do
@@ -51,12 +59,9 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
     }
   end
 
-  let(:v2_serialization_references) { {} }
-
   let(:v2_serialization) do
     {
       'data' => v2_serialization_data,
-      'references' => v2_serialization_references,
     }
   end
 
@@ -101,8 +106,7 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
     describe 'upwards' do
       let(:migrator) { up_migrator }
       let(:subject_data) { v2_serialization_data.deep_dup }
-      let(:subject_references) { v2_serialization_references.deep_dup }
-      let(:subject) { { 'data' => subject_data, 'references' => subject_references } }
+      let(:subject) { { 'data' => subject_data } }
 
       let(:expected_result) do
         current_serialization.deep_merge(
@@ -229,12 +233,9 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
       }
     end
 
-    let(:v1_serialization_references) { {} }
-
     let(:v1_serialization) do
       {
         'data' => v1_serialization_data,
-        'references' => v1_serialization_references,
       }
     end
 
@@ -331,7 +332,6 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
           ViewModel::MIGRATED_ATTRIBUTE => true,
           'name' => viewmodel.name,
         },
-        'references' => {},
       }
     end
 
@@ -370,4 +370,114 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
       end
     end
   end
+
+  describe 'changing references' do
+    include ViewModelSpecHelpers::ParentAndBelongsToChild
+    let(:migration_versions) { { viewmodel_class => 1 } }
+
+    let(:old_field_serialization) do
+      {
+        ViewModel::TYPE_ATTRIBUTE    => 'SyntheticType',
+        ViewModel::VERSION_ATTRIBUTE => 1,
+      }
+    end
+
+    def model_attributes
+      old_field_serialization = self.old_field_serialization
+      super.merge(
+        viewmodel: ->(_v) {
+          self.schema_version = 2
+
+          # The migration from 1 -> 2 deleted a referenced field.
+          migrates from: 1, to: 2 do
+            # Put the referenced field back with a canned serialization
+            down do |view, refs|
+              view['old_field'] = { ViewModel::REFERENCE_ATTRIBUTE => 'ref:old_field' }
+              refs['ref:old_field'] = old_field_serialization
+            end
+
+            # Remove the referenced field
+            up do |view, _refs|
+              view.delete('old_field')
+            end
+          end
+        },
+      )
+    end
+
+    let(:v1_serialization_data) do
+      base = {
+        ViewModel::TYPE_ATTRIBUTE => viewmodel_class.view_name,
+        ViewModel::VERSION_ATTRIBUTE => 1,
+        ViewModel::ID_ATTRIBUTE => viewmodel.id,
+        'name' => viewmodel.name,
+        'child' => {
+          ViewModel::TYPE_ATTRIBUTE => child_viewmodel_class.view_name,
+          ViewModel::VERSION_ATTRIBUTE => 1,
+          ViewModel::ID_ATTRIBUTE => viewmodel.child.id,
+          'name' => viewmodel.child.name,
+        },
+        'old_field' => { ViewModel::REFERENCE_ATTRIBUTE => 'ref:old_field' },
+      }
+    end
+
+    let(:v1_serialization_references) do
+      { 'ref:old_field' => old_field_serialization }
+    end
+
+    let(:v1_serialization) do
+      {
+        'data' => v1_serialization_data,
+        'references' => v1_serialization_references,
+      }
+    end
+
+    describe 'adding references' do
+      let(:migrator) { down_migrator }
+      let(:subject) do
+        ser = current_serialization.deep_dup
+        raise ArgumentError.new("Expected no references") if ser.has_key?('references')
+        ser
+      end
+
+      let(:expected_result) do
+        {
+          'data' => v1_serialization_data.deep_dup.deep_merge(
+            { ViewModel::MIGRATED_ATTRIBUTE => true }
+          ),
+          'references' => v1_serialization_references
+        }
+      end
+
+      it 'migrates and returns references' do
+        migrate!
+
+        assert_equal(expected_result, subject)
+      end
+    end
+
+    describe 'removing references' do
+      let(:migrator) { up_migrator }
+      let(:subject) do
+        ser = v1_serialization.deep_dup
+        raise ArgumentError.new("Expected references") unless ser.has_key?('references')
+        ser
+      end
+
+      let(:expected_result) do
+        {
+          'data' => current_serialization.fetch('data').deep_dup.merge({
+            ViewModel::MIGRATED_ATTRIBUTE => true,
+          }),
+          # references key is absent
+        }
+      end
+
+      it 'migrates and returns references' do
+        migrate!
+
+        assert_equal(expected_result, subject)
+      end
+    end
+  end
 end

data/test/unit/view_model/active_record_test.rb

@@ -146,7 +146,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
 
     assert_raises(ViewModel::AccessControlError) do
       no_view_context = ViewModelBase.new_serialize_context(can_view: false)
-      parentview.to_hash(serialize_context: no_view_context)
+      parentview.serialize_to_hash(serialize_context: no_view_context)
     end
 
     assert_raises(ViewModel::AccessControlError) do
@@ -191,7 +191,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
 
     ex = assert_raises(ViewModel::AccessControlError) do
       # edit
-      v = ParentView.new(@parent1).to_hash.merge('name' => 'p2')
+      v = ParentView.new(@parent1).serialize_to_hash.merge('name' => 'p2')
       ParentView.deserialize_from_view(v, deserialize_context: no_edit_context)
     end
     assert_match(/Illegal edit/, ex.message)
@@ -214,7 +214,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
 
     ex = assert_raises(ViewModel::AccessControlError) do
       # edit
-      v = ParentView.new(@parent1).to_hash.merge('name' => 'p2')
+      v = ParentView.new(@parent1).serialize_to_hash.merge('name' => 'p2')
       ParentView.deserialize_from_view(v, deserialize_context: no_edit_context)
     end
     assert_match(/Illegal edit/, ex.message)

data/test/unit/view_model/callbacks_test.rb

@@ -554,6 +554,35 @@ class ViewModel::CallbacksTest < ActiveSupport::TestCase
       end
     end
 
+    describe 'delegates to methods on the callback object' do
+      class TestCallback
+        include ViewModel::Callbacks
+
+        attr_reader :a_args, :b_args
+
+        def a(*args)
+          @a_args = args
+        end
+
+        def b(*args, **kwargs)
+          @b_args = [args, kwargs]
+        end
+
+        before_visit do
+          a(1, 2, x: 1, y: 2)
+          b(1, 2, x: 1, y: 2)
+        end
+      end
+
+      let(:callback) { TestCallback.new }
+
+      it 'delegates to callback methods including kwargs' do
+        serialize(vm)
+        value(callback.a_args).must_equal([1, 2, { x: 1, y: 2 }])
+        value(callback.b_args).must_equal([[1, 2], { x: 1, y: 2 }])
+      end
+    end
+
     describe 'provides details to the execution environment' do
       class EnvCallback
         include ViewModel::Callbacks

data/test/unit/view_model/record_test.rb

@@ -204,7 +204,7 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
       def self.included(base)
         base.instance_eval do
           it 'can serialize to the expected view' do
-            h = viewmodel_class.new(subject_model).to_hash
+            h = viewmodel_class.new(subject_model).serialize_to_hash
             assert_equal(expected_view, h)
           end
         end
@@ -318,7 +318,7 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
       it 'raises correctly on an undeserializable value' do
         bad_model = subject_model.tap { |m| m.moment = 2.7 }
         ex = assert_raises(ViewModel::SerializationError) do
-          viewmodel_class.new(bad_model).to_hash
+          viewmodel_class.new(bad_model).serialize_to_hash
         end
         assert_match(/Could not serialize invalid value.*'moment'.*Incorrect type/, ex.detail)
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: iknow_view_models
 version: !ruby/object:Gem::Version
-  version: 3.6.6
+  version: 3.7.0
 platform: ruby
 authors:
 - iKnow Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-10-05 00:00:00.000000000 Z
+date: 2022-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -502,7 +502,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.7'
+      version: 2.7.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="