slack-ruby-client 0.14.2 → 0.15.0

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

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Calls do
+  let(:client) { Slack::Web::Client.new }
+  context 'calls_add' do
+    it 'requires external_unique_id' do
+      expect { client.calls_add(join_url: 'https://example.com/calls/1234567890') }.to raise_error ArgumentError, /Required arguments :external_unique_id missing/
+    end
+    it 'requires join_url' do
+      expect { client.calls_add(external_unique_id: '025169F6-E37A-4E62-BB54-7F93A0FC4C1F') }.to raise_error ArgumentError, /Required arguments :join_url missing/
+    end
+  end
+  context 'calls_end' do
+    it 'requires id' do
+      expect { client.calls_end }.to raise_error ArgumentError, /Required arguments :id missing/
+    end
+  end
+  context 'calls_info' do
+    it 'requires id' do
+      expect { client.calls_info }.to raise_error ArgumentError, /Required arguments :id missing/
+    end
+  end
+  context 'calls_update' do
+    it 'requires id' do
+      expect { client.calls_update }.to raise_error ArgumentError, /Required arguments :id missing/
+    end
+  end
+end

data/spec/slack/web/api/endpoints/chat_scheduledMessages_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/conversations_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'
@@ -62,7 +63,7 @@ RSpec.describe Slack::Web::Api::Endpoints::Conversations do
   end
   context 'conversations_rename' do
     it 'requires channel' do
-      expect { client.conversations_rename(name: ' ') }.to raise_error ArgumentError, /Required arguments :channel missing/
+      expect { client.conversations_rename(name: '') }.to raise_error ArgumentError, /Required arguments :channel missing/
     end
     it 'requires name' do
       expect { client.conversations_rename(channel: 'C1234567890') }.to raise_error ArgumentError, /Required arguments :name missing/

data/spec/slack/web/api/endpoints/custom_specs/auth_spec.rb

@@ -1,17 +1,21 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Auth do
   let(:client) { Slack::Web::Client.new }
+
   context 'without auth', vcr: { cassette_name: 'web/auth_test_error' } do
     it 'fails with an exception' do
       expect { client.auth_test }.to raise_error Slack::Web::Api::Errors::SlackError, 'not_authed'
     end
   end
+
   context 'with auth', vcr: { cassette_name: 'web/auth_test_success' } do
     it 'succeeds' do
-      expect { client.auth_test }.to_not raise_error
+      expect { client.auth_test }.not_to raise_error
     end
   end
+
   context '429 error', vcr: { cassette_name: 'web/429_error' } do
     it 'fails with an specific exception' do
       begin

data/spec/slack/web/api/endpoints/custom_specs/channels_spec.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Channels do
   let(:client) { Slack::Web::Client.new }
+
   context 'channels' do
-    it 'info', vcr: { cassette_name: 'web/channels_info' } do
-      json = client.channels_info(channel: '#general')
-      expect(json.channel.name).to eq 'general'
+    it 'raises deprecation error', vcr: { cassette_name: 'web/channels_info' } do
+      expect { client.channels_info(channel: '#general') }
+        .to raise_error(Slack::Web::Api::Errors::MethodDeprecated, 'method_deprecated')
     end
   end
 end

data/spec/slack/web/api/endpoints/custom_specs/chat_spec.rb

@@ -1,10 +1,13 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Chat do
   let(:client) { Slack::Web::Client.new }
+
   context 'chat_postEphemeral' do
     let(:user) { OpenStruct.new(user: { id: '123' }) }
-    before(:each) do
+
+    before do
       allow(described_class).to receive(:users_id).and_return(user)
     end
 
@@ -17,38 +20,66 @@ RSpec.describe Slack::Web::Api::Endpoints::Chat do
         attachments: '[]',
         blocks: '[]'
       )
-      client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', attachments: [], blocks: [])
+      client.chat_postEphemeral(
+        channel: 'channel',
+        text: 'text',
+        user: '123',
+        attachments: [],
+        blocks: []
+      )
     end
     context 'text and user arguments' do
       it 'requires text or attachments' do
-        expect { client.chat_postEphemeral(channel: 'channel') }.to raise_error ArgumentError, /Required arguments :text, :attachments or :blocks missing/
+        expect { client.chat_postEphemeral(channel: 'channel') }.to(
+          raise_error(ArgumentError, /Required arguments :text, :attachments or :blocks missing/)
+        )
       end
       it 'requires user' do
