ar_transaction_changes 1.1.9 → 1.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5871403ece76d3c6fceb41b1a5aecd240d5460363e0d86a88f4b27ba4fc1d562
-  data.tar.gz: fda845379838e83afc84c11bc23005f4af0ac8e12df804c48c5010d724b52ba3
+  metadata.gz: 714d24f3fc939b5ab4170bd5863573aea0b5f9f918d3494f51656096579d0cc6
+  data.tar.gz: 62d4814dccd3e3f5623958e1182ba8654eca9d37cc01bf986cfa09527fc5b6b5
 SHA512:
-  metadata.gz: 386cd79e7207afdc17fc9f019183fe14674c5a3af5fe6b35c87433417bfe0ce94573abbe43954f46812e22911051aa3ab80a8d47ddee7a78b6f5801b6d10fef2
-  data.tar.gz: 628f82d89b3c54455dc9c5071e9beb1610deb710bbfa158f616d886b04c9136f446bc25ad04d8f1ba9e6f3096dd5c93f64340a207cc4541f53e6420ab74db115
+  metadata.gz: 1a77baf0fd44d182e407c79ae17499933aa0a85aa1a9d2c7be3c4b1df2a0a6bee34a4d18d10bb82a5aea3b45aaa65dfde4d6254bf49b5aa8abb2dd293900b6ac
+  data.tar.gz: ab7fd9f52be45e78f6911159cb7c472ac9d78151891b1968bfcf1629ee97e65a944ef71178c5b5411f935b680d5a460f99f6884c3d8ce08aad6274b882b175f5

data/.devcontainer/devcontainer.json

@@ -0,0 +1,26 @@
+// For format details, see https://aka.ms/devcontainer.json. For config options, see the
+// README at: https://github.com/devcontainers/templates/tree/main/src/ruby
+{
+	"name": "Ruby",
+	// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
+	"image": "ghcr.io/rails/devcontainer/images/ruby:3.4.7",
+	"features": {
+		"ghcr.io/devcontainers/features/github-cli:1": {},
+		"ghcr.io/rails/devcontainer/features/mysql-client:1": {}
+	}
+
+	// Features to add to the dev container. More info: https://containers.dev/features.
+	// "features": {},
+
+	// Use 'forwardPorts' to make a list of ports inside the container available locally.
+	// "forwardPorts": [],
+
+	// Use 'postCreateCommand' to run commands after the container is created.
+	// "postCreateCommand": "ruby --version",
+
+	// Configure tool-specific properties.
+	// "customizations": {},
+
+	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
+	// "remoteUser": "root"
+}

data/.github/workflows/ci.yml

@@ -6,28 +6,51 @@ on:
     types: [opened, synchronize]
 
 jobs:
-  build:
-    if: github.event_name == 'push' || github.event.pull_request.head.repo.owner.login != 'dylanahsmith'
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: ['3.2', '3.3', '3.4']
+        rails: ['7.0', '7.1', '7.2', '8.0']
+        exclude:
+          # Rails 7.0 requires Ruby < 3.4
+          - ruby: '3.4'
+            rails: '7.0'
+
+    name: Ruby ${{ matrix.ruby }} / Rails ${{ matrix.rails }}
+
+    env:
+      BUNDLE_GEMFILE: gemfiles/rails_${{ matrix.rails }}.gemfile
 
+    steps:
+    - name: Install required packages
+      run: |
+        sudo apt-get update
+        sudo apt-get -y install libsqlite3-dev
+    - uses: actions/checkout@v3
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+    - name: Run tests
+      run: bundle exec rake
+
+  test-head:
     runs-on: ubuntu-latest
+    continue-on-error: true
 
     strategy:
+      fail-fast: false
       matrix:
-        entry:
-          - name: 'Minimum supported'
-            ruby: 2.7
-            gemfile: Gemfile.rails6
-          - name: "Latest released versions"
-            ruby: 3.2
-            gemfile: Gemfile
-          - name: "Rails main branch"
-            ruby: 3.2
-            gemfile: Gemfile.rails_head
-
-    name: ${{ matrix.entry.name }}
+        ruby: ['3.2', '3.3', '3.4']
+
+    name: Ruby ${{ matrix.ruby }} / Rails head
 
     env:
-      BUNDLE_GEMFILE: ${{ matrix.entry.gemfile }}
+      BUNDLE_GEMFILE: gemfiles/rails_head.gemfile
 
     steps:
     - name: Install required packages
