redis-semaphore 0.2.3 → 0.2.3.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NmRjZDI2ZDcyOWE2MWM2ZjMwZDA4NjI4YmUwNjVhZjk2MzQ5OWU0Mg==
+    NThjZDI5NWY4MjY4Zjc2OWU3ODgxMTUwYjM1NDMyNjEyNjU0ZmY3YQ==
   data.tar.gz: !binary |-
-    Yjc5ZDg1MjM0MThkMTcyMDJiYjU5MThhY2JiOWExODdlMjZiMTVkYg==
+    MTMzNTlhMjUxZjY3Mzg2ZDAzNWQwMjgxNGUxYzUzNDQ5NTFlMDA3YQ==
 SHA512:
   metadata.gz: !binary |-
-    YmUxYmNjOWFhYTg3OGE4OWI2ZTZkMDM5OTcxMGI1OWY5MjcwZGY2M2ZmYzI3
-    N2RjMGFmMWQ3NmM2OTYyOGIyZTA2ZWJkMDc1MzlkYmU2MDc2YWUzOThhMDhm
-    YWUzNmY3MmU2MWM5YTBmY2UwYzFlOWIxNWRkMmJlZGVmMWJkZjQ=
+    MTRiMjNkYTdlYWQ3OTkxY2MwYTI0NjIyNzcyM2U0NjQ3MTRhNWRhM2VmNDNk
+    MDJkZDUzODE5NWQ5ZjE4YWM0NmVhMzNjMDllOTZiMGFlNjA5YWJjNmMwNWVl
+    Yjk2NTVjN2IzYjUxN2QzZTA3MjBmYmZjMzk2YWFlODViZmYyNjA=
   data.tar.gz: !binary |-
-    YzQzOTQ2ODJjMjJhYzI1MzJlZWQ5YjYxMTIyMjg5ZTgwZjk4N2M0N2I3OGQ4
-    ZGEwMTFhZDMyODU3NTEzY2I3OGI0Njk4YzI5ZTg1OTdjMTI1MmEzNDIyNWMw
-    NmZhOWUwZDg4NGJjZWZmZDNiYWIzNWI5YzkwMzQyOWZkZTFmMDI=
+    YzdmYjMzMzlhMDZjZGZjNTVjZGUwNWRlMTk4N2EzYTlkZWFkMjU5ZWIzY2I1
+    ZGVlYjM1N2QwYzg3YTFiMDFkYzE3YzM0YTY2OGJmYTgzYTI3Njg0YjEzYmY2
+    YTQ5ZDQ2YjY3MWRjNjIzZmIyYmNlMDNkMDQ5OWM0MmNjNzhmNjQ=

data/lib/redis/semaphore.rb

@@ -54,16 +54,17 @@ class Redis
       current_token = token_pair[1]
       @tokens.push(current_token)
       @redis.hset(grabbed_key, current_token, current_time.to_f)
-      
+      return_value = current_token
+
       if block_given?
         begin
-          yield current_token
+          return_value = yield current_token
         ensure
           signal(current_token)
         end
       end
 
-      current_token
+      return_value
     end
     alias_method :wait, :lock
 

data/spec/semaphore_spec.rb

@@ -95,6 +95,23 @@ describe "redis" do
 
       expect(semaphore.locked?).to eq(false)
     end
+
+    it "should return the value of the block if block-style locking is used" do
+      block_value = semaphore.lock(1) do
+        42
+      end
+      expect(block_value).to eq(42)
+    end
+
+    it "can return the passed in token to replicate old behaviour" do
+      lock_token = semaphore.lock(1)
+      semaphore.unlock()
+
+      block_value = semaphore.lock(1) do |token|
+        token
+      end
+      expect(block_value).to eq(lock_token)
+    end
   end
 
   describe "semaphore without staleness checking" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redis-semaphore
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.3.1
 platform: ruby
 authors:
 - David Verhasselt