activerecord-safer_migrations 2.0.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 22edc1b74891b126918870a164f1213a79e92627
-  data.tar.gz: 153a817bf13c44c28a22ba7048756e4d61dcaf43
+SHA256:
+  metadata.gz: eea049e010a0e70538da15050fc12dc357e3a2a4bd7a282cabd50d0662b73abc
+  data.tar.gz: b10bff633844c10c31604a9acb6019942e767da89a389c0a03e01df48bc80001
 SHA512:
-  metadata.gz: 5cd1cfa6177e67981bf0060740d899504ab15945f59e48627b7053450483fd50bd7bbeaac137ef1410ac30b5c35a07b6d86935114b469d7902519875711c9e78
-  data.tar.gz: 75866edc3cfb0eac725953860a47f80f72212803b5f40f49cd62cc474720da6b13e83b882dfccc78f5b6b3e88369126a36a460ea45b080bb65a1b95cca4b9157
+  metadata.gz: 86e5460e1a23eae1b87c34b0df46802315ac0dd0d092de4e2e9b5802af8c656b266e983f1b8ef9d5fe72957467b6ab0742ab95a168c97d7f7b5c1dfd213bde31
+  data.tar.gz: 0d79dcfd2741bd38e2e4048c4da1839ae7f0f991bc5f089695eb886eb8dbe8010ba520accfea9499b18af0dcdb5cd61671fdd79b91f3b11601fb85393e1a9cc5

data/.github/dependabot.yml

@@ -0,0 +1,7 @@
+version: 2
+updates:
+- package-ecosystem: bundler
+  directory: "/"
+  schedule:
+    interval: daily
+  open-pull-requests-limit: 10

data/.github/workflows/tests.yml

@@ -0,0 +1,60 @@
+name: tests
+
+on:
+  push:
+    branches:
+      - "master"
+  pull_request:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true
+      - run: bundle exec rubocop --extra-details --display-style-guide --parallel --force-exclusion
+
+  tests:
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby-version: ["3.1", "3.2", "3.3"]
+        activerecord-version:
+          - "7.0.8"
+          - "7.1.3.4"
+          - "7.2.0"
+    runs-on: ubuntu-latest
+    services:
+      postgres:
+        image: postgres:14
+        env:
+          POSTGRES_USER: postgres
+          POSTGRES_DB: safer_migrations_test
+          POSTGRES_PASSWORD: safer_migrations
+        ports:
+          - 5432:5432
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 10
+    env:
+      DATABASE_URL: postgres://postgres:safer_migrations@localhost/safer_migrations_test
+      DATABASE_DEPENDENCY_PORT: "5432"
+      ACTIVERECORD_VERSION: "${{ matrix.activerecord-version }}"
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true
+          ruby-version: "${{ matrix.ruby-version }}"
+      - name: Run specs
+        run: |
+          bundle exec rspec --profile --format progress

data/.rubocop.yml

@@ -1,30 +1,17 @@
-AllCops:
-  DisplayCopNames: true
-
-# Use trailing rather than leading dots on multi-line call chains
-Style/DotPosition:
-  EnforcedStyle: trailing
-
-Style/Documentation:
-  Enabled: false
-
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
+inherit_gem:
+  gc_ruboconfig: rubocop.yml
 
-Style/TrailingComma:
-  EnforcedStyleForMultiline: comma
+AllCops:
+  TargetRubyVersion: 3.2
+  NewCops: enable
 
-Style/AccessorMethodName:
+Gemspec/RequiredRubyVersion:
   Enabled: false
 
-Style/SignalException:
-  EnforcedStyle: only_raise
+Naming/FileName:
+  Exclude:
+    - lib/activerecord-safer_migrations.rb
 
 Style/GlobalVars:
-  Enabled: false
-
-Style/FileName:
-  Enabled: false
-
-Metrics/LineLength:
-  Max: 100
+  Exclude:
+    - "spec/active_record/safer_migrations/migration_spec.rb"

data/.ruby-version

