Hotfix for support of Rails 7.2.0

.github/workflows/test.yml

@@ -59,11 +59,11 @@ jobs:
       # have set this up with each database as a separate job, but then we'd be
       # duplicating the matrix configuration three times.
       matrix:
-        gemfile: [ 'rails_6.1', 'rails_7.0', 'rails_7.1' ]
+        gemfile: [ 'rails_6.1', 'rails_7.0', 'rails_7.1', 'rails_7.2' ]
 
         # To keep matrix size down, only test highest and lowest rubies.
         # See "Lowest supported ruby version" in CONTRIBUTING.md
-        ruby: [ '3.0', '3.2' ]
+        ruby: [ '3.1', '3.3' ]
     steps:
       - name: Checkout source
         uses: actions/checkout@v4

Appraisals

@@ -23,3 +23,8 @@ appraise "rails-7.1" do
   gem "rails", "~> 7.1.0"
   gem "rails-controller-testing", "~> 1.0.5"
 end
+
+appraise "rails-7.2" do
+  gem "rails", "~> 7.2.0"
+  gem "rails-controller-testing", "~> 1.0.5"
+end

README.md

@@ -92,7 +92,7 @@ Choose version:
 
 | paper_trail | ruby     | activerecord  |
 |-------------|----------|---------------|
-| unreleased  | >= 3.0.0 | >= 6.1, < 7.2 |
+| unreleased  | >= 3.0.0 | >= 6.1, <= 7.2 |
 | 15          | >= 3.0.0 | >= 6.1, < 7.2 |
 | 14          | >= 2.7.0 | >= 6.0, < 7.1 |
 | 13          | >= 2.6.0 | >= 5.2, < 7.1 |

gemfiles/rails_7.2.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.2.0"
+gem "rails-controller-testing", "~> 1.0.5"
+
+gemspec path: "../"

lib/paper_trail/active_record_version_concern.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+# Is used to handle the deprecation warning in the different versions of ActiveRecord.
+module ActiveRecordVersionConcern
+  module_function
+
+  def deprecation
+    if Gem::Version.new(ActiveRecord::VERSION::STRING) < Gem::Version.new("7.2")
+      ::ActiveSupport::Deprecation
+    else
+      ::ActiveSupport::Deprecation._instance
+    end
+  end
+end

lib/paper_trail/compatibility.rb

@@ -18,7 +18,7 @@ module PaperTrail
   # versions.
   module Compatibility
     ACTIVERECORD_GTE = ">= 6.1" # enforced in gemspec
-    ACTIVERECORD_LT = "< 7.2" # not enforced in gemspec
+    ACTIVERECORD_LT = "< 7.3" # not enforced in gemspec
 
     E_INCOMPATIBLE_AR = <<-EOS
       PaperTrail %s is not compatible with ActiveRecord %s. We allow PT

lib/paper_trail/model_config.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "paper_trail/active_record_version_concern"
+
 module PaperTrail
   # Configures an ActiveRecord model, mostly at application boot time, but also
   # sometimes mid-request, with methods like enable/disable.
@@ -156,7 +158,7 @@ def check_version_class_name(options)
       # @api private - `version_class_name`
       @model_class.class_attribute :version_class_name
       if options[:class_name]
-        ::ActiveSupport::Deprecation.warn(
+        ActiveRecordVersionConcern.deprecation.warn(
           format(
             DPR_CLASS_NAME_OPTION,
             class_name: options[:class_name].inspect
@@ -192,7 +194,7 @@ def define_has_many_versions(options)
     def ensure_versions_option_is_hash(options)
       unless options[:versions].is_a?(Hash)
         if options[:versions]
-          ::ActiveSupport::Deprecation.warn(
+          ActiveRecordVersionConcern.deprecation.warn(
             format(
               DPR_PASSING_ASSOC_NAME_DIRECTLY_TO_VERSIONS_OPTION,
               versions_name: options[:versions].inspect

spec/paper_trail/compatibility_spec.rb

@@ -14,7 +14,7 @@ module PaperTrail
 
       context "when incompatible" do
         it "writes a warning to stderr" do
-          ar_version = ::Gem::Version.new("7.2.0")
+          ar_version = ::Gem::Version.new("8.0.0")
           expect {
             described_class.check_activerecord(ar_version)
           }.to output(/not compatible/).to_stderr

spec/paper_trail/model_config_spec.rb

@@ -7,14 +7,14 @@ module PaperTrail
     describe "has_paper_trail" do
       describe "versions:" do
         it "name can be passed instead of an options hash", :deprecated do
-          allow(::ActiveSupport::Deprecation).to receive(:warn)
+          allow(ActiveRecordVersionConcern.deprecation).to receive(:warn)
           klass = Class.new(ApplicationRecord) do
             has_paper_trail versions: :drafts
           end
           expect(klass.reflect_on_association(:drafts)).to be_a(
             ActiveRecord::Reflection::HasManyReflection
           )
-          expect(::ActiveSupport::Deprecation).to have_received(:warn).once.with(
+          expect(ActiveRecordVersionConcern.deprecation).to have_received(:warn).once.with(
             a_string_starting_with("Passing versions association name"),
             array_including(/#{__FILE__}:/)
           )
@@ -78,14 +78,14 @@ module PaperTrail
 
       describe "class_name:" do
         it "can be used instead of versions: {class_name: ...}", :deprecated do
-          allow(::ActiveSupport::Deprecation).to receive(:warn)
+          allow(ActiveRecordVersionConcern.deprecation).to receive(:warn)
           klass = Class.new(ApplicationRecord) do
             has_paper_trail class_name: "NoObjectVersion"
           end
           expect(klass.reflect_on_association(:versions).options[:class_name]).to eq(
             "NoObjectVersion"
           )
-          expect(::ActiveSupport::Deprecation).to have_received(:warn).once.with(
+          expect(ActiveRecordVersionConcern.deprecation).to have_received(:warn).once.with(
             a_string_starting_with("Passing Version class name"),
             array_including(/#{__FILE__}:/)
           )

spec/spec_helper.rb

@@ -7,7 +7,7 @@
 SimpleCov.start do
   add_filter %w[Appraisals Gemfile Rakefile doc gemfiles spec]
 end
-SimpleCov.minimum_coverage(ENV["DB"] == "postgres" ? 96.79 : 92.4)
+SimpleCov.minimum_coverage(ENV["DB"] == "postgres" ? 96.72 : 92.4)
 
 require "byebug"
 require_relative "support/pt_arel_helpers"

spec/support/paper_trail_spec_migrator.rb

@@ -19,10 +19,15 @@ def initialize
   end
 
   def migrate
-    ::ActiveRecord::MigrationContext.new(
-      @migrations_path,
-      ::ActiveRecord::Base.connection.schema_migration
-    ).migrate
+    schema_migration = if Gem::Version.new(ActiveRecord::VERSION::STRING) < Gem::Version.new("7.2")
+                         ::ActiveRecord::Base.connection.schema_migration
+                       else
+                         ::ActiveRecord::SchemaMigration.new(
+                           ActiveRecord::Tasks::DatabaseTasks.migration_connection_pool
+                         )
+                       end
+
+    ::ActiveRecord::MigrationContext.new(@migrations_path, schema_migration).migrate
   end
 
   # Generate a migration, run it, and delete it. We use this for testing the