Merge pull request #202 from iknow/rails-7-1

Support Rails 7.1

.github/workflows/test.yml

@@ -26,7 +26,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby-version: ['2.7', '3.0', '3.1', '3.2']
+        ruby-version: ['2.7', '3.0', '3.1', '3.2', '3.3']
         include:
           - ruby-version: '2.7'
             bundle-gemfile: gemfiles/rails_5_2.gemfile
@@ -36,6 +36,8 @@ jobs:
             bundle-gemfile: gemfiles/rails_6_1.gemfile
           - ruby-version: '3.2'
             bundle-gemfile: gemfiles/rails_7_0.gemfile
+          - ruby-version: '3.3'
+            bundle-gemfile: gemfiles/rails_7_1.gemfile
 
     env: # $BUNDLE_GEMFILE must be set at the job level, so it is set for all steps
       BUNDLE_GEMFILE: ${{ github.workspace }}/${{ matrix.bundle-gemfile }}

gemfiles/rails_7_1.gemfile

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

lib/iknow_view_models/version.rb

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

lib/view_model/after_transaction_runner.rb

@@ -4,6 +4,23 @@
 # `add_to_transaction`, the abstract method `after_transaction` will be invoked
 # by AR's callbacks.
 module ViewModel::AfterTransactionRunner
+  extend ActiveSupport::Concern
+
+  class_methods do
+    # Rails 7.1+ expects after transaction hooks to declare whether to run the
+    # hook on the first equivalent instance to be enqueued or the last one. For
+    # ActiveRecord models, this class method is defined on ActiveRecord::Core
+    # and delegates to Rails configuration. For us, we don't expect multiple
+    # equivalent AfterTransactionRunners for the same callback, and opting into
+    # the new behaviour would require implementing more of the ActiveRecord
+    # model interface here such as #destroyed?, so we'll lock in the older
+    # behavior that doesn't require so many inapplicable heuristics to identify
+    # the last valid callback.
+    def run_commit_callbacks_on_first_saved_instances_in_transaction
+      true
+    end
+  end
+
   # Rails' internal API
   def committed!(*)
     after_commit

nix/dependencies.nix

@@ -1,5 +1,5 @@
 {pkgs}:
 {
-  ruby = pkgs.ruby_3_0;
-  postgresql = pkgs.postgresql_12;
+  ruby = pkgs.ruby_3_2;
+  postgresql = pkgs.postgresql_14;
 }