From 23e3e2bc2bfcc393b9164975e97d5ddc58f8bf5c Mon Sep 17 00:00:00 2001 From: Duncan Grazier Date: Tue, 20 Dec 2016 09:32:45 -0500 Subject: ActionCable should not raise when a connection is already open ActionCable was throwing a "Existing connection must be closed before opening" exception which was being picked up as a production issue in our error monitoring software. Since this happens pretty often on any device that allows the browser to sleep (mobile) this error was getting triggered often. This change removes the exception, but keeps logging the occurrence. We now return `false` to let the caller now that `open` failed. --- actioncable/test/javascript/src/unit/consumer_test.coffee | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'actioncable/test/javascript') diff --git a/actioncable/test/javascript/src/unit/consumer_test.coffee b/actioncable/test/javascript/src/unit/consumer_test.coffee index cf8a592255..41445274eb 100644 --- a/actioncable/test/javascript/src/unit/consumer_test.coffee +++ b/actioncable/test/javascript/src/unit/consumer_test.coffee @@ -2,8 +2,11 @@ {consumerTest} = ActionCable.TestHelpers module "ActionCable.Consumer", -> - consumerTest "#connect", connect: false, ({consumer, server, done}) -> - server.on("connection", done) + consumerTest "#connect", connect: false, ({consumer, server, assert, done}) -> + server.on "connection", -> + assert.equal consumer.connect(), false + done() + consumer.connect() consumerTest "#disconnect", ({consumer, client, done}) -> -- cgit v1.2.3