From 046e32656efa989a112e9870d8aa25a49f944204 Mon Sep 17 00:00:00 2001 From: Jay Hayes Date: Wed, 24 Feb 2016 12:35:36 -0600 Subject: Convert stream broadcasting to a string ActionCable does some things behind the scenes that expects these "broadcasting"s or "channel"s to be strings. However it's not immediately obvious that the value must be a string. So adding this conversion ensures things work as expected. --- actioncable/test/channel/stream_test.rb | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) (limited to 'actioncable/test/channel') diff --git a/actioncable/test/channel/stream_test.rb b/actioncable/test/channel/stream_test.rb index ecbee52a4f..526ea92e4f 100644 --- a/actioncable/test/channel/stream_test.rb +++ b/actioncable/test/channel/stream_test.rb @@ -16,6 +16,12 @@ class ActionCable::Channel::StreamTest < ActionCable::TestCase end end + class SymbolChannel < ActionCable::Channel::Base + def subscribed + stream_from :channel + end + end + test "streaming start and stop" do run_in_eventmachine do connection = TestConnection.new @@ -27,6 +33,17 @@ class ActionCable::Channel::StreamTest < ActionCable::TestCase end end + test "stream from non-string channel" do + run_in_eventmachine do + connection = TestConnection.new + connection.expects(:pubsub).returns mock().tap { |m| m.expects(:subscribe).with("channel", kind_of(Proc), kind_of(Proc)).returns stub_everything(:pubsub) } + channel = SymbolChannel.new connection, "" + + connection.expects(:pubsub).returns mock().tap { |m| m.expects(:unsubscribe) } + channel.unsubscribe_from_channel + end + end + test "stream_for" do run_in_eventmachine do connection = TestConnection.new -- cgit v1.2.3