Merge pull request #9 from centosadmin/nested-handlers

Fix recursive locking in nested handlers

lib/tdlib/update_manager.rb

@@ -37,8 +37,7 @@ def stopped?
 
   def handle_update
     update = TD::Api.client_receive(@td_client, TIMEOUT)
-    @mutex.synchronize do
-      @handlers.each { |h| h.call(update) } unless update.nil?
-    end
+    @mutex.synchronize { handlers = @handlers.dup }
+    handlers.each { |h| h.call(update) } unless update.nil?
   end
 end

spec/integration/tdlib_spec.rb

@@ -74,4 +74,19 @@
       expect(@result).to include('@type', 'entities')
     end
   end
+
+  describe 'nested handlers' do
+    before { Thread.abort_on_exception = true }
+
+    subject do
+      client.on_ready do
+        client.broadcast(payload) do
+          client.broadcast_and_receive(payload)
+        end
+        sleep 1
+      end
+    end
+
+    it { expect { subject }.not_to raise_error(Exception) }
+  end
 end