iknow_view_models 3.14.2 → 3.14.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1cf3f71de6d7c8c39cf70b91be824338142ad42ace6520b544cc6a4b7ebd243e
-  data.tar.gz: ba711696a0db14aefd68d09073be63804aeff9a64d591b59a877cd8c1b22d1da
+  metadata.gz: d2167f858fe318caa9d87ee64a0d3a3898226f5705c596549094894be334a62f
+  data.tar.gz: d45adf1829482a676d47a820be735d3ae7385b27ad6cb7406e0d6a62b77cf7fa
 SHA512:
-  metadata.gz: b603cb052812681a8f892d6297f6cb869f0f5ee7b8976e61829ebba2ee2f9ce96834a8d3cacb136c6a7acf87d89413bd23d16dd620877a57d97b72c398bab7fa
-  data.tar.gz: 1e5f60a31a7edbd084eaf69389b31f85efcee41df34b891e7f3d11d08b12b888ba19082da45f7b48a853a98e0581b2a98c36b6d4ed1841de62c7871919c687c2
+  metadata.gz: 897fb03fe571579b5b8d2d259ac2705ac176ae779deda665b662ef90421ae09aeec829468f863f3c63e6bc1946480ffd971687a57c920477bfb65a66025d7dae
+  data.tar.gz: d931e8d90d6cf5f4440053df982d507f090a75ee6fe96cd8a511952d7b0e04a86a51624afcc209c98f75b05a852c35192ce36cd095d9ea3de38bfe1cf6f18f30

data/.github/workflows/test.yml

@@ -26,10 +26,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby-version: ['3.1', '3.2', '3.3']
+        ruby-version: ['3.2', '3.3']
         include:
-          - ruby-version: '3.1'
-            bundle-gemfile: gemfiles/rails_6_1.gemfile
           - ruby-version: '3.2'
             bundle-gemfile: gemfiles/rails_7_0.gemfile
           - ruby-version: '3.3'

data/Appraisals

@@ -1,19 +1,11 @@
-appraise 'rails-5-2' do
-  gem 'activerecord', '~> 5.2.0'
-  gem 'activesupport', '~> 5.2.0'
-end
-
-appraise 'rails-6-0' do
-  gem 'activerecord', '~> 6.0.0'
-  gem 'activesupport', '~> 6.0.0'
-end
-
-appraise 'rails-6-1' do
-  gem 'activerecord', '~> 6.1.0'
-  gem 'activesupport', '~> 6.1.0'
-end
-
 appraise 'rails-7-0' do
+  gem 'minitest-ci'
   gem 'activerecord', '~> 7.0.0'
   gem 'activesupport', '~> 7.0.0'
 end
+
+appraise 'rails-7-1' do
+  gem 'minitest-ci'
+  gem 'activerecord', '~> 7.1.0'
+  gem 'activesupport', '~> 7.1.0'
+end

data/iknow_view_models.gemspec

@@ -44,7 +44,10 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'byebug'
   spec.add_development_dependency 'method_source'
+  spec.add_development_dependency 'minitest', '~> 6.0.0'
   spec.add_development_dependency 'minitest-hooks'
+  spec.add_development_dependency 'minitest-mock'
+  spec.add_development_dependency 'minitest-reporters'
   spec.add_development_dependency 'pg'
   spec.add_development_dependency 'pry'
   spec.add_development_dependency 'rake'

data/lib/iknow_view_models/version.rb

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

data/lib/view_model/config.rb

@@ -6,6 +6,7 @@ require 'keyword_builder'
 ViewModel::Config = Value.new(
   show_cause_in_error_view: false,
   debug_deserialization: false,
+  strict_migration_versions: false,
 )
 
 class ViewModel::Config

data/lib/view_model/controller/migration_versions.rb

@@ -8,9 +8,19 @@ module ViewModel::Controller::MigrationVersions
   def migration_versions
     @migration_versions ||=
       begin
-        specified_migration_versions.reject do |viewmodel_class, required_version|
-          viewmodel_class.schema_version == required_version
-        end.freeze
+        versions = specified_migration_versions.dup
+
+        unless ViewModel::Config.strict_migration_versions
+          # Ignore the current version. This allows skipping migrations if only
+          # current versions are requested. Can't be ignored when strict
+          # migrations are enabled, since we need to walk the tree to assert
+          # that all visited types are mentioned.
+          versions.reject! do |viewmodel_class, required_version|
+            viewmodel_class.schema_version == required_version
+          end
+        end
+
+        versions.freeze
       end
   end
 

