with_advisory_lock 4.6.0 → 5.1.0

data/lib/with_advisory_lock.rb

@@ -1,17 +1,15 @@
 require 'with_advisory_lock/version'
 require 'active_support'
+require 'zeitwerk'
 
-module WithAdvisoryLock
-  extend ActiveSupport::Autoload
+loader = Zeitwerk::Loader.for_gem
+loader.inflector.inflect(
+  'mysql' => 'MySQL',
+  'postgresql' => 'PostgreSQL',
+  )
+loader.setup
 
-  autoload :Concern
-  autoload :Base
-  autoload :DatabaseAdapterSupport
-  autoload :Flock
-  autoload :MySQL, 'with_advisory_lock/mysql'
-  autoload :MySQLNoNesting, 'with_advisory_lock/mysql_no_nesting'
-  autoload :NestedAdvisoryLockError
-  autoload :PostgreSQL, 'with_advisory_lock/postgresql'
+module WithAdvisoryLock
 end
 
 ActiveSupport.on_load :active_record do

data/release-please-config.json

@@ -0,0 +1,9 @@
+{
+  "release-type": "ruby",
+  "packages": {
+    ".": {
+      "release-type": "ruby",
+      "package-name": "with_advisory_lock"
+    }
+  }
+}

data/test/concern_test.rb

@@ -1,20 +1,33 @@
-require 'minitest_helper'
+# frozen_string_literal: true
 
-describe "with_advisory_lock.concern" do
-  it "adds with_advisory_lock to ActiveRecord classes" do
-    assert Tag.respond_to?(:with_advisory_lock)
+require 'test_helper'
+
+class WithAdvisoryLockConcernTest < GemTestCase
+  test 'adds with_advisory_lock to ActiveRecord classes' do
+    assert_respond_to(Tag, :with_advisory_lock)
   end
 
-  it "adds with_advisory_lock to ActiveRecord instances" do
-    assert Label.new.respond_to?(:with_advisory_lock)
+  test 'adds with_advisory_lock to ActiveRecord instances' do
+    assert_respond_to(Label.new, :with_advisory_lock)
   end
 
-  it "adds advisory_lock_exists? to ActiveRecord classes" do
-    assert Tag.respond_to?(:advisory_lock_exists?)
+  test 'adds advisory_lock_exists? to ActiveRecord classes' do
+    assert_respond_to(Tag, :advisory_lock_exists?)
   end
 
-  it "adds advisory_lock_exists? to ActiveRecord classes" do
-    assert Label.new.respond_to?(:advisory_lock_exists?)
+  test 'adds advisory_lock_exists? to ActiveRecord instances' do
+    assert_respond_to(Label.new, :advisory_lock_exists?)
   end
+end
 
+class ActiveRecordQueryCacheTest < GemTestCase
+  test 'does not disable quary cache by default' do
+    ActiveRecord::Base.expects(:uncached).never
+    Tag.with_advisory_lock('lock') { Tag.first }
+  end
+
+  test 'can disable ActiveRecord query cache' do
+    ActiveRecord::Base.expects(:uncached).once
+    Tag.with_advisory_lock('a-lock', disable_query_cache: true) { Tag.first }
+  end
 end

data/test/lock_test.rb

@@ -1,47 +1,80 @@
-require 'minitest_helper'
+# frozen_string_literal: true
 
-describe 'class methods' do
-  let(:lock_name) { 'test lock' }
+require 'test_helper'
 
