Support Rails 7.1, drop Ruby 2.x and Rails < 6.1

.circleci/config.yml

@@ -3,9 +3,9 @@ version: 2.1
 jobs:
   build:
     docker:
-      - image: circleci/<< parameters.ruby_version >>
-      - image: circleci/postgres:9.6.2-alpine
-      - image: circleci/mysql:5.7
+      - image: cimg/<< parameters.ruby_version >>
+      - image: cimg/postgres:12.13
+      - image: cimg/mysql:8.0
         environment:
           MYSQL_ALLOW_EMPTY_PASSWORD: "yes"
     parameters:
@@ -17,19 +17,12 @@ jobs:
       BUNDLE_GEMFILE: << parameters.gemfile >>
     steps:
       - checkout
-      # Restore Cached Dependencies
-      # - restore_cache:
-      #     keys:
-      #       - gem-cache-v1-{{ arch }}-{{ .Branch }}-{{ checksum "<< parameters.gemfile >>.lock" }}
-      #       - gem-cache-v1-{{ arch }}-{{ .Branch }}
-      #       - gem-cache-v1
 
       - run: bundle install --path vendor/bundle
 
-      # - save_cache:
-      #     key: gem-cache-v1-{{ arch }}-{{ .Branch }}-{{ checksum "<< parameters.gemfile >>.lock" }}
-      #     paths:
-      #       - vendor/bundle
+      - run:
+          name: Update apt
+          command: sudo apt update
 
       - run:
           name: Install postgres client
@@ -69,10 +62,5 @@ workflows:
       - build:
           matrix:
             parameters:
-              ruby_version: ["ruby:2.6-buster", "ruby:2.7-buster", "ruby:3.0-buster"]
-              gemfile: ["gemfiles/rails_5_2.gemfile", "gemfiles/rails_6_0.gemfile", "gemfiles/rails_6_1.gemfile", "gemfiles/rails_7_0.gemfile"]
-            exclude:
-              - ruby_version: "ruby:3.0-buster"
-                gemfile: "gemfiles/rails_5_2.gemfile"
-              - ruby_version: "ruby:2.6-buster"
-                gemfile: "gemfiles/rails_7_0.gemfile"
+              ruby_version: ["ruby:3.1.4", "ruby:3.2.2"]
+              gemfile: ["gemfiles/rails_6_1.gemfile", "gemfiles/rails_7_0.gemfile", "gemfiles/rails_7_1.gemfile"]

.ruby-version

@@ -1 +1 @@
-2.7.5
+3.1.4

Appraisals

@@ -1,28 +1,19 @@
 # frozen_string_literal: true
 
-appraise 'rails-5-2' do
-  gem 'rails', '~> 5.2.0'
-  platforms :jruby do
-    gem 'activerecord-jdbc-adapter', '~> 52.0'
-    gem 'activerecord-jdbcpostgresql-adapter', '~> 52.0'
-    gem 'activerecord-jdbcmysql-adapter', '~> 52.0'
-  end
-end
-
-appraise 'rails-6-0' do
-  gem 'rails', '~> 6.0.0'
+appraise 'rails-6-1' do
+  gem 'rails', '~> 6.1.0'
   platforms :ruby do
     gem 'sqlite3', '~> 1.4'
   end
   platforms :jruby do
-    gem 'activerecord-jdbc-adapter', '~> 60.0'
-    gem 'activerecord-jdbcpostgresql-adapter', '~> 60.0'
-    gem 'activerecord-jdbcmysql-adapter', '~> 60.0'
+    gem 'activerecord-jdbc-adapter', '~> 61.0'
+    gem 'activerecord-jdbcpostgresql-adapter', '~> 61.0'
+    gem 'activerecord-jdbcmysql-adapter', '~> 61.0'
   end
 end
 
-appraise 'rails-6-1' do
-  gem 'rails', '~> 6.1.0'
+appraise 'rails-7-0' do
+  gem 'rails', '~> 7.0.0'
   platforms :ruby do
     gem 'sqlite3', '~> 1.4'
   end
@@ -33,10 +24,10 @@ appraise 'rails-6-1' do
   end
 end
 
-appraise 'rails-7-0' do
-  gem 'rails', '~> 7.0.0'
+appraise 'rails-7-1' do
+  gem 'rails', '~> 7.1.0'
   platforms :ruby do
-    gem 'sqlite3', '~> 1.4'
+    gem 'sqlite3', '~> 1.6'
   end
   platforms :jruby do
     gem 'activerecord-jdbc-adapter', '~> 61.0'

Rakefile

@@ -132,20 +132,7 @@ def my_config
   config['mysql']
 end
 