data/lib/view_model/migratable_view.rb

@@ -19,14 +19,15 @@ module ViewModel::MigratableView
       @migrations_lock   = Monitor.new
       @migration_classes = {}
       @migration_paths   = {}
-      @realized_migration_paths = true
     end
 
     def migration_path(from:, to:)
       @migrations_lock.synchronize do
-        realize_paths! unless @realized_migration_paths
+        unless @migration_paths.has_key?(to)
+          @migration_paths[to] = compute_migration_paths(to)
+        end
 
-        migrations = @migration_paths.fetch([from, to]) do
+        migrations = @migration_paths[to].fetch(from) do
           raise ViewModel::Migration::NoPathError.new(self, from, to)
         end
 
@@ -64,25 +65,35 @@ module ViewModel::MigratableView
         const_set(:"Migration_#{from}_To_#{to}", migration_class)
         @migration_classes[[from, to]] = migration_class
 
-        @realized_migration_paths = false
+        clear_migration_paths!
+      end
+    end
+
+    def clear_migration_paths!
+      @migrations_lock.synchronize do
+        @migration_paths.clear
       end
     end
 
     # Internal: find and record possible paths to the current schema version.
-    def realize_paths!
-      @migration_paths.clear
+    def compute_migration_paths(to_version)
+      unless to_version <= self.schema_version
+        raise RuntimeError.new("Cannot compute path to future version '#{to_version}'")
+      end
 
       graph = RGL::DirectedAdjacencyGraph.new
 
-      # Add a vertex for the current version, in case no edges reach it
-      graph.add_vertex(self.schema_version)
+      # Add a vertex for the destination version, in case no edges reach it
+      graph.add_vertex(to_version)
 
       # Add edges backwards, as we care about paths from the latest version
       @migration_classes.each_key do |from, to|
         graph.add_edge(to, from)
       end
 
-      paths = graph.dijkstra_shortest_paths(Hash.new { 1 }, self.schema_version)
+      paths = graph.dijkstra_shortest_paths(Hash.new { 1 }, to_version)
+
+      result = {}
 
       paths.each do |target_version, path|
         next if path.nil? || path.length == 1
@@ -92,12 +103,10 @@ module ViewModel::MigratableView
           @migration_classes.fetch([from, to])
         end
 
-        key = [target_version, schema_version]
-
-        @migration_paths[key] = path_migration_classes.map(&:new)
+        result[target_version] = path_migration_classes.map(&:new)
       end
 
-      @realized_paths = true
+      result
     end
   end
 end

data/lib/view_model/migration/strict_migration_error.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+class ViewModel::Migration::StrictMigrationError < ViewModel::AbstractError
+  attr_reader :vm_name
+
+  status 400
+  code 'Migration.StrictMigrationError'
+
+  def initialize(vm_name)
+    @vm_name = vm_name
+    super()
+  end
+
+  def detail
+    "No version was provided for the view #{vm_name}"
+  end
+
+  def meta
+    {
+      viewmodel: vm_name,
+    }
+  end
+end

data/lib/view_model/migration.rb

@@ -4,6 +4,7 @@ class ViewModel::Migration
   require 'view_model/migration/no_path_error'
   require 'view_model/migration/one_way_error'
   require 'view_model/migration/unspecified_version_error'
