Merge pull request #73 from rkadyb/master

add a 'closed' event

CHANGELOG.md

@@ -1,6 +1,7 @@
 ### 0.7.1 (Next)
 
 * Your contribution here.
+* [#73](https://github.com/dblock/slack-ruby-client/issues/73): Add a `closed` event - [@rkadyb](https://github.com/rkadyb).
 * [#69](https://github.com/dblock/slack-ruby-client/issues/69): Add attachments support for `Slack::Web::Api::Endpoints::Chat.chat_update` - [@nicka](https://github.com/nicka).
 
 ### 0.7.0 (3/6/2016)

README.md

@@ -194,6 +194,14 @@ client.on :message do |data|
   end
 end
 
+client.on :close do |_data|
+  puts "Client is about to disconnect"
+end
+
+client.on :closed do |_data|
+  puts "Client has disconnected successfully!"
+end
+
 client.start!
 ```
 

examples/hi_real_time/hi.rb

@@ -27,8 +27,12 @@
 end
 
 client.on :close do |_data|
-  puts 'Connection closed, exiting.'
+  puts 'Connection closing, exiting.'
   EM.stop
 end
 
+client.on :closed do |_data|
+  puts 'Connection has been disconnected.'
+end
+
 client.start!

lib/slack/real_time/client.rb

@@ -119,6 +119,7 @@ def run_loop(socket)
             logger.debug("#{self.class}##{__method__}") { event.class.name }
             callback(event, :close)
             close(event)
+            callback(event, :closed)
           end
         end
       end

spec/integration/integration_spec.rb

@@ -116,4 +116,24 @@ def stop_server
 
     start_server
   end
+
+  it 'gets close, followed by closed' do
+    client.on :hello do
+      expect(client.started?).to be true
+      client.stop!
+    end
+
+    client.on :close do |data|
+      logger.debug "client.on :close, data=#{data}"
+      expect(client.started?).to be true
+      @close_called = true
+    end
+
+    client.on :closed do |data|
+      logger.debug "client.on :closed, data=#{data}"
+      expect(@close_called).to be true
+    end
+
+    start_server
+  end
 end