@@ -38,7 +61,28 @@ jobs:
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:
-        ruby-version: ${{ matrix.entry.ruby }}
+        ruby-version: ${{ matrix.ruby }}
         bundler-cache: true
     - name: Run tests
       run: bundle exec rake
+
+  ci-summary:
+    runs-on: ubuntu-latest
+    needs: [test, test-head]
+    if: always()
+
+    name: CI Summary
+
+    steps:
+    - name: Check test results
+      run: |
+        if [ "${{ needs.test.result }}" != "success" ]; then
+          echo "❌ Required tests failed"
+          exit 1
+        fi
+        echo "✅ All required tests passed"
+        if [ "${{ needs.test-head.result }}" != "success" ]; then
+          echo "⚠️  Rails head tests failed (non-blocking)"
+        else
+          echo "✅ Rails head tests passed"
+        fi

data/.github/workflows/release.yml

@@ -0,0 +1,26 @@
+name: Release Gem
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  release:
+    permissions:
+      contents: write
+      id-token: write
+
+    environment: release
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v5
+      with:
+        persist-credentials: false
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        bundler-cache: true
+        ruby-version: ruby
+    - uses: rubygems/release-gem@v1

data/Gemfile

@@ -2,4 +2,4 @@ source 'https://rubygems.org'
 
 gemspec
 
-gem 'sqlite3', '~> 1.4'
+gem 'sqlite3'

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2012 Shopify Inc.
+Copyright (c) 2012-today Shopify Inc.
 
 MIT License
 

data/ar_transaction_changes.gemspec

@@ -6,20 +6,21 @@ require 'ar_transaction_changes/version'
 Gem::Specification.new do |gem|
   gem.name          = "ar_transaction_changes"
   gem.version       = ArTransactionChanges::VERSION
-  gem.authors       = ["Dylan Thacker-Smith"]
-  gem.email         = ["Dylan.Smith@shopify.com"]
+  gem.authors       = ["Dylan Thacker-Smith", "Shopify's Rails Infrastructure Team"]
+  gem.email         = ["Dylan.Smith@shopify.com", "rails@shopify.com"]
   gem.description   = %q{Solves the problem of trying to get all the changes to an object during a transaction in an after_commit callbacks.}
   gem.summary       = %q{Store transaction changes for active record objects}
-  gem.homepage      = "https://github.com/dylanahsmith/ar_transaction_changes"
+  gem.homepage      = "https://github.com/Shopify/ar_transaction_changes"
   gem.license       = "MIT"
 
+  gem.required_ruby_version = ">= 3.2"
+
   gem.files         = `git ls-files`.split($/)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_dependency "activerecord", ">= 5.2.0"
+  gem.add_dependency "activerecord", ">= 7.0.0"
 
   gem.add_development_dependency("rake")
-  gem.add_development_dependency("mysql2")
 end

data/gemfiles/rails_7.0.gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activerecord', github: 'rails/rails', branch: '7-0-stable'
+gem 'sqlite3', '~> 1.4'
+
+gemspec path: ".."

data/{Gemfile.rails_head → gemfiles/rails_7.1.gemfile}

@@ -1,6 +1,6 @@
 source "https://rubygems.org"
 
-gem 'activerecord', github: 'rails/rails'
+gem 'activerecord', '~> 7.1.0'
 gem 'sqlite3', '~> 1.4'
 
-gemspec
+gemspec path: ".."

data/gemfiles/rails_7.2.gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activerecord', '~> 7.2.0'
+gem 'sqlite3', '~> 1.4'
+
+gemspec path: ".."

data/gemfiles/rails_8.0.gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activerecord', '~> 8.0.0'
+gem 'sqlite3', '~> 2.1'
+
+gemspec path: ".."

data/gemfiles/rails_head.gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activerecord', github: 'rails/rails', branch: 'main'
+gem 'sqlite3'
+
+gemspec path: ".."

data/lib/ar_transaction_changes/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ArTransactionChanges
-  VERSION = "1.1.9"
+  VERSION = "1.1.10"
 end

data/lib/ar_transaction_changes.rb

@@ -4,16 +4,38 @@ require "ar_transaction_changes/version"
 require "active_record"
 
 module ArTransactionChanges
-  def _run_commit_callbacks
-    super
-  ensure
-    @transaction_changed_attributes = nil
+  def self.included(base)
+    if base.respond_to?(:_commit_callbacks) && base.respond_to?(:_rollback_callbacks)
+      if !base._commit_callbacks.empty? || !base._rollback_callbacks.empty?
+        raise "ArTransactionChanges must be included before defining any after_commit or after_rollback callbacks"
+      end
+    end
   end
 