+  require 'view_model/migration/strict_migration_error'
 
   REFERENCE_ONLY_KEYS = [
     ViewModel::TYPE_ATTRIBUTE,

data/lib/view_model/migrator.rb

@@ -23,16 +23,19 @@ class ViewModel
       end
     end
 
+    MigrationPlan = Struct.new(:path, :required_version, :current_version, :viewmodel_class)
+
     def initialize(required_versions)
-      @paths = required_versions.each_with_object({}) do |(viewmodel_class, required_version), h|
-        if required_version != viewmodel_class.schema_version
-          path = viewmodel_class.migration_path(from: required_version, to: viewmodel_class.schema_version)
-          h[viewmodel_class.view_name] = path
-        end
-      end
+      # If strict migrations are enabled, a version must be explicitly specified
+      # for every view used
+      @strict_permitted_views = required_versions.each_key.to_set(&:view_name)
 
-      @versions = required_versions.each_with_object({}) do |(viewmodel_class, required_version), h|
-        h[viewmodel_class.view_name] = [required_version, viewmodel_class.schema_version]
+      @plans = required_versions.each_with_object({}) do |(viewmodel_class, required_version), h|
+        current_version = viewmodel_class.schema_version
+        next if required_version == current_version
+
+        path = viewmodel_class.migration_path(from: required_version, to: current_version)
+        h[viewmodel_class.view_name] = MigrationPlan.new(path, required_version, current_version, viewmodel_class)
       end
     end
 
@@ -69,6 +72,13 @@ class ViewModel
 
     private
 
+    def fetch_plan(view_name)
+      if ViewModel::Config.strict_migration_versions && !@strict_permitted_views.include?(view_name)
+        raise ViewModel::Migration::StrictMigrationError.new(view_name)
+      end
+      @plans[view_name]
+    end
+
     def migrate_tree!(node, references:)
       case node
       when Hash
@@ -128,23 +138,22 @@ class ViewModel
     end
 
     def migrate_viewmodel!(view_name, source_version, view_hash, references)
-      path = @paths[view_name]
-      return false unless path
+      plan = fetch_plan(view_name)
+      return false unless plan
 
-      required_version, current_version = @versions[view_name]
-      return false if source_version == current_version
+      return false if source_version == plan.current_version
 
       # We assume that an unspecified source version is the same as the required
       # version (i.e. the version demanded by the client request).
-      unless source_version.nil? || source_version == required_version
+      unless source_version.nil? || source_version == plan.required_version
         raise ViewModel::Migration::UnspecifiedVersionError.new(view_name, source_version)
       end
 
-      path.each do |migration|
+      plan.path.each do |migration|
         migration.up(view_hash, references)
       end
 
-      view_hash[ViewModel::VERSION_ATTRIBUTE] = current_version
+      view_hash[ViewModel::VERSION_ATTRIBUTE] = plan.current_version
 
       true
     end
@@ -156,15 +165,23 @@ class ViewModel
     private
 
     def migrate_viewmodel!(view_name, source_version, view_hash, references)
-      path = @paths[view_name]
-      return false unless path
+      plan = fetch_plan(view_name)
+      return false unless plan
 
       # In a serialized output, the source version should always be the present
       # and the current version, unless already modified by a parent migration
-      required_version, current_version = @versions[view_name]
-      return false if source_version == required_version
-
-      unless source_version == current_version
+      return false if source_version == plan.required_version
+
+      # If a parent migration has already partially migrated us to an
+      # intermediate version, we need to construct a new path to the required
+      # version.
+      if source_version == plan.current_version
+        path = plan.path
+      elsif view_hash[ViewModel::MIGRATED_ATTRIBUTE] &&
+            source_version > plan.required_version &&
+            source_version < plan.current_version
+        path = plan.viewmodel_class.migration_path(from: plan.required_version, to: source_version)
+      else
         raise ViewModel::Migration::UnspecifiedVersionError.new(view_name, source_version)
       end
 
@@ -172,7 +189,7 @@ class ViewModel
         migration.down(view_hash, references)
       end
 
-      view_hash[ViewModel::VERSION_ATTRIBUTE] = required_version
+      view_hash[ViewModel::VERSION_ATTRIBUTE] = plan.required_version
 
       true
     end

data/test/helpers/arvm_test_utilities.rb

@@ -2,7 +2,6 @@
 
 require 'logger'
 require 'active_support'
-require 'minitest/hooks'
 
 require 'view_model'
 require 'view_model/test_helpers'

data/test/helpers/match_enumerator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 # From https://stackoverflow.com/a/41293357
-module MiniTest::Assertions
+module Minitest::Assertions
   class MatchEnumerator
     def initialize(expected, actual)
       @expected = expected
@@ -54,6 +54,6 @@ module MiniTest::Assertions
     result, message = MatchEnumerator.new(expected, actual).match
     assert result, message
   end
-end # MiniTest::Assertions
+end # Minitest::Assertions
 
-Enumerator.infect_an_assertion :assert_match_enumerator, :must_contain_exactly
+Minitest::Expectation.infect_an_assertion :assert_match_enumerator, :must_contain_exactly

data/test/helpers/viewmodel_spec_helpers.rb

@@ -3,8 +3,7 @@
 require 'view_model'
 require 'view_model/test_helpers'
 
-require 'minitest/unit'
-require 'minitest/hooks'
+require 'minitest'
 
 module ViewModelSpecHelpers
   module Base

data/test/test_helper.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require 'bundler/setup'
+
+require 'minitest'
+Minitest.load_plugins
+
+require 'minitest/hooks/default'
+require 'minitest/hooks/test'
+require 'minitest/mock'
+require 'minitest/reporters'
+require 'minitest/reporters/junit_reporter'
+
+require 'rspec/expectations'
+require 'rspec/expectations/minitest_integration'
+
+FileUtils.mkdir_p('test/reports')
+Minitest::Reporters.use!(
+  [
+    Minitest::Reporters::DefaultReporter.new,
+    Minitest::Reporters::JUnitReporter.new(
+      'test/reports',
+      single_file: false,
+    ),
+  ],
+)
+
+require 'minitest/autorun'

data/test/unit/view_model/access_control_test.rb

@@ -1,14 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../test_helper'
+
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/arvm_test_models'
 require_relative '../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-require 'minitest/unit'
-
-require 'rspec/expectations/minitest_integration'
-
 require 'view_model/active_record'
 
 class ViewModel::AccessControlTest < ActiveSupport::TestCase

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::Alias < ActiveSupport::TestCase

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::BelongsToTest < ActiveSupport::TestCase

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

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative '../../../test_helper'
 
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/arvm_test_utilities'

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

@@ -1,13 +1,11 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative '../../../test_helper'
 
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-# MiniTest::Spec.register_spec_type(/./, Minitest::HooksSpec)
+# Minitest::Spec.register_spec_type(/./, Minitest::HooksSpec)
 
 require 'view_model'
 require 'view_model/active_record'

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

@@ -1,15 +1,13 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
-
-require 'view_model'
-require 'view_model/active_record'
+require_relative '../../../test_helper'
 
 require_relative '../../../helpers/controller_test_helpers'
 require_relative '../../../helpers/callback_tracer'
 
+require 'view_model'
+require 'view_model/active_record'
+
 class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
   include ARVMTestUtilities
   include ControllerTestModels

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

@@ -1,15 +1,13 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
-
-require 'view_model'
-require 'view_model/active_record'
+require_relative '../../../test_helper'
 
 require_relative '../../../helpers/controller_test_helpers'
 require_relative '../../../helpers/callback_tracer'
 
+require 'view_model'
+require 'view_model/active_record'
+
 class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
   include ARVMTestUtilities
   include ControllerTestModels

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::CounterTest < ActiveSupport::TestCase

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 require 'renum'

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::HasManyThroughPolyTest < ActiveSupport::TestCase

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::HasManyThroughTest < ActiveSupport::TestCase

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::HasOneTest < ActiveSupport::TestCase

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

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 require_relative '../../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
@@ -88,10 +88,18 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
 
       it 'migrates' do
         migrate!
-
         assert_equal(expected_result, subject)
       end
 
+      describe 'with strict migrations' do
+        it 'migrates' do
+          ViewModel::Config.stub(:strict_migration_versions, true) do
+            migrate!
+          end
+          assert_equal(expected_result, subject)
+        end
+      end
+
       describe 'to an unreachable version' do
         let(:migration_versions) { { viewmodel_class => 2, child_viewmodel_class => 1 } }
 
@@ -101,6 +109,30 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
           end
         end
       end
+
+      describe 'leaving a version out of the request' do
+        let(:migration_versions) { { viewmodel_class => 2 } }
+        let(:parent_only_expected_result) do
+          v = expected_result.deep_dup
+          v['data']['child'] = current_serialization['data']['child'].deep_dup
+          v
+        end
+
+        it 'migrates only the included view' do
+          migrate!
+          assert_equal(parent_only_expected_result, subject)
+        end
+
+        describe 'with strict migrations' do
+          it 'refuses to migrate' do
+            ViewModel::Config.stub(:strict_migration_versions, true) do
+              assert_raises(ViewModel::Migration::StrictMigrationError) do
+                migrate!
+              end
+            end
+          end
+        end
+      end
     end
 
     describe 'upwards' do
@@ -124,10 +156,18 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
 
       it 'migrates' do
         migrate!
-
         assert_equal(expected_result, subject)
       end
 
+      describe 'with strict migrations' do
+        it 'migrates' do
+          ViewModel::Config.stub(:strict_migration_versions, true) do
+            migrate!
+          end
+          assert_equal(expected_result, subject)
+        end
+      end
+
       describe 'with version unspecified' do
         let(:subject_data) do
           v2_serialization_data.except(ViewModel::VERSION_ATTRIBUTE)
@@ -139,6 +179,37 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
         end
       end
 
+      describe 'leaving a version out of the request' do
+        let(:migration_versions) { { viewmodel_class => 2 } }
+
+        let(:subject_data) do
+          v = v2_serialization_data.deep_dup
+          v['child'] = current_serialization['data']['child'].deep_dup
+          v
+        end
+
+        let(:parent_only_expected_result) do
+          v = expected_result.deep_dup
+          v['data']['child'] = current_serialization['data']['child'].deep_dup
+          v
+        end
+
+        it 'migrates only the included view' do
+          migrate!
+          assert_equal(parent_only_expected_result, subject)
+        end
+
+        describe 'with strict migrations' do
+          it 'refuses to migrate' do
+            ViewModel::Config.stub(:strict_migration_versions, true) do
+              assert_raises(ViewModel::Migration::StrictMigrationError) do
+                migrate!
+              end
+            end
+          end
+        end
+      end
+
       describe 'with a version not in the specification' do
         let(:subject_data) do
           v2_serialization_data
@@ -483,6 +554,87 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
     end
   end
 
+  describe 'partially migrating children' do
+    include ViewModelSpecHelpers::ParentAndBelongsToChild
+    let(:migration_versions) { { viewmodel_class => 1, child_viewmodel_class => 1 } }
+
+    # Parent 2->1 requires premigrating child 3->2
+    def model_attributes
+      child_viewmodel_class = self.child_viewmodel_class
+      super.merge(
+        viewmodel: ->(_v) {
+          self.schema_version = 2
+
+          # The migration from 1 -> 2 relies on a field only present in child v2
+          migrates from: 1, to: 2 do
+            down do |view, references|
+              child = view['child']
+
+              if child['_version'] > 2
+                migrator = ViewModel::DownMigrator.new({ child_viewmodel_class => 2 })
+                migrator.migrate!({ 'data' => child, 'references' => references.deep_dup })
+              end
+
+              view['child_v2_field'] = child['v2_field']
+            end
+          end
+        },
+      )
+    end
+
+    def child_attributes
+      super().merge(
+        viewmodel: ->(_v) {
+          self.schema_version = 3
+          migrates from: 1, to: 2 do
+            down do |view, _references|
+              view.delete('v2_field')
+            end
+          end
+
+          migrates from: 2, to: 3 do
+            down do |view, _references|
+              view['v2_field'] = 'v2_field_data'
+            end
+          end
+        })
+    end
+
+    let(:v1_serialization_data) do
+      {
+        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,
+        },
+        'child_v2_field' => 'v2_field_data',
+      }
+    end
+
+    let(:migrator) { down_migrator }
+    let(:subject) do
+      current_serialization.deep_dup
+    end
+
+    let(:expected_result) do
+      data = v1_serialization_data.deep_dup
+      data[ViewModel::MIGRATED_ATTRIBUTE] = true
+      data['child'][ViewModel::MIGRATED_ATTRIBUTE] = true
+
+      { 'data' => data }
+    end
+
+    it 'migrates' do
+      migrate!
+      assert_equal(expected_result, subject)
+    end
+  end
+
   describe 'concurrently inserting a reference' do
     include ViewModelSpecHelpers::ReferencedList
     let(:migration_versions) { { viewmodel_class => 1 } }

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

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative '../../../test_helper'
 
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 module ViewModel::ActiveRecord::PolyTest

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::SharedTest < ActiveSupport::TestCase

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

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../../test_helper'
+
 require_relative '../../../helpers/arvm_test_utilities'
 require_relative '../../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecord::VersionTest < ActiveSupport::TestCase

