slack-ruby-client 0.12.0 → 0.14.6

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

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

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'
@@ -9,4 +10,9 @@ RSpec.describe Slack::Web::Api::Endpoints::Dnd do
       expect { client.dnd_setSnooze }.to raise_error ArgumentError, /Required arguments :num_minutes missing/
     end
   end
+  context 'dnd_teamInfo' do
+    it 'requires users' do
+      expect { client.dnd_teamInfo }.to raise_error ArgumentError, /Required arguments :users missing/
+    end
+  end
 end

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

@@ -1,17 +1,10 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::FilesComments do
   let(:client) { Slack::Web::Client.new }
-  context 'files.comments_add' do
-    it 'requires comment' do
-      expect { client.files_comments_add(file: 'F1234467890') }.to raise_error ArgumentError, /Required arguments :comment missing/
-    end
-    it 'requires file' do
-      expect { client.files_comments_add(comment: 'Everyone should take a moment to read this file.') }.to raise_error ArgumentError, /Required arguments :file missing/
-    end
-  end
   context 'files.comments_delete' do
     it 'requires file' do
       expect { client.files_comments_delete(id: 'Fc1234567890') }.to raise_error ArgumentError, /Required arguments :file missing/
@@ -20,15 +13,4 @@ RSpec.describe Slack::Web::Api::Endpoints::FilesComments do
       expect { client.files_comments_delete(file: 'F1234567890') }.to raise_error ArgumentError, /Required arguments :id missing/
     end
   end
-  context 'files.comments_edit' do
-    it 'requires comment' do
-      expect { client.files_comments_edit(file: 'F1234567890', id: 'Fc1234567890') }.to raise_error ArgumentError, /Required arguments :comment missing/
-    end
-    it 'requires file' do
-      expect { client.files_comments_edit(comment: 'Everyone should take a moment to read this file, seriously.', id: 'Fc1234567890') }.to raise_error ArgumentError, /Required arguments :file missing/
-    end
-    it 'requires id' do
-      expect { client.files_comments_edit(comment: 'Everyone should take a moment to read this file, seriously.', file: 'F1234567890') }.to raise_error ArgumentError, /Required arguments :id missing/
-    end
-  end
 end

data/spec/slack/web/api/endpoints/files_remote_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::FilesRemote do
+  let(:client) { Slack::Web::Client.new }
+  context 'files.remote_add' do
+    it 'requires external_id' do
+      expect { client.files_remote_add(external_url: 'http://example.com/my_cloud_service_file/abc123', title: 'Danger, High Voltage!') }.to raise_error ArgumentError, /Required arguments :external_id missing/
+    end
+    it 'requires external_url' do
+      expect { client.files_remote_add(external_id: '123456', title: 'Danger, High Voltage!') }.to raise_error ArgumentError, /Required arguments :external_url missing/
+    end
+    it 'requires title' do
+      expect { client.files_remote_add(external_id: '123456', external_url: 'http://example.com/my_cloud_service_file/abc123') }.to raise_error ArgumentError, /Required arguments :title missing/
+    end
+  end
+  context 'files.remote_share' do
+    it 'requires channels' do
+      expect { client.files_remote_share }.to raise_error ArgumentError, /Required arguments :channels missing/
+    end
+  end
+end

data/spec/slack/web/api/endpoints/files_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/im_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/migration_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/mpim_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/oauth_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/oauth_v2_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::OauthV2 do
+  let(:client) { Slack::Web::Client.new }
+  context 'oauth.v2_access' do
+    it 'requires code' do
+      expect { client.oauth_v2_access }.to raise_error ArgumentError, /Required arguments :code missing/
+    end
+  end
+end

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

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'
@@ -6,7 +7,10 @@ RSpec.describe Slack::Web::Api::Endpoints::Pins do
   let(:client) { Slack::Web::Client.new }
   context 'pins_add' do
     it 'requires channel' do
-      expect { client.pins_add }.to raise_error ArgumentError, /Required arguments :channel missing/
+      expect { client.pins_add(timestamp: '1234567890.123456') }.to raise_error ArgumentError, /Required arguments :channel missing/
+    end
+    it 'requires timestamp' do
+      expect { client.pins_add(channel: 'C1234567890') }.to raise_error ArgumentError, /Required arguments :timestamp missing/
     end
   end
   context 'pins_list' do

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

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This file was auto-generated by lib/tasks/web.rake
 
 require 'spec_helper'
@@ -5,8 +6,14 @@ require 'spec_helper'
 RSpec.describe Slack::Web::Api::Endpoints::Reactions do
   let(:client) { Slack::Web::Client.new }
   context 'reactions_add' do
+    it 'requires channel' do
+      expect { client.reactions_add(name: 'thumbsup', timestamp: '1234567890.123456') }.to raise_error ArgumentError, /Required arguments :channel missing/
+    end
     it 'requires name' do
-      expect { client.reactions_add }.to raise_error ArgumentError, /Required arguments :name missing/
+      expect { client.reactions_add(channel: 'C1234567890', timestamp: '1234567890.123456') }.to raise_error ArgumentError, /Required arguments :name missing/
+    end
+    it 'requires timestamp' do
+      expect { client.reactions_add(channel: 'C1234567890', name: 'thumbsup') }.to raise_error ArgumentError, /Required arguments :timestamp missing/
     end
   end
   context 'reactions_remove' do

data/spec/slack/web/api/endpoints/reminders_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/rtm_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/search_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/stars_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/team_profile_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/team_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/usergroups_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/usergroups_users_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/users_admin_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/users_prefs_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/users_profile_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/views_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+# This file was auto-generated by lib/tasks/web.rake
+
+require 'spec_helper'
+
+RSpec.describe Slack::Web::Api::Endpoints::Views do
+  let(:client) { Slack::Web::Client.new }
+  context 'views_open' do
+    it 'requires trigger_id' do
+      expect { client.views_open(view: ' ') }.to raise_error ArgumentError, /Required arguments :trigger_id missing/
+    end
+    it 'requires view' do
+      expect { client.views_open(trigger_id: '12345.98765.abcd2358fdea') }.to raise_error ArgumentError, /Required arguments :view missing/
+    end
+  end
+  context 'views_push' do
+    it 'requires trigger_id' do
+      expect { client.views_push(view: ' ') }.to raise_error ArgumentError, /Required arguments :trigger_id missing/
+    end
+    it 'requires view' do
+      expect { client.views_push(trigger_id: '12345.98765.abcd2358fdea') }.to raise_error ArgumentError, /Required arguments :view missing/
+    end
+  end
+  context 'views_update' do
+    it 'requires view' do
+      expect { client.views_update }.to raise_error ArgumentError, /Required arguments :view missing/
+    end
+  end
+end

data/spec/slack/web/api/error_spec.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Error do
   let(:client) { Slack::Web::Client.new }
+
   it 'provides access to the response object', vcr: { cassette_name: 'web/auth_test_error' } do
     begin
       client.auth_test
       raise 'Expected to receive Slack::Web::Api::Error.'
-    rescue Slack::Web::Api::Error => e
-      expect(e.response).to_not be_nil
+    rescue described_class => e
+      expect(e.response).not_to be_nil
       expect(e.response.status).to eq 200
     end
   end