Provide default value when actions are missing in payload (#9)

Author: nijave

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 #### 0.3.1 (Next)
 
+* [#9](https://github.com/slack-ruby/slack-ruby-bot-server-events/pull/9): Provide default value when actions are missing in payload - [@nijave](https://github.com/nijave).
 * Your contribution here.
 
 #### 0.3.0 (12/30/2020)

lib/slack-ruby-bot-server-events/api/endpoints/slack/actions_endpoint.rb

@@ -73,7 +73,7 @@ class ActionsEndpoint < Grape::API
               action = SlackRubyBotServer::Events::Requests::Action.new(params, request)
               payload_type = params[:payload][:type]
               callback_id = params[:payload][:callback_id]
-              action_ids = params[:payload][:actions]&.map { |entity| entity[:action_id] }
+              action_ids = params[:payload].fetch(:actions, []).map { |entity| entity[:action_id] }
               SlackRubyBotServer::Events.config.run_callbacks(
                 :action,
                 ([payload_type, callback_id] + action_ids).compact,

spec/slack-ruby-bot-server-events/api/endpoints/slack/actions_endpoint_spec.rb

@@ -22,7 +22,6 @@
       let(:payload) do
         {
           type: 'message_action',
-          actions: [{ name: 'id', value: '43749' }],
           channel: { id: 'C12345', name: 'channel' },
           user: { id: 'user_id' },
           team: { id: 'team_id' },
@@ -31,6 +30,15 @@
         }
       end
 
+      shared_examples 'message_actions handler' do
+        it 'performs action' do
+          post '/api/slack/action', payload: payload.to_json
+          expect(last_response.status).to eq 201
+          response = JSON.parse(last_response.body)
+          expect(response).to eq('text' => 'message_action/action_id')
+        end
+      end
+
       context 'with an action handler' do
         before do
           SlackRubyBotServer::Events.configure do |config|
@@ -40,11 +48,14 @@
           end
         end
 
-        it 'performs action' do
-          post '/api/slack/action', payload: payload.to_json
-          expect(last_response.status).to eq 201
-          response = JSON.parse(last_response.body)
-          expect(response).to eq('text' => 'message_action/action_id')
+        it_behaves_like 'message_actions handler'
+
+        context 'with actions in the payload' do
+          before do
+            payload.merge!(actions: [{ name: 'id', value: '43749' }])
+          end
+
+          it_behaves_like 'message_actions handler'
         end
       end