Fix rails 7.1 on ruby 2.7

.github/workflows/test.yml

@@ -13,6 +13,8 @@ jobs:
           - rails_7
           - rails_71
         include:
+          - ruby-version: 2.7.8
+            appraisal: rails_71
           - ruby-version: 2.5.9
             appraisal: rails_none
     env:

CHANGELOG.md

@@ -1,3 +1,6 @@
+== 0.7.1
+- fix ruby 2.7 on rails 7.1
+
 == 0.7.0
 - support for Rails 7.1
 - make install critical section smaller to prevent installation deprecations preventing installation

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    deprecation_collector (0.7.0)
+    deprecation_collector (0.7.1)
       redis (>= 3.0)
 
 GEM

gemfiles/rails_6.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    deprecation_collector (0.7.0)
+    deprecation_collector (0.7.1)
       redis (>= 3.0)
 
 GEM

gemfiles/rails_7.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    deprecation_collector (0.7.0)
+    deprecation_collector (0.7.1)
       redis (>= 3.0)
 
 GEM

gemfiles/rails_71.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    deprecation_collector (0.7.0)
+    deprecation_collector (0.7.1)
       redis (>= 3.0)
 
 GEM

gemfiles/rails_none.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    deprecation_collector (0.7.0)
+    deprecation_collector (0.7.1)
       redis (>= 3.0)
 
 GEM

lib/deprecation_collector/collectors.rb

@@ -40,7 +40,7 @@ def tap_warning_class
     def tap_activesupport
       # TODO: a more polite hook
       # not polite to turn off all other possible behaviors, but otherwise may get duplicate calls
-      if Rails.respond_to?(:gem_version) && Rails.gem_version >= "7.1"
+      if Rails.respond_to?(:gem_version) && Rails.gem_version >= Gem::Version.new("7.1")
         Rails.application.deprecators.behavior = ACTIVE_SUPPORT_BEHAVIORS[:rails71] if Rails.application&.deprecators
         # Rails.application.deprecators.behavior only captures new-style deprecations, but we need all:
         if ActiveSupport::Deprecation.respond_to?(:_instance)

lib/deprecation_collector/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class DeprecationCollector
-  VERSION = "0.7.0"
+  VERSION = "0.7.1"
 end

lib/deprecation_collector/web/helpers.rb

@@ -51,7 +51,7 @@ def trigger_rails_deprecation
         return unless defined?(ActiveSupport::Deprecation)
 
         deprecator = ActiveSupport::Deprecation
-        deprecator = ActiveSupport::Deprecation.new("0.0", "deprecation_collector") if Rails.gem_version >= "7.1"
+        deprecator = ActiveSupport::Deprecation.new("0.0", "deprecation_collector") if Rails.gem_version >= Gem::Version.new("7.1")
 
         -> { deprecator.warn("Test deprecation") } []
       end

spec/deprecation_collector/web_spec.rb

@@ -138,12 +138,15 @@
 
   it "trigger" do
     if defined?(ActiveSupport)
-      if Rails.gem_version >= "7.1"
+      if Rails.gem_version >= Gem::Version.new("7.1")
         expect_any_instance_of(ActiveSupport::Deprecation).to receive(:warn).with(a_string_matching(/Test/))
       else
         expect(ActiveSupport::Deprecation).to receive(:warn).with(a_string_matching(/Test/))
       end
     end
+    if RUBY_VERSION.start_with?("2.7")
+      expect_any_instance_of(DeprecationCollector::Web::Router::ActionContext).to receive(:trigger_kwargs_error_warning)
+    end
     expect($stderr).to receive(:puts).with(a_string_matching(/Test/))
     post "/trigger"
     expect(last_response.status).to eq(302)

spec/deprecation_collector_spec.rb

@@ -93,7 +93,7 @@
 
     describe "activesupport" do
       let(:deprecator) do
-        if Rails.gem_version >= "7.1"
+        if Rails.gem_version >= Gem::Version.new("7.1")
           ActiveSupport::Deprecation._instance
         else
           ActiveSupport::Deprecation
@@ -147,7 +147,7 @@
         end
 
         it "also collects" do
-          skip unless Rails.gem_version >= "7.1"
+          skip unless Rails.gem_version >= Gem::Version.new("7.1")
 
           expect { trigger_deprecation[] }.to change(collector, :unsent_data?).from(false).to(true)
 
@@ -169,7 +169,7 @@
         end
 
         it "also collects" do
-          skip unless Rails.gem_version >= "7.1"
+          skip unless Rails.gem_version >= Gem::Version.new("7.1")
           expect { trigger_deprecation[] }.to change(collector, :unsent_data?).from(false).to(true)
 
           collector.write_to_redis(force: true)