with_advisory_lock 7.0.0 → 7.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2af1c67f9aa840ba30d4c1a280a74a02bd46e83cef6ee426a59e16d77afaf99f
-  data.tar.gz: 055c8a8f251605c01d91fb4ead5f5cd3ab31b4327ee1bdb2b5fbd7b24cd0e455
+  metadata.gz: f594bf1ed9b20028febee227bd33eddf70f2c3deb42ae7e8a4d83d794125281d
+  data.tar.gz: 6882cea1e84d517bdf847b8f66b7dd6bb7b6c93de09c156b459019aab324ace0
 SHA512:
-  metadata.gz: 9ea47f0a89759ee4573b7c0bf7a481e48bf7e5e6feac6b132ee42a5441478bb0d02840f1e40d5963ebc5c5bc93af47a21e4f58337e33c42091d07160994a0821
-  data.tar.gz: b584c732c8fd97dd74cc6c92286d19ef9f79fbb9911a626c092c0197028cc99540f3af5dce34fbc9579fd1c01d207c7426e57b91c0928cbb4ff3620f62bd0995
+  metadata.gz: a89eff2b9fccadd2f64ac7467c9daf19156fbac5196358b9ce7089c2d743c353e4fbb8b993d2c0440b0f253c78a56620a603cd9d41dd0e3f027e02b5f901f72c
+  data.tar.gz: 02b5903b081175e8f7ce07450738fd88d8f0f92dfc52fbb6901e9738aa90312313c3313c70ccfcdab998aa799399a173215edf913145b72d56958042681fb4e6

data/.release-please-manifest.json

@@ -1 +1 @@
-{".":"7.0.0"}
+{".":"7.0.1"}

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 ## Changelog
 
+## [7.0.1](https://github.com/ClosureTree/with_advisory_lock/compare/with_advisory_lock/v7.0.0...with_advisory_lock/v7.0.1) (2025-07-21)
+
+
+### Bug Fixes
+
+* handle ActiveRecord's release_advisory_lock signature for Rails 7.2+ ([#127](https://github.com/ClosureTree/with_advisory_lock/issues/127)) ([94253ca](https://github.com/ClosureTree/with_advisory_lock/commit/94253ca2af7f684a3c99645765853546c3da8e02)), closes [#126](https://github.com/ClosureTree/with_advisory_lock/issues/126)
+
 ## [7.0.0](https://github.com/ClosureTree/with_advisory_lock/compare/with_advisory_lock/v6.0.0...with_advisory_lock/v7.0.0) (2025-07-05)
 
 

data/lib/with_advisory_lock/mysql_advisory.rb

@@ -25,8 +25,17 @@ module WithAdvisoryLock
       execute_successful?("GET_LOCK(#{quote(lock_keys.first)}, #{mysql_timeout})")
     end
 
-    def release_advisory_lock(lock_keys, lock_name:, **)
-      execute_successful?("RELEASE_LOCK(#{quote(lock_keys.first)})")
+    def release_advisory_lock(*args, **kwargs)
+      # Handle both signatures - ActiveRecord's built-in and ours
+      if args.length == 1 && kwargs.empty?
+        # ActiveRecord's signature: release_advisory_lock(lock_id)
+        # Called by Rails migrations with a single positional argument
+        super
+      else
+        # Our signature: release_advisory_lock(lock_keys, lock_name:, **)
+        lock_keys = args.first
+        execute_successful?("RELEASE_LOCK(#{quote(lock_keys.first)})")
+      end
     rescue ActiveRecord::StatementInvalid => e
       # If the connection is broken, the lock is automatically released by MySQL
       # No need to fail the release operation

data/lib/with_advisory_lock/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module WithAdvisoryLock
-  VERSION = Gem::Version.new('7.0.0')
+  VERSION = Gem::Version.new('7.0.1')
 end

data/test/with_advisory_lock/mysql_release_lock_test.rb