data/test/unit/view_model/active_record_test.rb

@@ -1,11 +1,10 @@
 # frozen_string_literal: true
 
+require_relative '../../test_helper'
+
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/arvm_test_models'
 
-require 'minitest/autorun'
-require 'minitest/unit'
-
 require 'view_model/active_record'
 
 class ViewModel::ActiveRecordTest < ActiveSupport::TestCase

data/test/unit/view_model/callbacks_test.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
+require_relative '../../test_helper'
+
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/arvm_test_models'
 require_relative '../../helpers/callback_tracer'
 require_relative '../../helpers/viewmodel_spec_helpers'
 
-require 'minitest/autorun'
-
 require 'view_model/active_record'
 
 class ViewModel::CallbacksTest < ActiveSupport::TestCase

data/test/unit/view_model/controller_test.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative '../../test_helper'
 
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/viewmodel_spec_helpers'

data/test/unit/view_model/deserialization_error/unique_violation_test.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'rspec/expectations/minitest_integration'
+require_relative '../../../test_helper'
 
 require 'view_model'
 require 'view_model/deserialization_error'

data/test/unit/view_model/error_wrapping_test.rb

@@ -1,9 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
-require 'rspec/expectations/minitest_integration'
+require_relative '../../test_helper'
 
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/arvm_test_models'

