Updating the email delivery method

lib/noticed/delivery_methods/email.rb

@@ -5,9 +5,9 @@ class Email < Base
 
       def deliver
         if options[:enqueue]
-          mailer.with(format).send(mailer_method.to_sym).deliver_later
+          composed_mailer.deliver_later
         else
-          mailer.with(format).send(mailer_method.to_sym).deliver_now
+          composed_mailer.deliver_now
         end
       end
 
@@ -50,6 +50,20 @@ def format
         end
         params.merge(recipient: recipient, record: record)
       end
+
+      def args
+        return unless (option = options[:arguments])
+
+        notification.send(option)
+      end
+
+      def composed_mailer
+        if options[:arguments]
+          mailer.with(params).public_send(mailer_method.to_sym, args)
+        else
+          mailer.with(params).public_send(mailer_method.to_sym)
+        end
+      end
     end
   end
 end

test/delivery_methods/email_test.rb

@@ -8,6 +8,14 @@ class EmailDeliveryWithActiveJob < Noticed::Base
   deliver_by :email, mailer: "UserMailer", enqueue: true, method: "comment_notification"
 end
 
+class EmailDeliveryWithArguments < Noticed::Base
+  deliver_by :email, mailer: "UserMailer", method: :comment_notification_for, arguments: :email_arguments
+
+  def email_arguments
+    params[:recipient]
+  end
+end
+
 class EmailTest < ActiveSupport::TestCase
   setup do
     @user = users(:one)
@@ -43,4 +51,10 @@ class EmailTest < ActiveSupport::TestCase
     EmailDeliveryWithActiveJob.new.deliver(user)
     assert_enqueued_emails 1
   end
+
+  test "delivers an email when passing an argument" do
+    assert_emails 1 do
+      EmailDeliveryWithArguments.new.deliver(user)
+    end
+  end
 end

test/dummy/app/mailers/user_mailer.rb

@@ -2,4 +2,8 @@ class UserMailer < ApplicationMailer
   def comment_notification
     mail(body: "")
   end
+
+  def comment_notification_for(user)
+    mail(body: "")
+  end
 end