Merge pull request rails#25030 from mmmpa/pull_request

ActionCable, sometimes add_channel is not called.

Author: matthewd

actioncable/lib/action_cable/subscription_adapter/subscriber_map.rb

@@ -32,7 +32,11 @@ def remove_subscriber(channel, subscriber)
       end
 
       def broadcast(channel, message)
-        list = @sync.synchronize { @subscribers[channel].dup }
+        list = @sync.synchronize do
+          return if !@subscribers.key?(channel)
+          @subscribers[channel].dup
+        end
+
         list.each do |subscriber|
           invoke_callback(subscriber, message)
         end

actioncable/test/subscription_adapter/subscriber_map_test.rb

@@ -0,0 +1,17 @@
+require 'test_helper'
+
+class SubscriberMapTest < ActionCable::TestCase
+  test "broadcast should not change subscribers" do
+    setup_subscription_map
+    origin = @subscription_map.instance_variable_get(:@subscribers).dup
+
+    @subscription_map.broadcast('not_exist_channel', '')
+
+    assert_equal origin, @subscription_map.instance_variable_get(:@subscribers)
+  end
+
+  private
+    def setup_subscription_map
+      @subscription_map = ActionCable::SubscriptionAdapter::SubscriberMap.new
+    end
+end