@@ -0,0 +1,119 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class MySQLReleaseLockTest < GemTestCase
+  self.use_transactional_tests = false
+
+  def model_class
+    MysqlTag
+  end
+
+  def setup
+    super
+    begin
+      skip unless model_class.connection.adapter_name =~ /mysql/i
+      MysqlTag.delete_all
+    rescue ActiveRecord::NoDatabaseError
+      skip "MySQL database not available. Please create the database first."
+    rescue StandardError => e
+      skip "MySQL connection failed: #{e.message}"
+    end
+  end
+
+  test 'release_advisory_lock handles gem signature with lock_keys' do
+    lock_name = 'test_gem_signature'
+    lock_keys = model_class.connection.lock_keys_for(lock_name)
+
+    # Acquire the lock
+    result = model_class.connection.try_advisory_lock(
+      lock_keys,
+      lock_name: lock_name,
+      shared: false,
+      transaction: false
+    )
+    assert result, 'Failed to acquire lock'
+
+    # Release using gem signature
+    released = model_class.connection.release_advisory_lock(
+      lock_keys,
+      lock_name: lock_name,
+      shared: false,
+      transaction: false
+    )
+    assert released, 'Failed to release lock using gem signature'
+
+    # Verify lock is released by trying to acquire it again
+    result = model_class.connection.try_advisory_lock(
+      lock_keys,
+      lock_name: lock_name,
+      shared: false,
+      transaction: false
+    )
+    assert result, 'Lock was not properly released'
+
+    # Clean up
+    model_class.connection.release_advisory_lock(
+      lock_keys,
+      lock_name: lock_name,
+      shared: false,
+      transaction: false
+    )
+  end
+
+  test 'release_advisory_lock handles ActiveRecord signature' do
+    # Rails calls release_advisory_lock with a positional argument (lock_id)
+    # This test ensures our override doesn't break Rails' migration locking
+
+    lock_name = 'test_rails_signature'
+
+    # Acquire lock using SQL (ActiveRecord doesn't provide get_advisory_lock method)
+    lock_keys = model_class.connection.lock_keys_for(lock_name)
+    result = model_class.connection.select_value("SELECT GET_LOCK(#{model_class.connection.quote(lock_keys.first)}, 0)")
+    assert_equal 1, result, 'Failed to acquire lock using SQL'
+
+    # Release using ActiveRecord signature (positional argument, as Rails does)
+    released = model_class.connection.release_advisory_lock(lock_keys.first)
+    assert released, 'Failed to release lock using ActiveRecord signature'
+
+    # Verify lock is released
+    lock_keys = model_class.connection.lock_keys_for(lock_name)
+    result = model_class.connection.select_value("SELECT GET_LOCK(#{model_class.connection.quote(lock_keys.first)}, 0)")
+    assert_equal 1, result, 'Lock was not properly released'
+
+    # Clean up
+    model_class.connection.select_value("SELECT RELEASE_LOCK(#{model_class.connection.quote(lock_keys.first)})")
+  end
+
+  test 'release_advisory_lock handles connection errors gracefully' do
+    lock_name = 'test_connection_error'
+    lock_keys = model_class.connection.lock_keys_for(lock_name)
+
+    # Acquire the lock
+    result = model_class.connection.try_advisory_lock(
+      lock_keys,
+      lock_name: lock_name,
+      shared: false,
+      transaction: false
+    )
+    assert result, 'Failed to acquire lock'
+
+    # Simulate connection error handling
+    # The method should handle various connection error types without raising
+    begin
+      # Try to release - even if we can't simulate a real connection error,
+      # the code path exists and should work
+      model_class.connection.release_advisory_lock(
+        lock_keys,
+        lock_name: lock_name,
+        shared: false,
+        transaction: false
+      )
+    rescue StandardError => e
+      # Should not raise connection-related errors
+      refute_match(/Lost connection|MySQL server has gone away|Connection refused/i, e.message)
+      raise
+    end
+  end
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: with_advisory_lock
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.0.1
 platform: ruby
 authors:
 - Matthew McEachen
@@ -157,6 +157,7 @@ files:
 - test/with_advisory_lock/concern_test.rb
 - test/with_advisory_lock/lock_test.rb
 - test/with_advisory_lock/multi_adapter_test.rb
+- test/with_advisory_lock/mysql_release_lock_test.rb
 - test/with_advisory_lock/parallelism_test.rb
 - test/with_advisory_lock/postgresql_race_condition_test.rb
 - test/with_advisory_lock/shared_test.rb