slack-ruby-client 0.14.4 → 0.14.5

data/spec/slack/real_time/concurrency/celluloid_spec.rb

@@ -1,16 +1,17 @@
+# frozen_string_literal: true
 require 'spec_helper'
 require_relative './it_behaves_like_a_realtime_socket'
 
 begin
   RSpec.describe Slack::RealTime::Concurrency::Celluloid::Socket do
-    it_behaves_like 'a realtime socket'
-
-    let(:url) { 'wss://echo.websocket.org/websocket/xyz' }
-    let(:logger) { ::Logger.new(STDOUT) }
+    subject { socket }
 
     let(:driver) { WebSocket::Driver::Client }
+    let(:logger) { ::Logger.new(STDOUT) }
     let(:ws) { double(driver) }
-    subject { socket }
+    let(:url) { 'wss://echo.websocket.org/websocket/xyz' }
+
+    it_behaves_like 'a realtime socket'
 
     ['', nil].each do |data|
       context "finishing run_loop with #{data ? 'empty' : 'nil'} read" do
@@ -86,9 +87,13 @@ begin
 
           context 'consumes data' do
             let(:tcp_socket) { double(::Celluloid::IO::SSLSocket, connect: true) }
+
             before do
-              allow_any_instance_of(described_class).to receive(:build_socket).and_return(tcp_socket)
+              allow_any_instance_of(described_class).to(
+                receive(:build_socket).and_return(tcp_socket)
+              )
             end
+
             it 'runs' do
               expect(ws).to receive(:emit)
               expect(ws).to receive(:start)

data/spec/slack/real_time/concurrency/eventmachine_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 require_relative './it_behaves_like_a_realtime_socket'
 
@@ -9,21 +10,26 @@ begin
       let(:logger) { ::Logger.new($stdout) }
       let(:socket) { described_class.new(url, ping: 42, logger: logger) }
       let(:ws) { double(Faye::WebSocket::Client) }
+
       describe '#connect!' do
         before do
           allow(ws).to receive(:on).with(:close)
           allow(ws).to receive(:on).with(:message)
         end
+
         it 'connects' do
           allow(Faye::WebSocket::Client).to receive(:new).and_return(ws)
           socket.connect!
           expect(socket.instance_variable_get('@driver')).to eq ws
         end
         it 'pings every 30s' do
-          expect(Faye::WebSocket::Client).to receive(:new).with(url, nil, ping: 42, logger: logger).and_return(ws)
+          expect(Faye::WebSocket::Client).to(
+            receive(:new).with(url, nil, ping: 42, logger: logger).and_return(ws)
+          )
           socket.connect!
         end
       end
+
       describe '#disconnect!' do
         it 'closes and nils the websocket' do
           socket.instance_variable_set('@driver', ws)
@@ -32,12 +38,14 @@ begin
           socket.disconnect!
         end
       end
+
       describe 'send_data' do
         before do
           allow(Faye::WebSocket::Client).to receive(:new).and_return(ws)
           allow(ws).to receive(:on)
           socket.connect!
         end
+
         it 'sends data' do
           expect(ws).to receive(:send).with('data')
           socket.send_data('data')

data/spec/slack/real_time/concurrency/it_behaves_like_a_realtime_socket.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.shared_examples_for 'a realtime socket' do
@@ -9,6 +10,7 @@ RSpec.shared_examples_for 'a realtime socket' do
       expect(socket.url).to eq url
     end
   end
+
   describe 'api' do
     %i[start_sync start_sync disconnect! connect! connected? send_data close].each do |m|
       it m do

data/spec/slack/real_time/event_handlers/bot_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do
@@ -36,7 +37,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
           }
         )
         client.send(:dispatch, event)
-      end.to_not change(client.bots, :count)
+      end.not_to change(client.bots, :count)
       bot = client.bots['B0751JU2H']
       expect(bot['name']).to eq 'hugbot'
     end

data/spec/slack/real_time/event_handlers/channel_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do
@@ -31,13 +32,13 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
       )
       client.send(:dispatch, event)
       channel = client.channels['C024BE91L']
-      expect(channel).to_not be nil
+      expect(channel).not_to be nil
       expect(channel.name).to eq 'fun'
       expect(channel.creator).to eq 'U04KB5WQR'
       expect(channel.created).to eq 1_360_782_804
     end
     it 'channel_deleted' do
-      expect(client.channels['C0HLE0BBL']).to_not be nil
+      expect(client.channels['C0HLE0BBL']).not_to be nil
       event = Slack::RealTime::Event.new(
         'type' => 'channel_deleted',
         'channel' => 'C0HLE0BBL'
@@ -57,12 +58,13 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
         )
         client.send(:dispatch, event)
         channel = client.channels['CDEADBEEF']
