iknow_view_models 3.2.3 → 3.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e965432165fc2b67e6241382da68703db95e8a030ba74bd2f594ce604015522
-  data.tar.gz: 1cf2b06d0fa2eacf30b5d7eefdb0a42db988d3efdd4ce7e0be0d01f560384595
+  metadata.gz: 6a61a46fa1aa74a456ed6a3d639b5bfa5a2e246b8b3772e2a34fab2bf9013722
+  data.tar.gz: 9beba4e8d176ab4ae5da14e66da70f2747a8ee68faa5ba1b79821f7dd956ab56
 SHA512:
-  metadata.gz: 776ecf2affed43082c82f7997f2ee059d61ad7d3f0e61ef8328412e9425071d926274beae58d7287d1736bd8e49ab31a1b69bb665a733db2532cd1849c3c7cd4
-  data.tar.gz: 15a9a653862ac202cc53c29b6b458d8eeaa2dba58d5b5a37a3c71bded47a688e6a45881bbcc1a29b9b6ab273b6d94ca68cdf5d7cfded145c467a77b3d964ded3
+  metadata.gz: 1ffc79be3e1479fe94f53400558133e0d98a333112502f183e8d78da9da95cb08187ab3eff20dd408ca12ed9172b74c25d15a6343407349b46b21576d2698d99
+  data.tar.gz: be2a8e88134247eed00a3adc051c741f86ac3437d6f47c921ecfd1c16d5d4d39104901aaf8cc3b29804fcd62e1c8ccfd89943d1457b89fc6b4976513e2d57069

data/lib/iknow_view_models/version.rb

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

data/lib/view_model/registry.rb

@@ -6,7 +6,8 @@ class ViewModel::Registry
   DEFERRED_NAME = Object.new
 
   class << self
-    delegate :for_view_name, :register, :default_view_name, :infer_model_class_name, :clear_removed_classes!,
+    delegate :for_view_name, :register, :default_view_name, :infer_model_class_name,
+             :clear_removed_classes!, :all, :roots,
              to: :instance
   end
 
@@ -33,6 +34,17 @@ class ViewModel::Registry
     end
   end
 
+  def all
+    @lock.synchronize do
+      resolve_deferred_classes
+      @viewmodel_classes_by_name.values
+    end
+  end
+
+  def roots
+    all.select { |c| c.root? }
+  end
+
   def register(viewmodel, as: DEFERRED_NAME)
     @lock.synchronize do
       @deferred_viewmodel_classes << [viewmodel, as]

data/test/unit/view_model/record_test.rb

@@ -365,122 +365,140 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
       end
     end
 
-    Nested = Struct.new(:member)
+    describe 'nesting' do
+      let(:nested_model_class) do
+        klass = Struct.new(:member)
+        Object.const_set(:Nested, klass)
+        klass
+      end
 
-    class NestedView < TestViewModel
-      self.view_name = 'Nested'
-      self.model_class = Nested
-      attribute :member
-    end
+      let(:nested_viewmodel_class) do
+        mc = nested_model_class
+        klass = Class.new(TestViewModel) do
+          self.view_name = 'Nested'
+          self.model_class = mc
+          attribute :member
+        end
+        Object.const_set(:NestedView, klass)
+        klass
+      end
 
-    describe 'with nested viewmodel' do
-      let(:default_nested_model) { Nested.new('member') }
-      let(:default_nested_view)  { view_base.merge('_type' => 'Nested', 'member' => 'member') }
+      def teardown
+        Object.send(:remove_const, :Nested)
+        Object.send(:remove_const, :NestedView)
+        ActiveSupport::Dependencies::Reference.clear!
+        super
+      end
 
-      let(:attributes) { { simple: {}, nested: { using: NestedView } } }
+      describe 'with nested viewmodel' do
+        let(:default_nested_model) { nested_model_class.new('member') }
+        let(:default_nested_view)  { view_base.merge('_type' => 'Nested', 'member' => 'member') }
 
-      let(:default_view_values)  { { nested: default_nested_view } }
-      let(:default_model_values) { { nested: default_nested_model } }
+        let(:attributes) { { simple: {}, nested: { using: nested_viewmodel_class } } }
 
-      let(:update_context) do
-        TestDeserializeContext.new(targets: [default_model, default_nested_model],
-                                   access_control: access_control)
-      end
+        let(:default_view_values)  { { nested: default_nested_view } }
+        let(:default_model_values) { { nested: default_nested_model } }
 
-      include CanSerialize
-      include CanDeserializeToNew
-      include CanDeserializeToExisting
+        let(:update_context) do
+          TestDeserializeContext.new(targets: [default_model, default_nested_model],
+                                     access_control: access_control)
+        end
 
-      it 'can update the nested value' do
-        new_view = default_view.merge('nested' => default_nested_view.merge('member' => 'changed'))
+        include CanSerialize
+        include CanDeserializeToNew
+        include CanDeserializeToExisting
 
-        vm = viewmodel_class.deserialize_from_view(new_view, deserialize_context: update_context)
+        it 'can update the nested value' do
+          new_view = default_view.merge('nested' => default_nested_view.merge('member' => 'changed'))
 
