Merge pull request #137 from sahilas/main

Added filename in send_document
ignacio-chiazzo · May 1, 2024 · e7b413f · e7b413f
2 parents ad98efe + 3399da6
commit e7b413f
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -333,6 +333,8 @@ messages_api.send_document(
 )
 ```
 
+Note, you can specify the filename via file [`filename` argument](https://developers.facebook.com/docs/whatsapp/cloud-api/reference/messages). 
+
 **Send a sticker message**
 It could use a link or a sticker_id.
 

diff --git a/lib/whatsapp_sdk/api/messages.rb b/lib/whatsapp_sdk/api/messages.rb
@@ -236,11 +236,11 @@ def send_video(
         params(
           sender_id: Integer, recipient_number: Integer,
           document_id: T.nilable(String), link: T.nilable(String), caption: String,
-          message_id: T.nilable(String)
+          message_id: T.nilable(String), filename: T.nilable(String)
         ).returns(Api::Response)
       end
       def send_document(
-        sender_id:, recipient_number:, document_id: nil, link: nil, caption: "", message_id: nil
+        sender_id:, recipient_number:, document_id: nil, link: nil, caption: "", message_id: nil, filename: nil
       )
         if !document_id && !link
           raise Resource::Errors::MissingArgumentError,
@@ -258,6 +258,7 @@ def send_document(
                             else
                               { id: document_id, caption: caption }
                             end
+        params[:document] = params[:document].merge({ filename: filename }) if filename
         params[:context] = { message_id: message_id } if message_id
 
         response = send_request(

diff --git a/test/whatsapp/api/messages_test.rb b/test/whatsapp/api/messages_test.rb
@@ -373,6 +373,29 @@ def test_send_document_message_with_a_link
         assert_predicate(message_response, :ok?)
       end
 
+      def test_send_document_message_with_file_name
+        @messages_api.expects(:send_request).with(
+          endpoint: "123123/messages",
+          params: {
+            messaging_product: "whatsapp",
+            to: 56_789,
+            recipient_type: "individual",
+            type: "document",
+            document: { link: document_link, caption: "Ignacio Chiazzo Profile" }
+            filename: "custom_name"
+          },
+          headers: { "Content-Type" => "application/json" }
+        ).returns(valid_response(valid_contacts, valid_messages))
+
+        message_response = @messages_api.send_document(
+          sender_id: 123_123, recipient_number: 56_789,
+          link: document_link, caption: "Ignacio Chiazzo Profile", filename: "custom_name"
+        )
+
+        assert_mock_response(valid_contacts, valid_messages, message_response)
+        assert_predicate(message_response, :ok?)
+      end
+
       def test_send_document_message_with_a_document_id
         document_id = "12_345"
         @messages_api.expects(:send_request).with(