-  def _run_rollback_callbacks
-    super
-  ensure
-    @transaction_changed_attributes = nil
+  if ActiveRecord.version >= Gem::Version.new('8.1.0.alpha')
+    def _run_commit_callbacks!
+      super
+    ensure
+      @transaction_changed_attributes = nil
+    end
+
+    def _run_rollback_callbacks!
+      super
+    ensure
+      @transaction_changed_attributes = nil
+    end
+  else
+    def _run_commit_callbacks
+      super
+    ensure
+      @transaction_changed_attributes = nil
+    end
+
+    def _run_rollback_callbacks
+      super
+    ensure
+      @transaction_changed_attributes = nil
+    end
   end
 
   def transaction_changed_attributes

data/test/models/user.rb

@@ -9,8 +9,13 @@ class User < ActiveRecord::Base
     end
   end
 
-  serialize :connection_details, Array
-  serialize :notes, Array
+  if ActiveRecord.gem_version >= Gem::Version.new('7.1.0')
+    serialize :connection_details, type: Array
+    serialize :notes, type: Array
+  else
+    serialize :connection_details, Array
+    serialize :notes, Array
+  end
 
   attr_accessor :stored_transaction_changes
 

data/test/transaction_changes_test.rb

@@ -1,6 +1,6 @@
 require 'test_helper'
 
-class TransactionChangesTest < MiniTest::Unit::TestCase
+class TransactionChangesTest < Minitest::Test
   def setup
     @user = User.new(:name => "Dylan", :occupation => "Developer", age: 20)
     @user.save!
@@ -174,4 +174,28 @@ class TransactionChangesTest < MiniTest::Unit::TestCase
 
     assert_equal [[], ['a', 'b']], @user.stored_transaction_changes['notes']
   end
+
+  def test_do_not_allow_adding_this_gem_to_a_model_after_commit_callbacks_are_defined
+    assert_raises(RuntimeError) do
+      Class.new(ActiveRecord::Base) do
+        after_commit :some_method
+
+        include ArTransactionChanges
+
+        def some_method; end
+      end
+    end
+  end
+
+    def test_do_not_allow_adding_this_gem_to_a_model_after_rollback_callbacks_are_defined
+    assert_raises(RuntimeError) do
+      Class.new(ActiveRecord::Base) do
+        after_rollback :some_method
+
+        include ArTransactionChanges
+
+        def some_method; end
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ar_transaction_changes
 version: !ruby/object:Gem::Version
-  version: 1.1.9
+  version: 1.1.10
 platform: ruby
 authors:
 - Dylan Thacker-Smith
-autorequire:
+- Shopify's Rails Infrastructure Team
 bindir: bin
 cert_chain: []
-date: 2023-01-16 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 7.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: 7.0.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -38,48 +38,39 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: mysql2
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: Solves the problem of trying to get all the changes to an object during
   a transaction in an after_commit callbacks.
 email:
 - Dylan.Smith@shopify.com
+- rails@shopify.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".devcontainer/devcontainer.json"
 - ".github/workflows/ci.yml"
+- ".github/workflows/release.yml"
 - ".gitignore"
 - Gemfile
-- Gemfile.rails6
-- Gemfile.rails_head
 - LICENSE.txt
 - README.md
 - Rakefile
 - ar_transaction_changes.gemspec
+- gemfiles/rails_7.0.gemfile
+- gemfiles/rails_7.1.gemfile
+- gemfiles/rails_7.2.gemfile
+- gemfiles/rails_8.0.gemfile
+- gemfiles/rails_head.gemfile
 - lib/ar_transaction_changes.rb
 - lib/ar_transaction_changes/version.rb
 - test/database.yml.default
 - test/models/user.rb
 - test/test_helper.rb
 - test/transaction_changes_test.rb
-homepage: https://github.com/dylanahsmith/ar_transaction_changes
+homepage: https://github.com/Shopify/ar_transaction_changes
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -87,15 +78,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.1
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Store transaction changes for active record objects
 test_files:

data/Gemfile.rails6

@@ -1,6 +0,0 @@
-source "https://rubygems.org"
-
-gem 'activerecord', '~> 6.0.0'
-gem "sqlite3", "~> 1.3", ">= 1.3.6"
-
-gemspec