-        expect(channel).to_not be nil
+        expect(channel).not_to be nil
         expect(channel.name).to eq 'beef'
       end
       it 'updates channel' do
         expect(client.channels['CDEADBEEF']).to be nil
-        client.channels['CDEADBEEF'] = Slack::RealTime::Models::Channel.new('id' => 'CDEADBEEF', name: 'beef')
+        client.channels['CDEADBEEF'] =
+          Slack::RealTime::Models::Channel.new('id' => 'CDEADBEEF', name: 'beef')
         event = Slack::RealTime::Event.new(
           'type' => 'channel_joined',
           'channel' => {
@@ -73,10 +75,11 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
         )
         client.send(:dispatch, event)
         channel = client.channels['CDEADBEEF']
-        expect(channel).to_not be nil
+        expect(channel).not_to be nil
         expect(channel.updated).to be true
       end
     end
+
     it 'channel_left' do
       channel = client.channels['C0JHNAB5H']
       expect(channel.members).to include client.self.id
@@ -85,7 +88,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
         'channel' => 'C0JHNAB5H'
       )
       client.send(:dispatch, event)
-      expect(channel.members).to_not include client.self.id
+      expect(channel.members).not_to include client.self.id
     end
     it 'channel_rename' do
       channel = client.channels['C0HLE0BBL']

data/spec/slack/real_time/event_handlers/event_handlers_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do
@@ -10,6 +11,6 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
       'value' => 'updated.mp3'
     )
     expect(client.self).to receive(:prefs) { raise ArgumentError }
-    expect { client.send(:dispatch, event) }.to_not raise_error
+    expect { client.send(:dispatch, event) }.not_to raise_error
   end
 end

data/spec/slack/real_time/event_handlers/group_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do
@@ -18,7 +19,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
       )
       client.send(:dispatch, event)
       group = client.groups['CDEADBEEF']
-      expect(group).to_not be nil
+      expect(group).not_to be nil
       expect(group.name).to eq 'beef'
     end
     it 'group_left' do
@@ -29,7 +30,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
         'channel' => 'G0K7EV5A7'
       )
       client.send(:dispatch, event)
-      expect(group.members).to_not include client.self.id
+      expect(group.members).not_to include client.self.id
     end
     it 'group_archive' do
       group = client.groups['G0K7EV5A7']
@@ -67,7 +68,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
     end
     it 'group_open' do
       group = client.groups['G0K7EV5A7']