data/test/unit/view_model/garbage_collection_test.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
+require_relative '../../test_helper'
 
 require 'view_model'
 require 'view_model/garbage_collection'

data/test/unit/view_model/record_test.rb

@@ -1,9 +1,8 @@
 # frozen_string_literal: true
 
-require_relative '../../helpers/test_access_control'
+require_relative '../../test_helper'
 
-require 'minitest/autorun'
-require 'minitest/unit'
+require_relative '../../helpers/test_access_control'
 
 require 'view_model'
 require 'view_model/record'

data/test/unit/view_model/registry_test.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative '../../test_helper'
 
 require_relative '../../helpers/arvm_test_utilities'
 require_relative '../../helpers/arvm_test_models'

data/test/unit/view_model/traversal_context_test.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require 'minitest/autorun'
-require 'minitest/unit'
-require 'minitest/hooks'
+require_relative "../../test_helper"
 
 require_relative '../../helpers/match_enumerator'
 require_relative '../../helpers/arvm_test_utilities'

data/test/unit/view_model_test.rb

@@ -1,9 +1,6 @@
 # frozen_string_literal: true
 
-require 'bundler/setup'
-Bundler.require
-
-require 'minitest/autorun'
+require_relative '../test_helper'
 
 class DefaultViewModel < ViewModel
   self.view_name = 'DefaultViewModel'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: iknow_view_models
 version: !ruby/object:Gem::Version
