Merge branch 'develop' into fix/6120-whatsapp-multiple-images

app/services/whatsapp/providers/whatsapp_cloud_service.rb

@@ -1,7 +1,10 @@
 class Whatsapp::Providers::WhatsappCloudService < Whatsapp::Providers::BaseService
+  ATTACHMENT_TYPES = %w[image audio video].freeze
+  MESSAGE_CONTENT = %w[audio sticker].freeze
+
   def send_message(phone_number, message)
     if message.attachments.present?
-      send_attachment_message(phone_number, message)
+      process_attachments(phone_number, message)
     else
       send_text_message(phone_number, message)
     end
@@ -66,14 +69,20 @@ class Whatsapp::Providers::WhatsappCloudService < Whatsapp::Providers::BaseServi
     process_response(response)
   end
 
-  def send_attachment_message(phone_number, message)
-    attachment = message.attachments.first
-    type = %w[image audio video].include?(attachment.file_type) ? attachment.file_type : 'document'
-    type_content = {
-      'link': attachment.download_url
-    }
-    type_content['caption'] = message.content unless %w[audio sticker].include?(type)
+  def process_attachments(phone_number, message)
+    responses = []
+    message.attachments.each do |attachment|
+      responses << send_attachment_message(phone_number, message, attachment)
+    end
+    responses
+  end
+
+  def send_attachment_message(phone_number, message, attachment)
+    type = self.class::ATTACHMENT_TYPES.include?(attachment.file_type) ? attachment.file_type : 'document'
+    type_content = { 'link': attachment.download_url }
+    type_content['caption'] = message.content unless self.class::MESSAGE_CONTENT.include?(type)
     type_content['filename'] = attachment.file.filename if type == 'document'
+
     response = HTTParty.post(
       "#{phone_id_path}/messages",
       headers: api_headers,

spec/services/whatsapp/providers/whatsapp_cloud_service_spec.rb

@@ -42,7 +42,7 @@ describe Whatsapp::Providers::WhatsappCloudService do
                                  })
           )
           .to_return(status: 200, body: whatsapp_response.to_json, headers: response_headers)
-        expect(service.send_message('+123456789', message)).to eq 'message_id'
+        expect(service.send_message('+123456789', message)).to eq ['message_id']
       end
 
       it 'calls message endpoints for document attachment message messages' do
@@ -61,7 +61,7 @@ describe Whatsapp::Providers::WhatsappCloudService do
                                  })
           )
           .to_return(status: 200, body: whatsapp_response.to_json, headers: response_headers)
-        expect(service.send_message('+123456789', message)).to eq 'message_id'
+        expect(service.send_message('+123456789', message)).to eq ['message_id']
       end
     end
   end