-        assert(default_model.equal?(vm.model), 'returned model was not the same')
-        assert(default_nested_model.equal?(vm.model.nested), 'returned nested model was not the same')
+          vm = viewmodel_class.deserialize_from_view(new_view, deserialize_context: update_context)
 
-        assert_equal('changed', default_model.nested.member)
+          assert(default_model.equal?(vm.model), 'returned model was not the same')
+          assert(default_nested_model.equal?(vm.model.nested), 'returned nested model was not the same')
 
-        assert_unchanged(vm)
-        assert_edited(vm.nested, changed_attributes: [:member])
-      end
+          assert_equal('changed', default_model.nested.member)
 
-      it 'can replace the nested value' do
-        # The value will be unified if it is different after deserialization
-        new_view = default_view.merge('nested' => default_nested_view.merge('member' => 'changed'))
+          assert_unchanged(vm)
+          assert_edited(vm.nested, changed_attributes: [:member])
+        end
 
-        partial_update_context = TestDeserializeContext.new(targets: [default_model],
-                                                            access_control: access_control)
+        it 'can replace the nested value' do
+          # The value will be unified if it is different after deserialization
+          new_view = default_view.merge('nested' => default_nested_view.merge('member' => 'changed'))
 
-        vm = viewmodel_class.deserialize_from_view(new_view, deserialize_context: partial_update_context)
+          partial_update_context = TestDeserializeContext.new(targets: [default_model],
+                                                              access_control: access_control)
 
-        assert(default_model.equal?(vm.model), 'returned model was not the same')
-        refute(default_nested_model.equal?(vm.model.nested), 'returned nested model was the same')
+          vm = viewmodel_class.deserialize_from_view(new_view, deserialize_context: partial_update_context)
 
-        assert_edited(vm, new: false, changed_attributes: [:nested])
-        assert_edited(vm.nested, new: true, changed_attributes: [:member])
+          assert(default_model.equal?(vm.model), 'returned model was not the same')
+          refute(default_nested_model.equal?(vm.model.nested), 'returned nested model was the same')
+
+          assert_edited(vm, new: false, changed_attributes: [:nested])
+          assert_edited(vm.nested, new: true, changed_attributes: [:member])
+        end
       end
-    end
 
-    describe 'with array of nested viewmodel' do
-      let(:default_nested_model_1) { Nested.new('member1') }
-      let(:default_nested_view_1)  { view_base.merge('_type' => 'Nested', 'member' => 'member1') }
+      describe 'with array of nested viewmodel' do
+        let(:default_nested_model_1) { nested_model_class.new('member1') }
+        let(:default_nested_view_1)  { view_base.merge('_type' => 'Nested', 'member' => 'member1') }
 
-      let(:default_nested_model_2) { Nested.new('member2') }
-      let(:default_nested_view_2)  { view_base.merge('_type' => 'Nested', 'member' => 'member2') }
+        let(:default_nested_model_2) { nested_model_class.new('member2') }
+        let(:default_nested_view_2)  { view_base.merge('_type' => 'Nested', 'member' => 'member2') }
 
-      let(:attributes) { { simple: {}, nested: { using: NestedView, array: true } } }
+        let(:attributes) { { simple: {}, nested: { using: nested_viewmodel_class, array: true } } }
 
-      let(:default_view_values)  { { nested: [default_nested_view_1, default_nested_view_2] } }
-      let(:default_model_values) { { nested: [default_nested_model_1, default_nested_model_2] } }
+        let(:default_view_values)  { { nested: [default_nested_view_1, default_nested_view_2] } }
+        let(:default_model_values) { { nested: [default_nested_model_1, default_nested_model_2] } }
 
-      let(:update_context) {
-        TestDeserializeContext.new(targets: [default_model, default_nested_model_1, default_nested_model_2],
-                                   access_control: access_control)
-      }
+        let(:update_context) {
+          TestDeserializeContext.new(targets: [default_model, default_nested_model_1, default_nested_model_2],
+                                     access_control: access_control)
+        }
 
-      include CanSerialize
-      include CanDeserializeToNew
-      include CanDeserializeToExisting
+        include CanSerialize
+        include CanDeserializeToNew
+        include CanDeserializeToExisting
 
-      it 'rejects change to attribute' do
-        new_view = default_view.merge('nested' => 'terrible')
-        ex = assert_raises(ViewModel::DeserializationError::InvalidAttributeType) do
-          viewmodel_class.deserialize_from_view(new_view, deserialize_context: update_context)
+        it 'rejects change to attribute' do
+          new_view = default_view.merge('nested' => 'terrible')
+          ex = assert_raises(ViewModel::DeserializationError::InvalidAttributeType) do
+            viewmodel_class.deserialize_from_view(new_view, deserialize_context: update_context)
+          end
+          assert_equal('nested', ex.attribute)
+          assert_equal('Array',  ex.expected_type)
+          assert_equal('String', ex.provided_type)
         end
-        assert_equal('nested', ex.attribute)
-        assert_equal('Array',  ex.expected_type)
-        assert_equal('String', ex.provided_type)
-      end
 