@@ -1 +1 @@
-2.4.1
+3.3.4

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+# 4.0.0 / 2024-08-21
+
+- Remove support for Ruby =< 3.0 and Rails < 7.0
+- Add support for ActiveRecord 7.2 (#98)
+
+# 3.0.0 / 2020-09-28
+
+- [#55](https://github.com/gocardless/activerecord-safer_migrations/pull/55) Drop support for Ruby =< 2.4 and Rails =< 5.1
+
 # 2.0.0 / 2017-08-23
 
 - [#23](https://github.com/gocardless/activerecord-safer_migrations/pull/23) Drop support for Rails 4.0 and 4.1

data/Gemfile

@@ -1,5 +1,13 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 gemspec
 
 gem "activerecord", "~> #{ENV['ACTIVERECORD_VERSION']}" if ENV["ACTIVERECORD_VERSION"]
+
+group :test, :development do
+  gem "gc_ruboconfig", "~> 5.0"
+  gem "pg", "~> 1.4"
+  gem "rspec", "~> 3.13.0"
+end

data/activerecord-safer_migrations.gemspec

@@ -1,4 +1,6 @@
-require File.expand_path("../lib/active_record/safer_migrations/version", __FILE__)
+# frozen_string_literal: true
+
+require File.expand_path("lib/active_record/safer_migrations/version", __dir__)
 
 Gem::Specification.new do |gem|
   gem.name          = "activerecord-safer_migrations"
@@ -12,11 +14,8 @@ Gem::Specification.new do |gem|
   gem.homepage      = "https://github.com/gocardless/activerecord-safer_migrations"
   gem.license       = "MIT"
 
-  gem.required_ruby_version = "~> 2.2"
-
-  gem.add_runtime_dependency "activerecord", ">= 4.0"
+  gem.required_ruby_version = ">= 3.1"
 
-  gem.add_development_dependency "pg", "~> 0.21.0"
-  gem.add_development_dependency "rspec", "~> 3.3.0"
-  gem.add_development_dependency "rubocop", "~> 0.35.1"
+  gem.add_dependency "activerecord", ">= 7.0"
+  gem.metadata["rubygems_mfa_required"] = "true"
 end

data/lib/active_record/safer_migrations/migration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_record/safer_migrations/setting_helper"
 
 module ActiveRecord
@@ -19,9 +21,12 @@ module ActiveRecord
         def exec_migration(conn, direction)
           # lock_timeout is an instance accessor created by class_attribute
           lock_timeout_ms = lock_timeout || SaferMigrations.default_lock_timeout
-          statement_timeout_ms = statement_timeout || SaferMigrations.default_statement_timeout
+          statement_timeout_ms = statement_timeout || SaferMigrations.
+            default_statement_timeout
           SettingHelper.new(conn, :lock_timeout, lock_timeout_ms).with_setting do
-            SettingHelper.new(conn, :statement_timeout, statement_timeout_ms).with_setting do
+            SettingHelper.new(conn,
+                              :statement_timeout,
+                              statement_timeout_ms).with_setting do
               super(conn, direction)
             end
           end
@@ -29,8 +34,10 @@ module ActiveRecord
       end
 
       module ClassMethods
+        # rubocop:disable Naming/AccessorMethodName
         def set_lock_timeout(timeout)
-          if timeout == 0
+          # rubocop:enable Naming/AccessorMethodName
+          if timeout.zero?
             raise "Setting lock_timeout to 0 is dangerous - it disables the lock " \
                   "timeout rather than instantly timing out. If you *actually* " \
                   "want to disable the lock timeout (not recommended!), use the " \
@@ -44,12 +51,14 @@ module ActiveRecord
           self.lock_timeout = 0
         end
 
+        # rubocop:disable Naming/AccessorMethodName
         def set_statement_timeout(timeout)
-          if timeout == 0
-            raise "Setting statement_timeout to 0 is dangerous - it disables the statement " \
-                  "timeout rather than instantly timing out. If you *actually* " \
-                  "want to disable the statement timeout (not recommended!), use the " \
-                  "`disable_statement_timeout!` method."
+          # rubocop:enable Naming/AccessorMethodName
+          if timeout.zero?
+            raise "Setting statement_timeout to 0 is dangerous - it disables the " \
+                  "statement timeout rather than instantly timing out. If you " \
+                  "*actually* want to disable the statement timeout (not recommended!)" \
+                  ", use the `disable_statement_timeout!` method."
           end
           self.statement_timeout = timeout
         end

data/lib/active_record/safer_migrations/postgresql_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module SaferMigrations
     module PostgreSQLAdapter
@@ -31,7 +33,11 @@ module ActiveRecord
       end
 
       def fill_sql_values(sql, values)
-        ActiveRecord::Base.send(:replace_named_bind_variables, sql, values)
+        if ActiveRecord.version >= "7.2.0"
+          ActiveRecord::Base.send(:replace_named_bind_variables, self, sql, values)
+        else
+          ActiveRecord::Base.send(:replace_named_bind_variables, sql, values)
+        end
       end
     end
   end

data/lib/active_record/safer_migrations/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module SaferMigrations
     class Railtie < Rails::Railtie

data/lib/active_record/safer_migrations/setting_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module SaferMigrations
     class SettingHelper
@@ -20,7 +22,7 @@ module ActiveRecord
         set_new_setting
         yield
         reset_setting
-      rescue
+      rescue StandardError
         reset_setting unless in_transaction?
         raise
       end
@@ -42,7 +44,7 @@ module ActiveRecord
       end
 
       def in_transaction?
-        ActiveRecord::Base.connection.open_transactions > 0
+        ActiveRecord::Base.connection.open_transactions.positive?
       end
     end
   end

data/lib/active_record/safer_migrations/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module SaferMigrations
-    VERSION = "2.0.0".freeze
+    VERSION = "4.0.0"
   end
 end

data/lib/activerecord-safer_migrations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_record/connection_adapters/postgresql_adapter"
 require "active_record/safer_migrations/postgresql_adapter"
 require "active_record/safer_migrations/migration"
@@ -35,4 +37,4 @@ module ActiveRecord
   end
 end
 
-require "active_record/safer_migrations/railtie" if defined?(::Rails)
+require "active_record/safer_migrations/railtie" if defined?(Rails)

data/spec/active_record/safer_migrations/migration_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 RSpec.describe ActiveRecord::SaferMigrations::Migration do
@@ -9,13 +11,17 @@ RSpec.describe ActiveRecord::SaferMigrations::Migration do
     end
   end
 
-  before { nuke_migrations }
-  before { TimeoutTestHelpers.set(:lock_timeout, 0) }
-  before { TimeoutTestHelpers.set(:statement_timeout, 0) }
+  before do
+    nuke_migrations
+    TimeoutTestHelpers.set(:lock_timeout, 0)
+    TimeoutTestHelpers.set(:statement_timeout, 0)
+  end
 
   describe "setting timeouts explicitly" do
-    before { $lock_timeout = nil }
-    before { $statement_timeout = nil }
+    before do
+      $lock_timeout = nil
+      $statement_timeout = nil
+    end
 
     shared_examples_for "running the migration" do
       let(:migration) do
@@ -51,8 +57,10 @@ RSpec.describe ActiveRecord::SaferMigrations::Migration do
       end
 
       context "when the original timeout is not 0" do
-        before { TimeoutTestHelpers.set(:lock_timeout, 8000) }
-        before { TimeoutTestHelpers.set(:statement_timeout, 8001) }
+        before do
+          TimeoutTestHelpers.set(:lock_timeout, 8000)
+          TimeoutTestHelpers.set(:statement_timeout, 8001)
+        end
 
         it "unsets the lock timeout after the migration" do
           silence_stream($stdout) { run_migration.call }
@@ -82,10 +90,13 @@ RSpec.describe ActiveRecord::SaferMigrations::Migration do
   end
 
   describe "the default timeouts" do
-    before { $lock_timeout = nil }
-    before { $statement_timeout = nil }
-    before { ActiveRecord::SaferMigrations.default_lock_timeout = 6000 }
-    before { ActiveRecord::SaferMigrations.default_statement_timeout = 6001 }
+    before do
+      $lock_timeout = nil
+      $statement_timeout = nil
+      ActiveRecord::SaferMigrations.default_lock_timeout = 6000
+      ActiveRecord::SaferMigrations.default_statement_timeout = 6001
+    end
+
     let(:migration) do
       Class.new(migration_base_class) do
         def change
@@ -117,10 +128,13 @@ RSpec.describe ActiveRecord::SaferMigrations::Migration do
   end
 
   describe "when inheriting from a migration with timeouts defined" do
-    before { $lock_timeout = nil }
-    before { $statement_timeout = nil }
-    before { ActiveRecord::SaferMigrations.default_lock_timeout = 6000 }
-    before { ActiveRecord::SaferMigrations.default_statement_timeout = 6001 }
+    before do
+      $lock_timeout = nil
+      $statement_timeout = nil
+      ActiveRecord::SaferMigrations.default_lock_timeout = 6000
+      ActiveRecord::SaferMigrations.default_statement_timeout = 6001
+    end
+
     let(:base_migration) do
       Class.new(migration_base_class) do
         set_lock_timeout(7000)

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_record"
 require "activerecord-safer_migrations"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-safer_migrations
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 4.0.0
 platform: ruby
 authors:
 - GoCardless Engineering
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-08-23 00:00:00.000000000 Z
+date: 2024-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,66 +16,25 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
-- !ruby/object:Gem::Dependency
-  name: pg
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.21.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.21.0
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.3.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.3.0
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.35.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.35.1
+        version: '7.0'
 description: ''
 email: developers@gocardless.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
+- ".github/workflows/tests.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".ruby-version"
-- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -92,25 +51,25 @@ files:
 homepage: https://github.com/gocardless/activerecord-safer_migrations
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - "~>"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '2.2'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
-signing_key: 
+rubygems_version: 3.5.11
+signing_key:
 specification_version: 4
 summary: ActiveRecord migration helpers to avoid downtime
 test_files: []

data/.travis.yml

@@ -1,24 +0,0 @@
-language: ruby
-
-rvm:
-  - 2.4.1
-  - 2.3.4
-  - 2.2.6
-
-addons:
-  postgresql: "9.3"
-
-before_script:
-  - psql -c 'CREATE DATABASE safer_migrations_test;' -U postgres
-
-script:
-  - bundle exec rspec spec
-  - bundle exec rubocop
-
-env:
-  global:
-    - "DATABASE_URL=postgres://postgres@localhost/safer_migrations_test"
-  matrix:
-    - "ACTIVERECORD_VERSION=4.2.9"
-    - "ACTIVERECORD_VERSION=5.0.5"
-    - "ACTIVERECORD_VERSION=5.1.3"