-def activerecord_below_5_2?
-  ActiveRecord.version.release < Gem::Version.new('5.2.0')
-end
-
-def activerecord_below_6_0?
-  ActiveRecord.version.release < Gem::Version.new('6.0.0')
-end
-
 def migrate
-  if activerecord_below_5_2?
-    ActiveRecord::Migrator.migrate('spec/dummy/db/migrate')
-  elsif activerecord_below_6_0?
-    ActiveRecord::MigrationContext.new('spec/dummy/db/migrate').migrate
-  else
     # TODO: Figure out if there is any other possibility that can/should be
     # passed here as the second argument for the migration context
     ActiveRecord::MigrationContext.new('spec/dummy/db/migrate', ActiveRecord::SchemaMigration).migrate

gemfiles/rails_7_1.gemfile

@@ -0,0 +1,17 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "rails", "~> 7.1.0"
+
+platforms :ruby do
+  gem "sqlite3", "~> 1.6"
+end
+
+platforms :jruby do
+  gem "activerecord-jdbc-adapter", "~> 61.0"
+  gem "activerecord-jdbcpostgresql-adapter", "~> 61.0"
+  gem "activerecord-jdbcmysql-adapter", "~> 61.0"
+end
+
+gemspec path: "../"

lib/apartment.rb

@@ -7,15 +7,9 @@
 require 'apartment/tenant'
 
 require_relative 'apartment/log_subscriber'
-
-if ActiveRecord.version.release >= Gem::Version.new('6.0')
-  require_relative 'apartment/active_record/connection_handling'
-end
-
-if ActiveRecord.version.release >= Gem::Version.new('6.1')
-  require_relative 'apartment/active_record/schema_migration'
-  require_relative 'apartment/active_record/internal_metadata'
-end
+require_relative 'apartment/active_record/connection_handling'
+require_relative 'apartment/active_record/schema_migration'
+require_relative 'apartment/active_record/internal_metadata'
 
 # Apartment main definitions
 module Apartment
@@ -33,14 +27,10 @@ class << self
     attr_accessor(*ACCESSOR_METHODS)
     attr_writer(*WRITER_METHODS)
 
-    if ActiveRecord.version.release >= Gem::Version.new('6.1')
-      def_delegators :connection_class, :connection, :connection_db_config, :establish_connection
+    def_delegators :connection_class, :connection, :connection_db_config, :establish_connection
 
-      def connection_config
-        connection_db_config.configuration_hash
-      end
-    else
-      def_delegators :connection_class, :connection, :connection_config, :establish_connection
+    def connection_config
+      connection_db_config.configuration_hash
     end
 
     # configure apartment with available options

lib/apartment/active_record/connection_handling.rb

@@ -1,22 +1,20 @@
 # frozen_string_literal: true
 
 module ActiveRecord # :nodoc:
-  if ActiveRecord::VERSION::MAJOR >= 6
-    # This is monkeypatching Active Record to ensure that whenever a new connection is established it
-    # switches to the same tenant as before the connection switching. This problem is more evident when
-    # using read replica in Rails 6
-    module ConnectionHandling
-      def connected_to_with_tenant(role: nil, prevent_writes: false, &blk)
-        current_tenant = Apartment::Tenant.current
+  # This is monkeypatching Active Record to ensure that whenever a new connection is established it
+  # switches to the same tenant as before the connection switching. This problem is more evident when
+  # using read replica in Rails 6
+  module ConnectionHandling
+    def connected_to_with_tenant(role: nil, prevent_writes: false, &blk)
+      current_tenant = Apartment::Tenant.current
 
-        connected_to_without_tenant(role: role, prevent_writes: prevent_writes) do
-          Apartment::Tenant.switch!(current_tenant)
-          yield(blk)
-        end
+      connected_to_without_tenant(role: role, prevent_writes: prevent_writes) do
+        Apartment::Tenant.switch!(current_tenant)
+        yield(blk)
       end
-
-      alias connected_to_without_tenant connected_to
-      alias connected_to connected_to_with_tenant
     end
+
+    alias connected_to_without_tenant connected_to
+    alias connected_to connected_to_with_tenant
   end
 end

lib/apartment/active_record/schema_migration.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module ActiveRecord
-  class SchemaMigration < ActiveRecord::Base # :nodoc:
+  class SchemaMigration # :nodoc:
     class << self
       def table_exists?
         connection.table_exists?(table_name)

lib/apartment/migrator.rb

@@ -13,40 +13,22 @@ def migrate(database)
 
         migration_scope_block = ->(migration) { ENV['SCOPE'].blank? || (ENV['SCOPE'] == migration.scope) }
 
-        if activerecord_below_5_2?
-          ActiveRecord::Migrator.migrate(ActiveRecord::Migrator.migrations_paths, version, &migration_scope_block)
-        else
-          ActiveRecord::Base.connection.migration_context.migrate(version, &migration_scope_block)
-        end
+        ActiveRecord::Base.connection.migration_context.migrate(version, &migration_scope_block)
       end
     end
 
     # Migrate up/down to a specific version
     def run(direction, database, version)
       Tenant.switch(database) do
