views_spec.rb in slack-ruby-client-2.0.0

- old
+ new

@@ -2,10 +2,11 @@
 require 'spec_helper'
 
 RSpec.describe Slack::Web::Api::Endpoints::Views do
   let(:client) { Slack::Web::Client.new }
   let(:trigger_id) { '12345.98765.abcd2358fdea' }
+  let(:view_id) { 'abc123567' }
 
   describe 'views_open' do
     context 'with a hash for view' do
       let(:view_str) { '{"celery":"man"}' }
 
@@ -84,20 +85,28 @@
   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' })
+        expect(client).to receive(:post).with('views.update', { view_id: view_id, view: view_str })
+        client.views_update(view_id: view_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.update', { view: view_str })
-        client.views_update(view: 'celery man')
+        expect(client).to receive(:post).with('views.update', { view_id: view_id, view: view_str })
+        client.views_update(view_id: view_id, view: 'celery man')
+      end
+
+      context 'with both an external_id and view_id' do
+        it 'raises error' do
+          expect do
+            client.views_update(external_id: trigger_id, view_id: view_id, view: 'celery man')
+          end.to raise_error ArgumentError, 'One of :external_id, :view_id is required'
+        end
       end
     end
   end
 end