slack-ruby-client 0.13.1 → 2.0.0

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

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AdminUsers do
+  let(:client) { Slack::Web::Client.new }
+  context 'admin.users_assign' do
+    it 'requires team_id' do
+      expect { client.admin_users_assign(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_assign(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+  context 'admin.users_invite' do
+    it 'requires channel_ids' do
+      expect { client.admin_users_invite(email: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
+    end
+    it 'requires email' do
+      expect { client.admin_users_invite(channel_ids: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :email missing/
+    end
+    it 'requires team_id' do
+      expect { client.admin_users_invite(channel_ids: %q[], email: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+  end
+  context 'admin.users_remove' do
+    it 'requires team_id' do
+      expect { client.admin_users_remove(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_remove(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+  context 'admin.users_setAdmin' do
+    it 'requires team_id' do
+      expect { client.admin_users_setAdmin(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_setAdmin(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+  context 'admin.users_setExpiration' do
+    it 'requires expiration_ts' do
+      expect { client.admin_users_setExpiration(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :expiration_ts missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_setExpiration(expiration_ts: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+  context 'admin.users_setOwner' do
+    it 'requires team_id' do
+      expect { client.admin_users_setOwner(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_setOwner(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+  context 'admin.users_setRegular' do
+    it 'requires team_id' do
+      expect { client.admin_users_setRegular(user_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
+    end
+    it 'requires user_id' do
+      expect { client.admin_users_setRegular(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :user_id missing/
+    end
+  end
+end

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

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_connections_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::AppsConnections do
+  let(:client) { Slack::Web::Client.new }
+end

data/spec/slack/web/api/endpoints/apps_event_authorizations_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::AppsEventAuthorizations do
+  let(:client) { Slack::Web::Client.new }
+  context 'apps.event.authorizations_list' do
+    it 'requires event_context' do
+      expect { client.apps_event_authorizations_list }.to raise_error ArgumentError, /Required arguments :event_context missing/
+    end
+  end
+end

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::AppsManifest do
+  let(:client) { Slack::Web::Client.new }
+  context 'apps.manifest_create' do
+    it 'requires manifest' do
+      expect { client.apps_manifest_create }.to raise_error ArgumentError, /Required arguments :manifest missing/
+    end
+  end
+  context 'apps.manifest_delete' do
+    it 'requires app_id' do
+      expect { client.apps_manifest_delete }.to raise_error ArgumentError, /Required arguments :app_id missing/
+    end
+  end
+  context 'apps.manifest_export' do
+    it 'requires app_id' do
+      expect { client.apps_manifest_export }.to raise_error ArgumentError, /Required arguments :app_id missing/
+    end
+  end
+  context 'apps.manifest_update' do
+    it 'requires app_id' do
+      expect { client.apps_manifest_update(manifest: %q[]) }.to raise_error ArgumentError, /Required arguments :app_id missing/
+    end
+    it 'requires manifest' do
+      expect { client.apps_manifest_update(app_id: %q[]) }.to raise_error ArgumentError, /Required arguments :manifest missing/
+    end
+  end
+  context 'apps.manifest_validate' do
+    it 'requires manifest' do
+      expect { client.apps_manifest_validate }.to raise_error ArgumentError, /Required arguments :manifest missing/
+    end
+  end
+end

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

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Apps do
+  let(:client) { Slack::Web::Client.new }
+  context 'apps_uninstall' do
+    it 'requires client_id' do
+      expect { client.apps_uninstall(client_secret: %q[]) }.to raise_error ArgumentError, /Required arguments :client_id missing/
+    end
+    it 'requires client_secret' do
+      expect { client.apps_uninstall(client_id: %q[]) }.to raise_error ArgumentError, /Required arguments :client_secret missing/
+    end
+  end
+end

data/spec/slack/web/api/endpoints/{apps_permissions_resources_spec.rb → auth_teams_spec.rb}

@@ -1,7 +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::AppsPermissionsResources do
+RSpec.describe Slack::Web::Api::Endpoints::AuthTeams do
   let(:client) { Slack::Web::Client.new }
 end

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

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Bookmarks do
+  let(:client) { Slack::Web::Client.new }
+  context 'bookmarks_add' do
+    it 'requires channel_id' do
+      expect { client.bookmarks_add(title: %q[], type: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
+    end
+    it 'requires title' do
+      expect { client.bookmarks_add(channel_id: %q[], type: %q[]) }.to raise_error ArgumentError, /Required arguments :title missing/
+    end
+    it 'requires type' do
+      expect { client.bookmarks_add(channel_id: %q[], title: %q[]) }.to raise_error ArgumentError, /Required arguments :type missing/
+    end
+  end
+  context 'bookmarks_edit' do
+    it 'requires bookmark_id' do
+      expect { client.bookmarks_edit(channel_id: %q[]) }.to raise_error ArgumentError, /Required arguments :bookmark_id missing/
+    end
+    it 'requires channel_id' do
+      expect { client.bookmarks_edit(bookmark_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
+    end
+  end
+  context 'bookmarks_list' do
+    it 'requires channel_id' do
+      expect { client.bookmarks_list }.to raise_error ArgumentError, /Required arguments :channel_id missing/
+    end
+  end
+  context 'bookmarks_remove' do
+    it 'requires bookmark_id' do
+      expect { client.bookmarks_remove(channel_id: %q[]) }.to raise_error ArgumentError, /Required arguments :bookmark_id missing/
+    end
+    it 'requires channel_id' do
+      expect { client.bookmarks_remove(bookmark_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
+    end
+  end
+end

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

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::CallsParticipants do
+  let(:client) { Slack::Web::Client.new }
+  context 'calls.participants_add' do
+    it 'requires id' do
+      expect { client.calls_participants_add(users: %q[]) }.to raise_error ArgumentError, /Required arguments :id missing/
+    end
+    it 'requires users' do
+      expect { client.calls_participants_add(id: %q[]) }.to raise_error ArgumentError, /Required arguments :users missing/
+    end
+  end
+  context 'calls.participants_remove' do
+    it 'requires id' do
+      expect { client.calls_participants_remove(users: %q[]) }.to raise_error ArgumentError, /Required arguments :id missing/
+    end
+    it 'requires users' do
+      expect { client.calls_participants_remove(id: %q[]) }.to raise_error ArgumentError, /Required arguments :users missing/
+    end
+  end
+end

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: %q[]) }.to raise_error ArgumentError, /Required arguments :external_unique_id missing/
+    end
+    it 'requires join_url' do
+      expect { client.calls_add(external_unique_id: %q[]) }.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

@@ -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::ChatScheduledmessages do
+  let(:client) { Slack::Web::Client.new }
+end

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

@@ -1,25 +1,27 @@
+# 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
-        client.auth_test
-      rescue Slack::Web::Api::Errors::TooManyRequestsError => e
-        expect(e.message).to eq('Retry after 3600 seconds')
-        expect(e.retry_after).to eq(3600)
-      end
+      client.auth_test
+    rescue Slack::Web::Api::Errors::TooManyRequestsError => e
+      expect(e.message).to eq('Retry after 3600 seconds')
+      expect(e.retry_after).to eq(3600)
     end
   end
 end

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

@@ -1,108 +1,212 @@
+# 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
 
-    it 'automatically converts attachments into JSON' do
+    it 'automatically converts attachments and blocks into JSON' do
       expect(client).to receive(:post).with(
         'chat.postEphemeral',
+        {
+          channel: 'channel',
+          text: 'text',
+          user: '123',
+          attachments: '[]',
+          blocks: '[]'
+        }
+      )
+      client.chat_postEphemeral(
         channel: 'channel',
         text: 'text',
         user: '123',
-        attachments: '[]'
+        attachments: [],
+        blocks: []
       )
-      client.chat_postEphemeral(channel: 'channel', text: 'text', user: '123', attachments: [])
     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 or :attachments 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 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 do
+          client.chat_postEphemeral(channel: 'channel', blocks: [], user: '123')
+        end.not_to raise_error
       end
     end
   end
 
   context 'chat_postMessage' do
-    it 'automatically converts attachments into JSON' do
+    it 'automatically converts attachments and blocks into JSON' do
       expect(client).to receive(:post).with(
         'chat.postMessage',
-        channel: 'channel',
-        text: 'text',
-        attachments: '[]'
+        {
+          channel: 'channel',
+          text: 'text',
+          attachments: '[]',
+          blocks: '[]'
+        }
       )
-      client.chat_postMessage(channel: 'channel', text: 'text', attachments: [])
+      client.chat_postMessage(channel: 'channel', text: 'text', attachments: [], blocks: [])
     end
-    context 'text and attachment arguments' do
-      it 'requires text or attachments' do
-        expect { client.chat_postMessage(channel: 'channel') }.to raise_error ArgumentError, /Required arguments :text or :attachments missing/
+
+    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/)
+        )
       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 'both text and attachments' do
-        expect(client).to receive(:post).with('chat.postMessage', hash_including(text: 'text', attachments: '[]'))
-        expect { client.chat_postMessage(channel: 'channel', text: 'text', attachments: []) }.to_not raise_error
+
+      it 'only blocks' do
+        expect(client).to receive(:post).with('chat.postMessage', hash_including(blocks: '[]'))
+        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 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 into JSON' do
+
+    it 'automatically converts attachments and blocks into JSON' do
       expect(client).to receive(:post).with(
         'chat.update',
-        attachments: '[]',
-        channel: 'channel',
-        text: 'text',
-        ts: ts
+        {
+          channel: 'channel',
+          text: 'text',
+          ts: ts,
+          attachments: '[]',
+          blocks: '[]'
+        }
       )
-      client.chat_update(attachments: [], channel: 'channel', text: 'text', ts: ts)
+      client.chat_update(channel: 'channel', text: 'text', ts: ts, attachments: [], blocks: [])
     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 and attachment arguments' do
-      it 'requires text or attachments' do
-        expect { client.chat_update(channel: 'channel', ts: ts) }.to raise_error ArgumentError, /Required arguments :text or :attachments missing/
+
+    context 'text, attachment and blocks arguments' do
+      it 'requires text, attachments, blocks or reply_broadcast' do
+        expect { client.chat_update(channel: 'channel', ts: ts) }.to(
+          raise_error(ArgumentError, /Required arguments :text, :attachments, :blocks or :reply_broadcast 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(attachments: [], channel: 'channel', ts: ts) }.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 do
+          client.chat_update(channel: 'channel', ts: ts, blocks: [])
+        end.not_to raise_error
+      end
+
+      it 'only reply_broadcast' do
+        expect(client).to receive(:post).with('chat.update', hash_including(reply_broadcast: true))
+        expect do
+          client.chat_update(channel: 'channel', ts: ts, reply_broadcast: true)
+        end.not_to raise_error
       end
-      it 'both text and attachments' do
-        expect(client).to receive(:post).with('chat.update', hash_including(text: 'text', attachments: '[]'))
-        expect { client.chat_update(attachments: [], channel: 'channel', text: 'text', ts: ts) }.to_not raise_error
+
+      it 'all text, attachments and blocks' do
+        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/conversations_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Conversations do
+  let(:client) { Slack::Web::Client.new }
+
+  context 'groups' do
+    it 'info', vcr: { cassette_name: 'web/conversations_info' } do
+      json = client.conversations_info(channel: '#mpdm-dblock--rubybot--player1-1')
+      expect(json.channel.name).to eq 'mpdm-dblock--rubybot--player1-1'
+    end
+  end
+end

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

@@ -1,27 +1,39 @@
+# 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(
         'dialog.open',
-        trigger_id: '12345.98765.abcd2358fdea',
-        dialog: '[]'
+        {
+          trigger_id: '12345.98765.abcd2358fdea',
+          dialog: '[]'
+        }
       )
       client.dialog_open(trigger_id: '12345.98765.abcd2358fdea', dialog: [])
     end
 
     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