slack-ruby-client 0.15.0 → 0.15.1

data/spec/slack/real_time/client_spec.rb

@@ -166,6 +166,23 @@ RSpec.describe Slack::RealTime::Client do # rubocop:disable Metrics/BlockLength
           expect(client.store.team.name).to eq 'New Team Name Inc.'
         end
       end
+
+      describe '#run_handlers' do
+        describe 'empty events' do
+          before do
+            @e = client.store.class.events
+            client.store.class.events = nil
+          end
+
+          after do
+            client.store.class.events = @e
+          end
+
+          it 'returns false when event is nil' do
+            expect(client.send(:run_handlers, 'example', {})).to be nil
+          end
+        end
+      end
     end
 
     describe '#start_async' do

data/spec/slack/web/api/mixins/channels_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe Slack::Web::Api::Mixins::Channels do
   end
 
   before do
-    allow(channels).to receive(:channels_list).and_return(
+    allow(channels).to receive(:channels_list).and_yield(
       Slack::Messages::Message.new(
         'channels' => [{
           'id' => 'CDEADBEEF',

data/spec/slack/web/api/mixins/conversations_list_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Client do
+  context 'conversations_setTopic' do
+    it 'does not invoke conversations_list', vcr: { cassette_name: 'web/conversations_setTopic' } do
+      rc = subject.conversations_setTopic({ channel: 'C018Y6VH39D', topic: 'new topic' })
+      expect(rc.channel.topic.value).to eq 'new topic'
+    end
+
+    it 'resolves IDs via conversations_list', vcr: { cassette_name: 'web/conversations_setTopic_one_page' } do
+      rc = subject.conversations_setTopic({ channel: '#1', topic: 'new topic' })
+      expect(rc.channel.topic.value).to eq 'new topic'
+    end
+
+    it 'paginates to resolve IDs', vcr: { cassette_name: 'web/conversations_setTopic_paginated' } do
+      rc = subject.conversations_setTopic({ channel: '#topic', topic: 'new topic' })
+      expect(rc.channel.topic.value).to eq 'new topic'
+    end
+  end
+end

data/spec/slack/web/api/mixins/conversations_spec.rb

@@ -9,18 +9,20 @@ RSpec.describe Slack::Web::Api::Mixins::Conversations do
   let(:klass) do
     Class.new do
       include Slack::Web::Api::Mixins::Conversations
-
-      def conversations_list
-        Slack::Messages::Message.new(
-          'channels' => [{
-            'id' => 'CDEADBEEF',
-            'name' => 'general'
-          }]
-        )
-      end
     end
   end
 
+  before do
+    allow(conversations).to receive(:conversations_list).and_yield(
+      Slack::Messages::Message.new(
+        'channels' => [{
+          'id' => 'CDEADBEEF',
+          'name' => 'general'
+        }]
+      )
+    )
+  end
+
   describe '#conversations_id' do
     it 'leaves channels specified by ID alone' do
       expect(conversations.conversations_id(channel: 'C123456')).to(

data/spec/slack/web/api/mixins/groups_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe Slack::Web::Api::Mixins::Groups do
   end
 
   before do
-    allow(groups).to receive(:groups_list).and_return(
+    allow(groups).to receive(:groups_list).and_yield(
       Slack::Messages::Message.new(
         'groups' => [{
           'id' => 'CDEADBEEF',

data/spec/slack/web/api/mixins/users_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe Slack::Web::Api::Mixins::Users do
   end
 
   before do
-    allow(users).to receive(:users_list).and_return(
+    allow(users).to receive(:users_list).and_yield(
       Slack::Messages::Message.new(
         'members' => [{
           'id' => 'UDEADBEEF',

data/spec/slack/web/client_spec.rb

@@ -150,6 +150,50 @@ RSpec.describe Slack::Web::Client do
       end
     end
 
+    context 'adapter option' do
+      let(:adapter) { Faraday.default_adapter }
+      let(:adapter_class) { Faraday::Adapter::NetHttp }
+
+      around do |ex|
+        previous_adapter = Faraday.default_adapter
+        # ensure default adapter is set for this spec
+        Faraday.default_adapter = :net_http
+        ex.run
+        Faraday.default_adapter = previous_adapter
+      end
+
+      context 'default adapter' do
+        describe '#initialize' do
+          it 'sets adapter' do
+            expect(client.adapter).to eq adapter
+          end
+          it 'creates a connection with an adapter' do
+            expect(client.send(:connection).adapter).to eq adapter_class
+          end
+        end
+      end
+
+      context 'non default adapter' do
+        let(:adapter) { :typhoeus }
+        let(:adapter_class) { Faraday::Adapter::Typhoeus }
+
+        before do
+          described_class.configure do |config|
+            config.adapter = adapter
+          end
+        end
+
+        describe '#initialize' do
+          it 'sets adapter' do
+            expect(client.adapter).to eq adapter
+          end
+          it 'creates a connection with an adapter' do
+            expect(client.send(:connection).adapter).to eq adapter_class
+          end
+        end
+      end
+    end
+
     context 'timeout options' do
       before do
         described_class.configure do |config|

data/spec/support/vcr.rb

@@ -7,4 +7,8 @@ VCR.configure do |config|
   config.hook_into :webmock
   # config.default_cassette_options = { record: :new_episodes }
   config.configure_rspec_metadata!
+  config.before_record do |i|
+    i.request.body.gsub!(ENV['SLACK_API_TOKEN'], 'token') if ENV.key?('SLACK_API_TOKEN')
+    i.response.body.force_encoding('UTF-8')
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack-ruby-client
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.15.1
 platform: ruby
 authors:
 - Daniel Doubrovkine
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-26 00:00:00.000000000 Z
+date: 2020-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -479,6 +479,9 @@ files:
 - spec/fixtures/slack/web/auth_test_error.yml
 - spec/fixtures/slack/web/auth_test_success.yml
 - spec/fixtures/slack/web/channels_info.yml
+- spec/fixtures/slack/web/conversations_setTopic.yml
+- spec/fixtures/slack/web/conversations_setTopic_one_page.yml
+- spec/fixtures/slack/web/conversations_setTopic_paginated.yml
 - spec/fixtures/slack/web/groups_info.yml
 - spec/fixtures/slack/web/paginated_users_list.yml
 - spec/fixtures/slack/web/rtm_connect.yml
@@ -574,6 +577,7 @@ files:
 - spec/slack/web/api/errors/service_unavailable_spec.rb
 - spec/slack/web/api/errors/slack_error_spec.rb
 - spec/slack/web/api/mixins/channels_spec.rb
+- spec/slack/web/api/mixins/conversations_list_spec.rb
 - spec/slack/web/api/mixins/conversations_spec.rb
 - spec/slack/web/api/mixins/groups_spec.rb
 - spec/slack/web/api/mixins/users_spec.rb
@@ -606,7 +610,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.3
 signing_key:
 specification_version: 4
 summary: Slack Web and RealTime API client.
@@ -616,6 +620,9 @@ test_files:
 - spec/fixtures/slack/web/auth_test_error.yml
 - spec/fixtures/slack/web/auth_test_success.yml
 - spec/fixtures/slack/web/channels_info.yml
+- spec/fixtures/slack/web/conversations_setTopic.yml
+- spec/fixtures/slack/web/conversations_setTopic_one_page.yml
+- spec/fixtures/slack/web/conversations_setTopic_paginated.yml
 - spec/fixtures/slack/web/groups_info.yml
 - spec/fixtures/slack/web/paginated_users_list.yml
 - spec/fixtures/slack/web/rtm_connect.yml
@@ -711,6 +718,7 @@ test_files:
 - spec/slack/web/api/errors/service_unavailable_spec.rb
 - spec/slack/web/api/errors/slack_error_spec.rb
 - spec/slack/web/api/mixins/channels_spec.rb
+- spec/slack/web/api/mixins/conversations_list_spec.rb
 - spec/slack/web/api/mixins/conversations_spec.rb
 - spec/slack/web/api/mixins/groups_spec.rb
 - spec/slack/web/api/mixins/users_spec.rb