slack-ruby-client 0.7.9 → 0.16.0

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,8 +10,9 @@ RSpec.shared_examples_for 'a realtime socket' do
       expect(socket.url).to eq url
     end
   end
+
   describe 'api' do
-    [:start_sync, :start_sync, :disconnect!, :connect!, :connected?, :send_data, :close].each do |m|
+    %i[start_sync start_sync disconnect! connect! connected? send_data close].each do |m|
       it m do
         expect(socket).to respond_to(m)
       end

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
@@ -17,7 +18,8 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
             'icons' => {
               'image_48' => 'https:\/\/slack.com\/path\/to\/hugbot_48.png'
             }
-          })
+          }
+        )
         client.send(:dispatch, event)
       end.to change(client.bots, :count).by(1)
       bot = client.bots['B024BE7LH']
@@ -32,9 +34,10 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
           'bot' => {
             'id' => 'B0751JU2H',
             'name' => 'hugbot'
-          })
+          }
+        )
         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
@@ -9,7 +10,7 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
       'name' => 'push_sound',
       'value' => 'updated.mp3'
     )
-    expect(client.self).to receive(:prefs) { fail ArgumentError }
-    expect { client.send(:dispatch, event) }.to_not raise_error
+    expect(client.self).to receive(:prefs) { raise ArgumentError }
+    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,16 +1,20 @@
+# 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: 'web/rtm_start' } 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
       it 'sets team data on rtm.start' do
         expect(client.team.name).to eq 'dblock'
         expect(client.team.domain).to eq 'dblockdotorg'
-        expect(client.team.email_domain).to eq 'dblock.org'
-        expect(client.team.prefs.invites_only_admins).to be true
-        expect(client.team.plan).to eq ''
+        if store_class == Slack::RealTime::Stores::Store
+          expect(client.team.email_domain).to eq 'dblock.org'
+          expect(client.team.prefs.invites_only_admins).to be true
+          expect(client.team.plan).to eq ''
+        end
       end
       it 'team_domain_change' do
         event = Slack::RealTime::Event.new(

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
@@ -14,7 +15,8 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
         'type' => 'user_change',
         'user' => {
           'id' => 'U07KECJ77', 'name' => 'renamed'
-        })
+        }
+      )
       client.send(:dispatch, event)
       expect(client.users['U07KECJ77'].name).to eq 'renamed'
     end
@@ -24,7 +26,8 @@ RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_start' }
           'type' => 'team_join',
           'user' => {
             'id' => 'DEADBEEF', 'name' => 'added'
-          })
+          }
+        )
         client.send(:dispatch, event)
       end.to change(client.users, :count).by(1)
       expect(client.users['DEADBEEF'].name).to eq 'added'

data/spec/slack/real_time/rtm_connect_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+# require 'spec_helper'
+
+#
+# to re-record a rtm_connect fixture run with
+# SLACK_API_TOKEN=... CONCURRENCY=faye-websocket rspec spec/slack/real_time/rtm_connect_spec.rb
+# edit rtm_connect.yml and remove the token, fix wss:// path (run specs, fix failures)
+#
+
+# RSpec.describe Slack::RealTime::Client, vcr: { cassette_name: 'web/rtm_connect' } do
+#   it 'connects' do
+#     Slack::Web::Client.new.rtm_connect(mpim_aware: true)
+#   end
+# end

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`
-      expect(output).to include 'post https://slack.com/api/auth.test'
-      expect(output).to include 'Status: 200'
+      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_analytics_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AdminAnalytics do
+  let(:client) { Slack::Web::Client.new }
+  context 'admin.analytics_getFile' do
+    it 'requires type' do
+      expect { client.admin_analytics_getFile }.to raise_error ArgumentError, /Required arguments :type missing/
+    end
+  end
+end

data/spec/slack/web/api/endpoints/admin_apps_approved_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::AdminAppsApproved do
+  let(:client) { Slack::Web::Client.new }
+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_restricted_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::AdminAppsRestricted do
+  let(:client) { Slack::Web::Client.new }
+end

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

@@ -0,0 +1,13 @@
+# 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 }
+  context 'admin.apps_clearResolution' do
+    it 'requires app_id' do
+      expect { client.admin_apps_clearResolution }.to raise_error ArgumentError, /Required arguments :app_id missing/
+    end
+  end
+end

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

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AdminBarriers do
+  let(:client) { Slack::Web::Client.new }
+  context 'admin.barriers_create' do
+    it 'requires barriered_from_usergroup_ids' do
+      expect { client.admin_barriers_create(primary_usergroup_id: %q[], restricted_subjects: %q[]) }.to raise_error ArgumentError, /Required arguments :barriered_from_usergroup_ids missing/
+    end
+    it 'requires primary_usergroup_id' do
+      expect { client.admin_barriers_create(barriered_from_usergroup_ids: %q[], restricted_subjects: %q[]) }.to raise_error ArgumentError, /Required arguments :primary_usergroup_id missing/
+    end
+    it 'requires restricted_subjects' do
+      expect { client.admin_barriers_create(barriered_from_usergroup_ids: %q[], primary_usergroup_id: %q[]) }.to raise_error ArgumentError, /Required arguments :restricted_subjects missing/
+    end
+  end
+  context 'admin.barriers_delete' do
+    it 'requires barrier_id' do
+      expect { client.admin_barriers_delete }.to raise_error ArgumentError, /Required arguments :barrier_id missing/
+    end
+  end
+  context 'admin.barriers_update' do
+    it 'requires barrier_id' do
+      expect { client.admin_barriers_update(barriered_from_usergroup_ids: %q[], primary_usergroup_id: %q[], restricted_subjects: %q[]) }.to raise_error ArgumentError, /Required arguments :barrier_id missing/
+    end
+    it 'requires barriered_from_usergroup_ids' do
+      expect { client.admin_barriers_update(barrier_id: %q[], primary_usergroup_id: %q[], restricted_subjects: %q[]) }.to raise_error ArgumentError, /Required arguments :barriered_from_usergroup_ids missing/
+    end
+    it 'requires primary_usergroup_id' do
+      expect { client.admin_barriers_update(barrier_id: %q[], barriered_from_usergroup_ids: %q[], restricted_subjects: %q[]) }.to raise_error ArgumentError, /Required arguments :primary_usergroup_id missing/
+    end
+    it 'requires restricted_subjects' do
+      expect { client.admin_barriers_update(barrier_id: %q[], barriered_from_usergroup_ids: %q[], primary_usergroup_id: %q[]) }.to raise_error ArgumentError, /Required arguments :restricted_subjects missing/
+    end
+  end
+end