Use service class instance logger.

Author: dblock

lib/slack-ruby-bot-server-rtm/server.rb

@@ -24,11 +24,11 @@ def self.start_server!(team, server, wait = 1)
         SlackRubyBotServer::Config.service_class.instance.send(:run_callbacks, :error, team, e)
         case e.message
         when 'account_inactive', 'invalid_auth' then
-          SlackRubyBotServer::Config.logger.error "#{team.name}: #{e.message}, team will be deactivated."
-          SlackRubyBotServer::RealTime::Service.instance.deactivate! team
-          elsef
+          SlackRubyBotServer::Config.service_class.instance.logger.error "#{team.name}: #{e.message}, team will be deactivated."
+          SlackRubyBotServer::Config.service_class.instance.deactivate! team
+        else
           wait = e.retry_after if e.is_a?(Slack::Web::Api::Errors::TooManyRequestsError)
-          SlackRubyBotServer::Config.logger.error "#{team.name}: #{e.message}, restarting in #{wait} second(s)."
+          SlackRubyBotServer::Config.service_class.instance.logger.error "#{team.name}: #{e.message}, restarting in #{wait} second(s)."
           sleep(wait)
           start_server! team, server, [wait * 2, 60].min
         end
@@ -37,8 +37,8 @@ def self.start_server!(team, server, wait = 1)
       def restart!(_wait = 1)
         # when an integration is disabled, a live socket is closed, which causes the default behavior of the client to restart
         # it would keep retrying without checking for account_inactive or such, we want to restart via service which will disable an inactive team
-        logger.info "#{team.name}: socket closed, restarting ..."
-        SlackRubyBotServer::RealTime::Service.instance.restart! team
+        SlackRubyBotServer::Config.service_class.instance.logger.info "#{team.name}: socket closed, restarting ..."
+        SlackRubyBotServer::Config.service_class.instance.restart! team
         open!
       end
 

spec/slack-ruby-bot-server-rtm/server_spec.rb

@@ -24,4 +24,10 @@
       expect(subject.send(:client).names).to include 'foo'
     end
   end
+  context 'start_server!' do
+    pending 'starts server'
+    pending 'handles account_inactive'
+    pending 'handles invalid_auth'
+    pending 'handles generic error'
+  end
 end