-  describe '.current_advisory_lock' do
-    it 'returns nil outside an advisory lock request' do
-      Tag.current_advisory_lock.must_be_nil
+class LockTest < GemTestCase
+  setup do
+    @lock_name = 'test lock'
+    @return_val = 1900
+  end
+
+  test 'returns nil outside an advisory lock request' do
+    assert_nil(Tag.current_advisory_lock)
+  end
+
+  test 'returns the name of the last lock acquired' do
+    Tag.with_advisory_lock(@lock_name) do
+      assert_match(/#{@lock_name}/, Tag.current_advisory_lock)
     end
+  end
 
-    it 'returns the name of the last lock acquired' do
-      Tag.with_advisory_lock(lock_name) do
-        # The lock name may have a prefix if WITH_ADVISORY_LOCK_PREFIX env is set
-        Tag.current_advisory_lock.must_match(/#{lock_name}/)
-      end
+  test 'can obtain a lock with a name that attempts to disrupt a SQL comment' do
+    dangerous_lock_name = 'test */ lock /*'
+    Tag.with_advisory_lock(dangerous_lock_name) do
+      assert_match(/#{Regexp.escape(dangerous_lock_name)}/, Tag.current_advisory_lock)
     end
+  end
 
-    it 'can obtain a lock with a name that attempts to disrupt a SQL comment' do
-      dangerous_lock_name = 'test */ lock /*'
-      Tag.with_advisory_lock(dangerous_lock_name) do
-        Tag.current_advisory_lock.must_match(/#{Regexp.escape(dangerous_lock_name)}/)
-      end
+  test 'returns false for an unacquired lock' do
+    refute(Tag.advisory_lock_exists?(@lock_name))
+  end
 
+  test 'returns true for an acquired lock' do
+    Tag.with_advisory_lock(@lock_name) do
+      assert(Tag.advisory_lock_exists?(@lock_name))
     end
   end
 
-  describe '.advisory_lock_exists?' do
-    it 'returns false for an unacquired lock' do
-      Tag.advisory_lock_exists?(lock_name).must_be_false
+  test 'returns block return value if lock successful' do
+    assert_equal(@return_val, Tag.with_advisory_lock!(@lock_name) { @return_val })
+  end
+
+  test 'returns false on lock acquisition failure' do
+    thread_with_lock = Thread.new do
+      Tag.with_advisory_lock(@lock_name, timeout_seconds: 0) do
+        @locked_elsewhere = true
+        loop { sleep 0.01 }
+      end
     end
 
-    it 'returns the name of the last lock acquired' do
-      Tag.with_advisory_lock(lock_name) do
-        Tag.advisory_lock_exists?(lock_name).must_be_true
+    sleep 0.01 until @locked_elsewhere
+    assert_not(Tag.with_advisory_lock(@lock_name, timeout_seconds: 0) { @return_val })
+
+    thread_with_lock.kill
+  end
+
+  test 'raises an error on lock acquisition failure' do
+    thread_with_lock = Thread.new do
+      Tag.with_advisory_lock(@lock_name, timeout_seconds: 0) do
+        @locked_elsewhere = true
+        loop { sleep 0.01 }
       end
     end
+
+    sleep 0.01 until @locked_elsewhere
+    assert_raises(WithAdvisoryLock::FailedToAcquireLock) do
+      Tag.with_advisory_lock!(@lock_name, timeout_seconds: 0) { @return_val }
+    end
+
+    thread_with_lock.kill
   end
 
-  describe 'zero timeout_seconds' do
-    it 'attempts the lock exactly once with no timeout' do
-      expected = SecureRandom.base64
-      Tag.with_advisory_lock(lock_name, 0) do
-        expected
-      end.must_equal expected
+  test 'attempts the lock exactly once with no timeout' do
+    expected = SecureRandom.base64
+    actual = Tag.with_advisory_lock(@lock_name, 0) do
+      expected
     end
+
+    assert_equal(expected, actual)
   end
 end

data/test/nesting_test.rb

@@ -1,93 +1,28 @@
-require 'minitest_helper'
+# frozen_string_literal: true
 
-describe "lock nesting" do
-  # This simplifies what we expect from the lock name:
-  before :each do
+require 'test_helper'
+
+class LockNestingTest < GemTestCase
+  setup do
     @prior_prefix = ENV['WITH_ADVISORY_LOCK_PREFIX']
     ENV['WITH_ADVISORY_LOCK_PREFIX'] = nil
   end
 
-  after :each do
+  teardown do
     ENV['WITH_ADVISORY_LOCK_PREFIX'] = @prior_prefix
   end
 
-  it "doesn't request the same lock twice" do
+  test "doesn't request the same lock twice" do
     impl = WithAdvisoryLock::Base.new(nil, nil, nil)
-    impl.lock_stack.must_be_empty
-    Tag.with_advisory_lock("first") do
-      impl.lock_stack.map(&:name).must_equal %w(first)
+    assert_empty(impl.lock_stack)
+    Tag.with_advisory_lock('first') do
+      assert_equal(%w[first], impl.lock_stack.map(&:name))
       # Even MySQL should be OK with this:
-      Tag.with_advisory_lock("first") do
-        impl.lock_stack.map(&:name).must_equal %w(first)
-      end
-      impl.lock_stack.map(&:name).must_equal %w(first)
-    end
-    impl.lock_stack.must_be_empty
-  end
-
-  it "raises errors with MySQL < 5.7.5 when acquiring nested lock" do
-    skip unless env_db == :mysql && ENV['MYSQL_VERSION'] != '5.7'
-    exc = proc {
-      Tag.with_advisory_lock("first") do
-        Tag.with_advisory_lock("second") do
-        end
-      end
-    }.must_raise WithAdvisoryLock::NestedAdvisoryLockError
-    exc.lock_stack.map(&:name).must_equal %w(first)
-  end
-
-  it "does not raise errors with MySQL < 5.7.5 when acquiring nested error force enabled" do
-    skip unless env_db == :mysql && ENV['MYSQL_VERSION'] != '5.7'
-    impl = WithAdvisoryLock::Base.new(nil, nil, nil)
-    impl.lock_stack.must_be_empty
-    Tag.with_advisory_lock("first", force_nested_lock_support: true) do
-      impl.lock_stack.map(&:name).must_equal %w(first)
-      Tag.with_advisory_lock("second", force_nested_lock_support: true) do
-        impl.lock_stack.map(&:name).must_equal %w(first second)
-        Tag.with_advisory_lock("first", force_nested_lock_support: true) do
-          # Shouldn't ask for another lock:
-          impl.lock_stack.map(&:name).must_equal %w(first second)
-          Tag.with_advisory_lock("second", force_nested_lock_support: true) do
-            # Shouldn't ask for another lock:
-            impl.lock_stack.map(&:name).must_equal %w(first second)
-          end
-        end
+      Tag.with_advisory_lock('first') do
+        assert_equal(%w[first], impl.lock_stack.map(&:name))
       end
-      impl.lock_stack.map(&:name).must_equal %w(first)
+      assert_equal(%w[first], impl.lock_stack.map(&:name))
     end
-    impl.lock_stack.must_be_empty
-  end
-
-  it "supports nested advisory locks with !MySQL 5.6" do
-    skip if env_db == :mysql && ENV['MYSQL_VERSION'] != '5.7'
-    impl = WithAdvisoryLock::Base.new(nil, nil, nil)
-    impl.lock_stack.must_be_empty
-    Tag.with_advisory_lock("first") do
-      impl.lock_stack.map(&:name).must_equal %w(first)
-      Tag.with_advisory_lock("second") do
-        impl.lock_stack.map(&:name).must_equal %w(first second)
-        Tag.with_advisory_lock("first") do
-          # Shouldn't ask for another lock:
-          impl.lock_stack.map(&:name).must_equal %w(first second)
-          Tag.with_advisory_lock("second") do
-            # Shouldn't ask for another lock:
-            impl.lock_stack.map(&:name).must_equal %w(first second)
-          end
-        end
-      end
-      impl.lock_stack.map(&:name).must_equal %w(first)
-    end
-    impl.lock_stack.must_be_empty
-  end
-
-  it "raises with !MySQL 5.6 and nested error force disabled" do
-    skip unless env_db == :mysql && ENV['MYSQL_VERSION'] != '5.7'
-    exc = proc {
-      Tag.with_advisory_lock("first", force_nested_lock_support: false) do
-        Tag.with_advisory_lock("second", force_nested_lock_support: false) do
-        end
-      end
-    }.must_raise WithAdvisoryLock::NestedAdvisoryLockError
-    exc.lock_stack.map(&:name).must_equal %w(first)
+    assert_empty(impl.lock_stack)
   end
 end

data/test/options_test.rb

@@ -1,64 +1,66 @@
-require 'minitest_helper'
+# frozen_string_literal: true
 
-describe 'options parsing' do
+require 'test_helper'
+
+class OptionsParsingTest < GemTestCase
   def parse_options(options)
     WithAdvisoryLock::Base.new(mock, mock, options)
   end
 
-  specify 'defaults (empty hash)' do
+  test 'defaults (empty hash)' do
     impl = parse_options({})
-    impl.timeout_seconds.must_be_nil
-    impl.shared.must_equal false
-    impl.transaction.must_equal false
+    assert_nil(impl.timeout_seconds)
+    assert_not(impl.shared)
+    assert_not(impl.transaction)
   end
 
-  specify 'nil sets timeout to nil' do
+  test 'nil sets timeout to nil' do
     impl = parse_options(nil)
-    impl.timeout_seconds.must_be_nil
-    impl.shared.must_equal false
-    impl.transaction.must_equal false
+    assert_nil(impl.timeout_seconds)
+    assert_not(impl.shared)
+    assert_not(impl.transaction)
   end
 
-  specify 'integer sets timeout to value' do
+  test 'integer sets timeout to value' do
     impl = parse_options(42)
-    impl.timeout_seconds.must_equal 42
-    impl.shared.must_equal false
-    impl.transaction.must_equal false
+    assert_equal(42, impl.timeout_seconds)
+    assert_not(impl.shared)
+    assert_not(impl.transaction)
   end
 
-  specify 'hash with invalid key errors' do
-    proc {
+  test 'hash with invalid key errors' do
+    assert_raises(ArgumentError) do
       parse_options(foo: 42)
-    }.must_raise ArgumentError
+    end
   end
 
-  specify 'hash with timeout_seconds sets timeout to value' do
+  test 'hash with timeout_seconds sets timeout to value' do
     impl = parse_options(timeout_seconds: 123)
-    impl.timeout_seconds.must_equal 123
-    impl.shared.must_equal false
-    impl.transaction.must_equal false
+    assert_equal(123, impl.timeout_seconds)
+    assert_not(impl.shared)
+    assert_not(impl.transaction)
   end
 
-  specify 'hash with shared option sets shared to true' do
+  test 'hash with shared option sets shared to true' do
     impl = parse_options(shared: true)
-    impl.timeout_seconds.must_be_nil
-    impl.shared.must_equal true
-    impl.transaction.must_equal false
+    assert_nil(impl.timeout_seconds)
+    assert(impl.shared)
+    assert_not(impl.transaction)
   end
 
-  specify 'hash with transaction option set transaction to true' do
+  test 'hash with transaction option set transaction to true' do
     impl = parse_options(transaction: true)
-    impl.timeout_seconds.must_be_nil
-    impl.shared.must_equal false
-    impl.transaction.must_equal true
+    assert_nil(impl.timeout_seconds)
+    assert_not(impl.shared)
+    assert(impl.transaction)
   end
 
-  specify 'hash with multiple keys sets options' do
+  test 'hash with multiple keys sets options' do
     foo = mock
     bar = mock
     impl = parse_options(timeout_seconds: foo, shared: bar)
-    impl.timeout_seconds.must_equal foo
-    impl.shared.must_equal bar
-    impl.transaction.must_equal false
+    assert_equal(foo, impl.timeout_seconds)
+    assert_equal(bar, impl.shared)
+    assert_not(impl.transaction)
   end
 end

data/test/parallelism_test.rb

@@ -1,35 +1,37 @@
-require 'minitest_helper'
+# frozen_string_literal: true
+
+require 'test_helper'
 require 'forwardable'
 
-describe 'parallelism' do
-  class FindOrCreateWorker
-    extend Forwardable
-    def_delegators :@thread, :join, :wakeup, :status, :to_s
+class FindOrCreateWorker
+  extend Forwardable
+  def_delegators :@thread, :join, :wakeup, :status, :to_s
 
-    def initialize(name, use_advisory_lock)
-      @name = name
-      @use_advisory_lock = use_advisory_lock
-      @thread = Thread.new { work_later }
-    end
+  def initialize(name, use_advisory_lock)
+    @name = name
+    @use_advisory_lock = use_advisory_lock
+    @thread = Thread.new { work_later }
+  end
 
-    def work_later
-      sleep
-      ActiveRecord::Base.connection_pool.with_connection do
-        if @use_advisory_lock
-          Tag.with_advisory_lock(@name) { work }
-        else
-          work
-        end
+  def work_later
+    sleep
+    ActiveRecord::Base.connection_pool.with_connection do
+      if @use_advisory_lock
+        Tag.with_advisory_lock(@name) { work }
+      else
+        work
       end
     end
+  end
 
-    def work
-      Tag.transaction do
-        Tag.where(name: @name).first_or_create
-      end
+  def work
+    Tag.transaction do
+      Tag.where(name: @name).first_or_create
     end
   end
+end
 
+class ParallelismTest < GemTestCase
   def run_workers
     @names = @iterations.times.map { |iter| "iteration ##{iter}" }
     @names.each do |name|
@@ -37,9 +39,7 @@ describe 'parallelism' do
         FindOrCreateWorker.new(name, @use_advisory_lock)
       end
       # Wait for all the threads to get ready:
-      until workers.all? { |ea| ea.status == 'sleep' }
-        sleep(0.1)
-      end
+      sleep(0.1) until workers.all? { |ea| ea.status == 'sleep' }
       # OK, GO!
       workers.each(&:wakeup)
       # Then wait for them to finish:
@@ -49,29 +49,27 @@ describe 'parallelism' do
     ActiveRecord::Base.connection_pool.connection
   end
 
-  before :each do
+  setup do
     ActiveRecord::Base.connection.reconnect!
     @workers = 10
   end
 
-  # < SQLite, understandably, throws "The database file is locked (database is locked)"
-
-  it 'creates multiple duplicate rows without advisory locks' do
-    skip if env_db == :sqlite
+  test 'creates multiple duplicate rows without advisory locks' do
+    skip if %i[sqlite3 jdbcsqlite3].include?(env_db)
     @use_advisory_lock = false
     @iterations = 1
     run_workers
-    Tag.all.size.must_be :>, @iterations # <- any duplicated rows will make me happy.
-    TagAudit.all.size.must_be :>, @iterations # <- any duplicated rows will make me happy.
-    Label.all.size.must_be :>, @iterations # <- any duplicated rows will make me happy.
+    assert_operator(Tag.all.size,      :>, @iterations) # <- any duplicated rows will make me happy.
+    assert_operator(TagAudit.all.size, :>, @iterations) # <- any duplicated rows will make me happy.
+    assert_operator(Label.all.size,    :>, @iterations) # <- any duplicated rows will make me happy.
   end
 
-  it "doesn't create multiple duplicate rows with advisory locks" do
+  test "doesn't create multiple duplicate rows with advisory locks" do
     @use_advisory_lock = true
     @iterations = 10
     run_workers
-    Tag.all.size.must_equal @iterations # <- any duplicated rows will NOT make me happy.
-    TagAudit.all.size.must_equal @iterations # <- any duplicated rows will NOT make me happy.
-    Label.all.size.must_equal @iterations # <- any duplicated rows will NOT make me happy.
+    assert_equal(@iterations, Tag.all.size)       # <- any duplicated rows will NOT make me happy.
+    assert_equal(@iterations, TagAudit.all.size)  # <- any duplicated rows will NOT make me happy.
+    assert_equal(@iterations, Label.all.size)     # <- any duplicated rows will NOT make me happy.
   end
 end