-      it 'can edit a nested value' do
-        default_view['nested'][0]['member'] = 'changed'
-        vm = viewmodel_class.deserialize_from_view(default_view, deserialize_context: update_context)
-        assert(default_model.equal?(vm.model), 'returned model was not the same')
-        assert_equal(2, vm.model.nested.size)
-        assert(default_nested_model_1.equal?(vm.model.nested[0]))
-        assert(default_nested_model_2.equal?(vm.model.nested[1]))
+        it 'can edit a nested value' do
+          default_view['nested'][0]['member'] = 'changed'
+          vm = viewmodel_class.deserialize_from_view(default_view, deserialize_context: update_context)
+          assert(default_model.equal?(vm.model), 'returned model was not the same')
+          assert_equal(2, vm.model.nested.size)
+          assert(default_nested_model_1.equal?(vm.model.nested[0]))
+          assert(default_nested_model_2.equal?(vm.model.nested[1]))
 
-        assert_unchanged(vm)
-        assert_edited(vm.nested[0], changed_attributes: [:member])
-      end
+          assert_unchanged(vm)
+          assert_edited(vm.nested[0], changed_attributes: [:member])
+        end
 
-      it 'can append a nested value' do
-        default_view['nested'] << view_base.merge('_type' => 'Nested', 'member' => 'member3')
+        it 'can append a nested value' do
+          default_view['nested'] << view_base.merge('_type' => 'Nested', 'member' => 'member3')
 
-        vm = viewmodel_class.deserialize_from_view(default_view, deserialize_context: update_context)
+          vm = viewmodel_class.deserialize_from_view(default_view, deserialize_context: update_context)
 
-        assert(default_model.equal?(vm.model), 'returned model was not the same')
-        assert_equal(3, vm.model.nested.size)
-        assert(default_nested_model_1.equal?(vm.model.nested[0]))
-        assert(default_nested_model_2.equal?(vm.model.nested[1]))
+          assert(default_model.equal?(vm.model), 'returned model was not the same')
+          assert_equal(3, vm.model.nested.size)
+          assert(default_nested_model_1.equal?(vm.model.nested[0]))
+          assert(default_nested_model_2.equal?(vm.model.nested[1]))
 
-        vm.model.nested.each_with_index do |nvm, i|
-          assert_equal("member#{i + 1}", nvm.member)
-        end
+          vm.model.nested.each_with_index do |nvm, i|
+            assert_equal("member#{i + 1}", nvm.member)
+          end
 
-        assert_edited(vm, changed_attributes: [:nested])
-        assert_edited(vm.nested[2], new: true, changed_attributes: [:member])
+          assert_edited(vm, changed_attributes: [:nested])
+          assert_edited(vm.nested[2], new: true, changed_attributes: [:member])
+        end
       end
     end
   end

data/test/unit/view_model/registry_test.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require 'minitest/autorun'
+require 'minitest/unit'
+require 'minitest/hooks'
+
+require_relative '../../helpers/arvm_test_utilities'
+require_relative '../../helpers/arvm_test_models'
+require_relative '../../helpers/viewmodel_spec_helpers'
+
+require 'view_model'
+require 'view_model/active_record'
+
+class ViewModel
+  class RegistryTest < ActiveSupport::TestCase
+    using ViewModel::Utils::Collections
+    include ARVMTestUtilities
+    extend Minitest::Spec::DSL
+    include ViewModelSpecHelpers::ParentAndBelongsToChild
+
+    before(:each) do
+      ViewModel::Registry.clear_removed_classes!
+    end
+
+    it 'registers the views' do
+      assert_equal(ViewModel::Registry.for_view_name(view_name), viewmodel_class)
+      assert_equal(ViewModel::Registry.for_view_name(child_view_name), child_viewmodel_class)
+    end
+
+    it 'enumerates the views' do
+      assert_contains_exactly([ViewModel::ErrorView, viewmodel_class, child_viewmodel_class], ViewModel::Registry.all)
+    end
+
+    it 'enumerates the root views' do
+      assert_contains_exactly([viewmodel_class], ViewModel::Registry.roots)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: iknow_view_models
 version: !ruby/object:Gem::Version
-  version: 3.2.3
+  version: 3.2.4
 platform: ruby
 authors:
 - iKnow Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-19 00:00:00.000000000 Z
+date: 2020-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -468,6 +468,7 @@ files:
 - test/unit/view_model/controller_test.rb
 - test/unit/view_model/deserialization_error/unique_violation_test.rb
 - test/unit/view_model/record_test.rb
+- test/unit/view_model/registry_test.rb
 - test/unit/view_model/traversal_context_test.rb
 - test/unit/view_model_test.rb
 homepage: https://github.com/iknow/cerego_view_models
@@ -527,5 +528,6 @@ test_files:
 - test/unit/view_model/controller_test.rb
 - test/unit/view_model/deserialization_error/unique_violation_test.rb
 - test/unit/view_model/record_test.rb
+- test/unit/view_model/registry_test.rb
 - test/unit/view_model/traversal_context_test.rb
 - test/unit/view_model_test.rb