lock_method 0.5.4 → 0.5.5

data/CHANGELOG

@@ -1,3 +1,10 @@
+0.5.5 / 2012-05-09
+
+* Enhancements
+
+  * Provide a more helpful exception message that names the locked method
+  * A singleton doesn't need its own single-use mutex - just use Thread.exclusive
+
 0.5.4 / 2012-04-18
 
 * Enhancements

data/lib/lock_method.rb

@@ -16,10 +16,8 @@ module LockMethod
   class Locked < ::StandardError
   end
 
-  CONFIG_MUTEX = ::Mutex.new
-
   def LockMethod.config #:nodoc:
-    @config || CONFIG_MUTEX.synchronize do
+    @config || ::Thread.exclusive do
       @config ||= Config.new
     end
   end

data/lib/lock_method/lock.rb

@@ -49,7 +49,7 @@ module LockMethod
         ::Kernel.sleep 0.5
       end
       if locked?
-        raise Locked
+        raise Locked, %{#{method_signature} is currently locked.}
       else
         begin
           save

data/lib/lock_method/version.rb

@@ -1,3 +1,3 @@
 module LockMethod
-  VERSION = "0.5.4"
+  VERSION = "0.5.5"
 end

data/test/shared_tests.rb

@@ -228,4 +228,20 @@ module SharedTests
       BlogBlock.get_latest_entries { $stderr.write "i'm now allowed" }
     end
   end
+
+  def test_016_gives_clear_exception_message
+    blocker = Thread.new { Blog2.get_latest_entries }
+  
+    # give it a bit of time to lock
+    sleep 1
+    
+    # the blocker won't have finished
+    exception = assert_raises(LockMethod::Locked) do
+      Blog2.get_latest_entries
+    end
+    assert_match /get_latest_entries/, exception.message
+  
+    # wait to finish
+    blocker.join
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lock_method
 version: !ruby/object:Gem::Version
-  version: 0.5.4
+  version: 0.5.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-18 00:00:00.000000000 Z
+date: 2012-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cache