-        if activerecord_below_5_2?
-          ActiveRecord::Migrator.run(direction, ActiveRecord::Migrator.migrations_paths, version)
-        else
-          ActiveRecord::Base.connection.migration_context.run(direction, version)
-        end
+        ActiveRecord::Base.connection.migration_context.run(direction, version)
       end
     end
 
     # rollback latest migration `step` number of times
     def rollback(database, step = 1)
       Tenant.switch(database) do
-        if activerecord_below_5_2?
-          ActiveRecord::Migrator.rollback(ActiveRecord::Migrator.migrations_paths, step)
-        else
-          ActiveRecord::Base.connection.migration_context.rollback(step)
-        end
+        ActiveRecord::Base.connection.migration_context.rollback(step)
       end
     end
-
-    private
-
-    def activerecord_below_5_2?
-      ActiveRecord.version.release < Gem::Version.new('5.2.0')
-    end
   end
 end

ros-apartment.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |s|
   s.homepage = 'https://github.com/rails-on-services/apartment'
   s.licenses = ['MIT']
 
-  s.add_dependency 'activerecord', '>= 5.0.0', '< 7.1'
+  s.add_dependency 'activerecord', '>= 6.1.0', '< 7.2'
   s.add_dependency 'parallel', '< 2.0'
   s.add_dependency 'public_suffix', '>= 2.0.5', '< 5.0'
   s.add_dependency 'rack', '>= 1.3.6', '< 3.0'
@@ -39,7 +39,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rake', '~> 13.0'
   s.add_development_dependency 'rspec', '~> 3.4'
   s.add_development_dependency 'rspec_junit_formatter'
-  s.add_development_dependency 'rspec-rails', '~> 3.4'
+  s.add_development_dependency 'rspec-rails', '~> 6.1'
   s.add_development_dependency 'rubocop', '~> 0.93'
   s.add_development_dependency 'rubocop-performance', '~> 1.10'
   s.add_development_dependency 'rubocop-rails', '~> 2.1'

spec/dummy/db/migrate/20110613152810_create_dummy_models.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-migration_class = ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[4.2] : ActiveRecord::Migration
-class CreateDummyModels < migration_class
+class CreateDummyModels < ActiveRecord::Migration[4.2]
   def self.up
     create_table :companies do |t|
       t.boolean :dummy

spec/dummy/db/migrate/20111202022214_create_table_books.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-migration_class = ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[4.2] : ActiveRecord::Migration
-class CreateTableBooks < migration_class
+class CreateTableBooks < ActiveRecord::Migration[4.2]
   def up
     create_table :books do |t|
       t.string :name

spec/dummy/db/migrate/20180415260934_create_public_tokens.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-migration_class = ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[4.2] : ActiveRecord::Migration
-class CreatePublicTokens < migration_class
+class CreatePublicTokens < ActiveRecord::Migration[4.2]
   def up
     create_table :public_tokens do |t|
       t.string :token

spec/examples/schema_adapter_examples.rb

@@ -213,7 +213,7 @@
       it 'should not raise any errors' do
         expect do
           subject.switch! 'unknown_schema'
-        end.not_to raise_error(Apartment::TenantNotFound)
+        end.not_to raise_error
       end
     end
 

spec/integration/apartment_rake_integration_spec.rb

@@ -47,36 +47,9 @@
       Company.delete_all
     end
 
-    context 'with ActiveRecord below 5.2.0' do
-      before do
-        allow(ActiveRecord::Migrator).to receive(:migrations_paths) { %w[spec/dummy/db/migrate] }
-        allow(Apartment::Migrator).to receive(:activerecord_below_5_2?) { true }
-      end
-
-      describe '#migrate' do
-        it 'should migrate all databases' do
-          expect(ActiveRecord::Migrator).to receive(:migrate).exactly(company_count).times
-
-          @rake['apartment:migrate'].invoke
-        end
-      end
-
-      describe '#rollback' do
-        it 'should rollback all dbs' do
-          expect(ActiveRecord::Migrator).to receive(:rollback).exactly(company_count).times
-
-          @rake['apartment:rollback'].invoke
-        end
-      end
-    end
-
     context 'with ActiveRecord above or equal to 5.2.0' do
       let(:migration_context_double) { double(:migration_context) }
 
-      before do
-        allow(Apartment::Migrator).to receive(:activerecord_below_5_2?) { false }
-      end
-
       describe '#migrate' do
         it 'should migrate all databases' do
           allow(ActiveRecord::Base.connection).to receive(:migration_context) { migration_context_double }