-        expect { client.chat_postEphemeral(channel: 'channel', text: 'text') }.to raise_error ArgumentError, /Required arguments :user missing/
+        expect { client.chat_postEphemeral(channel: 'channel', text: 'text') }.to(
+          raise_error(ArgumentError, /Required arguments :user missing/)
+        )
       end
       it 'both text and user' do
-        expect(client).to receive(:post).with('chat.postEphemeral', hash_including(text: 'text', user: '123'))
-        expect { client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123') }.to_not raise_error
+        expect(client).to(
+          receive(:post).with('chat.postEphemeral', hash_including(text: 'text', user: '123'))
+        )
+        expect do
+          client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123')
+        end.not_to raise_error
       end
     end
+
     context 'attachments argument' do
       it 'optional attachments' do
-        expect(client).to receive(:post).with('chat.postEphemeral', hash_including(attachments: '[]'))
-        expect { client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', attachments: []) }.to_not raise_error
+        expect(client).to(
+          receive(:post).with('chat.postEphemeral', hash_including(attachments: '[]'))
+        )
+        expect do
+          client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', attachments: [])
+        end.not_to raise_error
       end
       it 'attachments without text' do
-        expect(client).to receive(:post).with('chat.postEphemeral', hash_including(attachments: '[]'))
-        expect { client.chat_postEphemeral(channel: 'channel', attachments: [], user: '123') }.to_not raise_error
+        expect(client).to(
+          receive(:post).with('chat.postEphemeral', hash_including(attachments: '[]'))
+        )
+        expect do
+          client.chat_postEphemeral(channel: 'channel', attachments: [], user: '123')
+        end.not_to raise_error
       end
     end
+
     context 'blocks argument' do
       it 'optional blocks' do
         expect(client).to receive(:post).with('chat.postEphemeral', hash_including(blocks: '[]'))
-        expect { client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', blocks: []) }.to_not raise_error
+        expect do
+          client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', blocks: [])
+        end.not_to raise_error
       end
       it 'blocks without text' do
         expect(client).to receive(:post).with('chat.postEphemeral', hash_including(blocks: '[]'))
-        expect { client.chat_postEphemeral(channel: 'channel', blocks: [], user: '123') }.to_not raise_error
+        expect do
+          client.chat_postEphemeral(channel: 'channel', blocks: [], user: '123')
+        end.not_to raise_error
       end
     end
   end
@@ -66,29 +97,37 @@ RSpec.describe Slack::Web::Api::Endpoints::Chat do
     end
     context 'text, attachment and blocks arguments' do
       it 'requires text, attachments or blocks' do
-        expect { client.chat_postMessage(channel: 'channel') }.to raise_error ArgumentError, /Required arguments :text, :attachments or :blocks missing/
+        expect { client.chat_postMessage(channel: 'channel') }.to(
+          raise_error(ArgumentError, /Required arguments :text, :attachments or :blocks missing/)
+        )
       end
       it 'only text' do
         expect(client).to receive(:post).with('chat.postMessage', hash_including(text: 'text'))
-        expect { client.chat_postMessage(channel: 'channel', text: 'text') }.to_not raise_error
+        expect { client.chat_postMessage(channel: 'channel', text: 'text') }.not_to raise_error
       end
       it 'only attachments' do
         expect(client).to receive(:post).with('chat.postMessage', hash_including(attachments: '[]'))
-        expect { client.chat_postMessage(channel: 'channel', attachments: []) }.to_not raise_error
+        expect { client.chat_postMessage(channel: 'channel', attachments: []) }.not_to raise_error
       end
       it 'only blocks' do
         expect(client).to receive(:post).with('chat.postMessage', hash_including(blocks: '[]'))
-        expect { client.chat_postMessage(channel: 'channel', blocks: []) }.to_not raise_error
+        expect { client.chat_postMessage(channel: 'channel', blocks: []) }.not_to raise_error
       end
       it 'all text, attachments and blocks' do
-        expect(client).to receive(:post).with('chat.postMessage', hash_including(text: 'text', attachments: '[]', blocks: '[]'))
-        expect { client.chat_postMessage(channel: 'channel', text: 'text', attachments: [], blocks: []) }.to_not raise_error
+        expect(client).to(
+          receive(:post)
+            .with('chat.postMessage', hash_including(text: 'text', attachments: '[]', blocks: '[]'))
+        )
+        expect do
+          client.chat_postMessage(channel: 'channel', text: 'text', attachments: [], blocks: [])
+        end.not_to raise_error
       end
     end
   end
 
   context 'chat_update' do
     let(:ts) { '1405894322.002768' }
+
     it 'automatically converts attachments and blocks into JSON' do
       expect(client).to receive(:post).with(
         'chat.update',
@@ -102,28 +141,44 @@ RSpec.describe Slack::Web::Api::Endpoints::Chat do
     end
     context 'ts arguments' do
       it 'requires ts' do
-        expect { client.chat_update(channel: 'channel', text: 'text') }.to raise_error ArgumentError, /Required arguments :ts missing>/
+        expect do
+          client.chat_update(channel: 'channel', text: 'text')
+        end.to raise_error(ArgumentError, /Required arguments :ts missing>/)
       end
     end
+
     context 'text, attachment and blocks arguments' do
       it 'requires text, attachments or blocks' do
-        expect { client.chat_update(channel: 'channel', ts: ts) }.to raise_error ArgumentError, /Required arguments :text, :attachments or :blocks missing/
+        expect { client.chat_update(channel: 'channel', ts: ts) }.to(
+          raise_error(ArgumentError, /Required arguments :text, :attachments or :blocks missing/)
+        )
       end
       it 'only text' do
         expect(client).to receive(:post).with('chat.update', hash_including(text: 'text'))
-        expect { client.chat_update(channel: 'channel', text: 'text', ts: ts) }.to_not raise_error
+        expect do
+          client.chat_update(channel: 'channel', text: 'text', ts: ts)
+        end.not_to raise_error
       end
       it 'only attachments' do
         expect(client).to receive(:post).with('chat.update', hash_including(attachments: '[]'))
-        expect { client.chat_update(channel: 'channel', ts: ts, attachments: []) }.to_not raise_error
+        expect do
+          client.chat_update(channel: 'channel', ts: ts, attachments: [])
+        end.not_to raise_error
       end
       it 'only blocks' do
         expect(client).to receive(:post).with('chat.update', hash_including(blocks: '[]'))
-        expect { client.chat_update(channel: 'channel', ts: ts, blocks: []) }.to_not raise_error
+        expect do
+          client.chat_update(channel: 'channel', ts: ts, blocks: [])
+        end.not_to raise_error
       end
       it 'all text, attachments and blocks' do
-        expect(client).to receive(:post).with('chat.update', hash_including(text: 'text', attachments: '[]', blocks: '[]'))
-        expect { client.chat_update(channel: 'channel', text: 'text', ts: ts, attachments: [], blocks: []) }.to_not raise_error
+        expect(client).to(
+          receive(:post)
+            .with('chat.update', hash_including(text: 'text', attachments: '[]', blocks: '[]'))
+        )
+        expect do
+          client.chat_update(channel: 'channel', text: 'text', ts: ts, attachments: [], blocks: [])
+        end.not_to raise_error
       end
     end
   end

data/spec/slack/web/api/endpoints/custom_specs/dialog_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Dialog do
   let(:client) { Slack::Web::Client.new }
+
   context 'dialog_open' do
     it 'automatically converts dialog into JSON' do
       expect(client).to receive(:post).with(
@@ -14,14 +16,20 @@ RSpec.describe Slack::Web::Api::Endpoints::Dialog do
 
     context 'arguments' do
       it 'requires dialog' do
-        expect { client.dialog_open(trigger_id: '123') }.to raise_error ArgumentError, /Required arguments :dialog missing/
+        expect { client.dialog_open(trigger_id: '123') }.to(
+          raise_error(ArgumentError, /Required arguments :dialog missing/)
+        )
       end
       it 'requires trigger_id' do
-        expect { client.dialog_open(dialog: []) }.to raise_error ArgumentError, /Required arguments :trigger_id missing/
+        expect { client.dialog_open(dialog: []) }.to(
+          raise_error(ArgumentError, /Required arguments :trigger_id missing/)
+        )
       end
       it 'likes both dialog and trigger_id' do
-        expect(client).to receive(:post).with('dialog.open', hash_including(trigger_id: '123', dialog: '[]'))
-        expect { client.dialog_open(dialog: [], trigger_id: '123') }.to_not raise_error
+        expect(client).to(
+          receive(:post).with('dialog.open', hash_including(trigger_id: '123', dialog: '[]'))
+        )
+        expect { client.dialog_open(dialog: [], trigger_id: '123') }.not_to raise_error
       end
     end
   end

data/spec/slack/web/api/endpoints/custom_specs/groups_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Groups do
   let(:client) { Slack::Web::Client.new }
+
   context 'groups' do
     it 'info', vcr: { cassette_name: 'web/groups_info' } do
       json = client.groups_info(channel: '#mpdm-dblock--rubybot--player1-1')

data/spec/slack/web/api/endpoints/custom_specs/users_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Users do
   let(:client) { Slack::Web::Client.new }
+
   context 'users' do
     it 'list', vcr: { cassette_name: 'web/users_list' } do
       json = client.users_list(presence: true)
@@ -9,7 +11,8 @@ RSpec.describe Slack::Web::Api::Endpoints::Users do
       expect(json.members.size).to eq 9
       expect(json.members.first.presence).to eq 'away'
     end
-    it 'list', vcr: { cassette_name: 'web/paginated_users_list' } do
+
+    it 'paginated list', vcr: { cassette_name: 'web/paginated_users_list' } do
       members = []
       client.users_list(presence: true, limit: 5) do |json|
         expect(json.ok).to be true
@@ -17,10 +20,12 @@ RSpec.describe Slack::Web::Api::Endpoints::Users do
       end
       expect(members.size).to eq 23
     end
+
     it 'info', vcr: { cassette_name: 'web/users_info' } do
       json = client.users_info(user: '@aws')
       expect(json.user.name).to eq 'aws'
     end
+
     if defined?(Picky)
       it 'search', vcr: { cassette_name: 'web/users_info' } do
         json = client.users_search(user: 'aws')

data/spec/slack/web/api/endpoints/custom_specs/views_spec.rb

@@ -0,0 +1,95 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Views do
+  let(:client) { Slack::Web::Client.new }
+  let(:trigger_id) { '12345.98765.abcd2358fdea' }
+
+  describe 'views_open' do
+    context 'with a hash for view' do
+      let(:view_str) { '{"celery":"man"}' }
+
+      it 'automatically converts view into JSON' do
+        expect(client).to receive(:post).with('views.open', trigger_id: trigger_id, view: view_str)
+        client.views_open(trigger_id: trigger_id, view: { celery: 'man' })
+      end
+    end
+
+    context 'with a string for view' do
+      let(:view_str) { 'celery man' }
+
+      it 'leaves view as is' do
+        expect(client).to receive(:post).with('views.open', trigger_id: trigger_id, view: view_str)
+        client.views_open(trigger_id: trigger_id, view: 'celery man')
+      end
+    end
+  end
+
+  describe 'views_publish' do
+    let(:user_id) { 'U1234' }
+
+    context 'with a hash for view' do
+      let(:view_str) { '{"celery":"man"}' }
+
+      it 'automatically converts view into JSON' do
+        expect(client).to receive(:post).with('views.publish',
+                                              user_id: user_id,
+                                              trigger_id: trigger_id,
+                                              view: view_str)
+        client.views_publish(user_id: user_id, trigger_id: trigger_id, view: { celery: 'man' })
+      end
+    end
+
+    context 'with a string for view' do
+      let(:view_str) { 'celery man' }
+
+      it 'leaves view as is' do
+        expect(client).to receive(:post).with('views.publish',
+                                              user_id: user_id,
+                                              trigger_id: trigger_id,
+                                              view: view_str)
+        client.views_publish(user_id: user_id, trigger_id: trigger_id, view: 'celery man')
+      end
+    end
+  end
+
+  describe 'views_push' do
+    context 'with a hash for view' do
+      let(:view_str) { '{"celery":"man"}' }
+
+      it 'automatically converts view into JSON' do
+        expect(client).to receive(:post).with('views.push', trigger_id: trigger_id, view: view_str)
+        client.views_push(trigger_id: trigger_id, view: { celery: 'man' })
+      end
+    end
+
+    context 'with a string for view' do
+      let(:view_str) { 'celery man' }
+
+      it 'leaves view as is' do
+        expect(client).to receive(:post).with('views.push', trigger_id: trigger_id, view: view_str)
+        client.views_push(trigger_id: trigger_id, view: 'celery man')
+      end
+    end
+  end
+
+  describe 'views_update' do
+    context 'with a hash for view' do
+      let(:view_str) { '{"celery":"man"}' }
+
+      it 'automatically converts view into JSON' do
+        expect(client).to receive(:post).with('views.update', view: view_str)
+        client.views_update(view: { celery: 'man' })
+      end
+    end
+
+    context 'with a string for view' do
+      let(:view_str) { 'celery man' }
+
+      it 'leaves view as is' do
+        expect(client).to receive(:post).with('views.update', view: view_str)
+        client.views_update(view: 'celery man')
+      end
+    end
+  end
+end