-  version: 3.14.2
+  version: 3.14.4
 platform: ruby
 authors:
 - iKnow Team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-21 00:00:00.000000000 Z
+date: 2026-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -290,6 +290,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.0.0
 - !ruby/object:Gem::Dependency
   name: minitest-hooks
   requirement: !ruby/object:Gem::Requirement
@@ -304,6 +318,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-mock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-reporters
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
@@ -392,7 +434,6 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
-- gemfiles/rails_6_1.gemfile
 - gemfiles/rails_7_0.gemfile
 - gemfiles/rails_7_1.gemfile
 - iknow_view_models.gemspec
@@ -437,6 +478,7 @@ files:
 - lib/view_model/migration.rb
 - lib/view_model/migration/no_path_error.rb
 - lib/view_model/migration/one_way_error.rb
+- lib/view_model/migration/strict_migration_error.rb
 - lib/view_model/migration/unspecified_version_error.rb
 - lib/view_model/migrator.rb
 - lib/view_model/record.rb
@@ -465,6 +507,7 @@ files:
 - test/helpers/query_logging.rb
 - test/helpers/test_access_control.rb
 - test/helpers/viewmodel_spec_helpers.rb
+- test/test_helper.rb
 - test/unit/view_model/access_control_test.rb
 - test/unit/view_model/active_record/alias_test.rb
 - test/unit/view_model/active_record/belongs_to_test.rb
@@ -528,6 +571,7 @@ test_files:
 - test/helpers/query_logging.rb
 - test/helpers/test_access_control.rb
 - test/helpers/viewmodel_spec_helpers.rb
+- test/test_helper.rb
 - test/unit/view_model/access_control_test.rb
 - test/unit/view_model/active_record/alias_test.rb
 - test/unit/view_model/active_record/belongs_to_test.rb

data/gemfiles/rails_6_1.gemfile

@@ -1,10 +0,0 @@
-# This file was generated by Appraisal
-
-source 'https://rubygems.org'
-
-gem 'minitest-ci'
-gem 'activerecord', '~> 6.1.0'
-gem 'activesupport', '~> 6.1.0'
-gem 'actionpack', '~> 6.1.0'
-
-gemspec path: '../'