-      expect(group).to_not be_nil
+      expect(group).not_to be_nil
       event = Slack::RealTime::Event.new(
         'type' => 'group_open',
         'channel' => 'G0K7EV5A7'
@@ -77,7 +78,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
     end
     it 'group_close' do
       group = client.groups['G0K7EV5A7']
-      expect(group).to_not be_nil
+      expect(group).not_to be_nil
       group.is_open = true
       event = Slack::RealTime::Event.new(
         'type' => 'group_close',

data/spec/slack/real_time/event_handlers/im_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do
@@ -18,12 +19,12 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
       )
       client.send(:dispatch, event)
       im = client.ims['CDEADBEEF']
-      expect(im).to_not be nil
+      expect(im).not_to be nil
       expect(im.name).to eq 'beef'
     end
     it 'im_open' do
       im = client.ims['D07519J57']
-      expect(im).to_not be_nil
+      expect(im).not_to be_nil
       im.is_open = false
       event = Slack::RealTime::Event.new(
         'type' => 'im_open',
@@ -34,7 +35,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
     end
     it 'im_close' do
       im = client.ims['D07519J57']
-      expect(im).to_not be_nil
+      expect(im).not_to be_nil
       expect(im.is_open).to be true
       event = Slack::RealTime::Event.new(
         'type' => 'im_close',

data/spec/slack/real_time/event_handlers/team_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 [Slack::RealTime::Stores::Store, Slack::RealTime::Stores::Starter].each do |store_class|
-  RSpec.describe store_class, vcr: { cassette_name: store_class == Slack::RealTime::Stores::Store ? 'web/rtm_start' : 'web/rtm_connect' } do
+  cassette = store_class == Slack::RealTime::Stores::Store ? 'web/rtm_start' : 'web/rtm_connect'
+  RSpec.describe store_class, vcr: { cassette_name: cassette } do
     include_context 'connected client', store_class: store_class
 
     context 'team' do

data/spec/slack/real_time/event_handlers/user_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' } do

data/spec/slack/real_time/rtm_connect_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # require 'spec_helper'
 
 #

data/spec/slack/real_time/rtm_start_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # require 'spec_helper'
 
 #

data/spec/slack/real_time/store_spec.rb

@@ -1,8 +1,9 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::RealTime::Store do
   it 'can be initialized with an empty hash' do
-    store = Slack::RealTime::Store.new(Hashie::Mash.new)
+    store = described_class.new(Hashie::Mash.new)
     expect(store.self).to be nil
     expect(store.groups.count).to eq 0
     expect(store.team).to be nil

data/spec/slack/slack_spec.rb

@@ -1,40 +1,66 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 describe Slack do
   let(:slack) { File.expand_path(File.join(__FILE__, '../../../bin/slack')) }
+
   before do
     @token = ENV.delete('SLACK_API_TOKEN')
   end
+
   after do
     ENV['SLACK_API_TOKEN'] = @token if @token
   end
+
   describe '#help' do
     it 'displays help' do
       help = `"#{slack}" help`
       expect(help).to include 'slack - Slack client.'
     end
   end
+
   context 'globals' do
+    let(:command) do
+      "\"#{slack}\" --vcr-cassette-name=web/auth_test_success " \
+      '--slack-api-token=token -d auth test 2>&1'
+    end
+
     it 'enables request and response logging with -d' do
-      output = `"#{slack}" --vcr-cassette-name=web/auth_test_success --slack-api-token=token -d auth test 2>&1`
+      output = `#{command}`
       expect(output).to include 'POST https://slack.com/api/auth.test'
       expect(output).to include 'Status 200'
     end
     it 'requires --slack-api-token' do
       err = `"#{slack}" auth test 2>&1`
-      expect(err).to start_with 'error: parse error: Set Slack API token via --slack-api-token or SLACK_API_TOKEN.'
+      expect(err).to(
+        start_with(
+          'error: parse error: Set Slack API token via --slack-api-token or SLACK_API_TOKEN.'
+        )
+      )
     end
   end
+
   describe '#auth' do
     context 'bad auth' do
+      let(:command) do
+        "\"#{slack}\" --vcr-cassette-name=web/auth_test_error " \
+        '--slack-api-token=token auth test 2>&1'
+      end
+
       it 'fails with an exception' do
-        err = `"#{slack}" --vcr-cassette-name=web/auth_test_error --slack-api-token=token auth test 2>&1`
+        err = `#{command}`
         expect(err).to eq "error: not_authed\n"
       end
     end
+
     context 'good auth' do
+      let(:command) do
+        "\"#{slack}\" --vcr-cassette-name=web/auth_test_success " \
+        '--slack-api-token=token auth test 2>&1'
+      end
+
       it 'succeeds' do
-        json = Slack::Messages::Message.new(JSON.parse(`"#{slack}" --vcr-cassette-name=web/auth_test_success --slack-api-token=token auth test 2>&1`))
+        json = Slack::Messages::Message.new(JSON[`#{command}`])
         expect(json).to eq(
           'ok' => true,
           'url' => 'https://rubybot.slack.com/',
@@ -47,9 +73,15 @@ describe Slack do
       end
     end
   end
+
   describe '#users' do
+    let(:command) do
+      "\"#{slack}\" --vcr-cassette-name=web/users_list " \
+      '--slack-api-token=token users list --presence=true 2>&1'
+    end
+
     it 'list' do
-      json = Slack::Messages::Message.new(JSON.parse(`"#{slack}" --vcr-cassette-name=web/users_list --slack-api-token=token users list --presence=true 2>&1`))
+      json = Slack::Messages::Message.new(JSON[`#{command}`])
       expect(json.ok).to be true
       expect(json.members.size).to eq 9
       expect(json.members.first['presence']).to eq 'away'

data/spec/slack/version_spec.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 describe Slack do
   it 'has a version' do
-    expect(Slack::VERSION).to_not be nil
+    expect(Slack::VERSION).not_to be nil
   end
 end

data/spec/slack/web/api/endpoints/admin_apps_requests_spec.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AdminAppsRequests do
+  let(:client) { Slack::Web::Client.new }
+end

data/spec/slack/web/api/endpoints/admin_apps_spec.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AdminApps do
+  let(:client) { Slack::Web::Client.new }
+end

data/spec/slack/web/api/endpoints/admin_users_session_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'

data/spec/slack/web/api/endpoints/api_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'

data/spec/slack/web/api/endpoints/apps_permissions_resources_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'

data/spec/slack/web/api/endpoints/apps_permissions_scopes_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'

data/spec/slack/web/api/endpoints/apps_permissions_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'