broadcaster 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0ae6565dba10ebb367ec4454148293dd205c48cb
-  data.tar.gz: fd6102f4293dc93488a2e0a82828b01fe294e821
+  metadata.gz: 9db2fe4bfde7c7696999349310f62e331fcf4bda
+  data.tar.gz: 36341eaf5595452ba33947f7144cefd6004372f7
 SHA512:
-  metadata.gz: e6766995f6d199435b7dcb63aa0cf90d1633d89b61e7d72e3eb2d7e4f8f13ac30163800ae51e0e6db75a57cb0f0fd0204e44f82d9b06bfccb60e5c3e1aaf3ff0
-  data.tar.gz: 8e875c87c94387862f724c57f41bcf7ade8156f117e3bade30680fed816ab7c8c29c44b8ab0d1eb910d0f1bd16551eac20e8deae32d7e477cf01effd8e9aa889
+  metadata.gz: 81c0b0bef9dd55b41082930634067078f166c7864d372a2bcf9b89baa9ec610232e9f2bc8cff4410e2260b7f62d1db6ecadb74a947834e67c8862dd6ae015ef8
+  data.tar.gz: c0abdbd7b65a60f070358959eec6772daf27f8fa29beeb58f7ea4208324108d8c1fcb05ca40b5578bd40f9152871ff040fd29087a2976862cc0924d2e549dbd9

data/broadcaster.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = 'broadcaster'
-  spec.version       = '0.1.0'
+  spec.version       = '0.1.1'
   spec.authors       = ['Gabriel Naiman']
   spec.email         = ['gabynaiman@gmail.com']
   spec.summary       = 'Broadcasting based on Redis PubSub'

data/lib/broadcaster.rb

@@ -72,13 +72,11 @@ class Broadcaster
         channel = notification[2]
         message = Marshal.load notification[3]
         logger.debug(self.class) { "Broadcasting (#{subscriptions[channel].count}) | #{channel} | #{message}" }
-        mutex.synchronize do
-          subscriptions[channel].each do |subscription_id, block|
-            begin
-              block.call message
-            rescue => ex
-              logger.error(self.class) { "Failed | #{scoped(channel)} | #{subscription_id} | #{message}\n#{ex.class}: #{ex.message}\n#{ex.backtrace.join("\n")}" }
-            end
+        subscriptions[channel].each do |subscription_id, block|
+          begin
+            block.call message
+          rescue => ex
+            logger.error(self.class) { "Failed | #{channel} | #{subscription_id} | #{message}\n#{ex.class}: #{ex.message}\n#{ex.backtrace.join("\n")}" }
           end
         end
       end

data/spec/broadcast_spec.rb

@@ -2,6 +2,18 @@ require 'minitest_helper'
 
 describe Broadcaster do
 
+  class Receiver
+    attr_reader :messages
+
+    def initialize
+      @messages = []
+    end
+
+    def call(message)
+      messages << message
+    end
+  end
+
   def wait_for(&block)
     Timeout.timeout(3) do
       while !block.call
@@ -142,4 +154,20 @@ describe Broadcaster do
     error.message.must_match 'invalid_host'
   end
 
+  it 'Subscribe object' do
+    broadcaster = Broadcaster.new
+
+    receiver = Receiver.new
+
+    broadcaster.subscribe 'channel_1', receiver
+
+    broadcaster.publish 'channel_1', 'message 1'
+    broadcaster.publish 'channel_1', 'message 2'
+    broadcaster.publish 'channel_1', 'message 3'
+
+    wait_for { receiver.messages.count == 3 }
+
+    receiver.messages.must_equal ['message 1', 'message 2', 'message 3']
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: broadcaster
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Gabriel Naiman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-03 00:00:00.000000000 Z
+date: 2017-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redic