Update Messages API implementation (#308)

* Minor usage improvements to Messages API implementation

.github/workflows/ci.yml

@@ -32,8 +32,7 @@ jobs:
       uses: ruby/setup-ruby@v1
       with:
         ruby-version: ${{ matrix.ruby }}
-    - name: Install dependencies
-      run: bundle install
+        bundler-cache: true
     - name: Run tests
       run: bundle exec rake test
       env:

CHANGES.md

@@ -1,6 +1,7 @@
 # 7.24.0
 
 * Updating Video API functionality with methods for Live Captions, Audio Connector, Experience Composer, and a `publisheronly` cleint token role. [#307](https://github.com/Vonage/vonage-ruby-sdk/pull/307)
+* Updating Messages API implementation to add a delegator for the `Message` class, and enforce some required args in `Messaging#send`. [#308](https://github.com/Vonage/vonage-ruby-sdk/pull/308)
 
 # 7.23.0
 

lib/vonage/messaging.rb

@@ -1,16 +1,21 @@
 # typed: true
 # frozen_string_literal: true
+require 'forwardable'
 
 module Vonage
   class Messaging < Namespace
+    extend Forwardable
+
     self.authentication = BearerToken
 
     self.request_body = JSON
 
+    def_delegators Message, *Message::CHANNELS.keys
+
     # Send a Message.
     #
     # @example
-    #   message = Vonage::Messaging::Message.sms(message: "Hello world!")
+    #   message = client.messaging.sms(message: "Hello world!")
     #   response = client.messaging.send(to: "447700900000", from: "447700900001", **message)
     #
     # @option params [required, String] :to
@@ -22,8 +27,8 @@ class Messaging < Namespace
     #
     # @see https://developer.vonage.com/api/messages-olympus#SendMessage
     #
-    def send(params)
-      request('/v1/messages', params: params, type: Post)
+    def send(to:, from:, **message)
+      request('/v1/messages', params: {to: to, from: from, **message}, type: Post)
     end
 
     # Validate a JSON Web Token from a Messages API Webhook.

test/vonage/messaging_test.rb

@@ -10,6 +10,15 @@ def messaging_uri
     'https://api.nexmo.com/v1/messages'
   end
 
+  def test_valid_message_delegator
+    message = Vonage::Messaging::Message.sms(message: "Hello world!")
+    assert_equal message, messaging.sms(message: "Hello world!")
+  end
+
+  def test_invalid_message_delegator
+    assert_raises { messaging.invalid }
+  end
+
   def test_send_method
     params = {
       to: "447700900000",
@@ -26,6 +35,14 @@ def test_send_method
     assert_kind_of Vonage::Response, messaging.send(to: "447700900000", from: "447700900001", **message)
   end
 
+  def test_send_method_without_to
+    assert_raises(ArgumentError) { messaging.send(from: "447700900001", message: "Hello world!") }
+  end
+
+  def test_send_method_without_from
+    assert_raises(ArgumentError) { messaging.send(to: "447700900000", message: "Hello world!") }
+  end
+
   def test_verify_webhook_token_method_with_valid_secret_passed_in
     verification = messaging.verify_webhook_token(token: sample_webhook_token, signature_secret: sample_valid_signature_secret)