Add specs for mail delivery

Author: bradgessler

lib/supermail.rb

@@ -8,13 +8,25 @@ module Supermail
   class Error < StandardError; end
 
   module Rails
+    # This is a bizzare work around for a commit that broke https://github.com/rails/rails/commit/c594ba4ffdb016c7b2a22055f41dfb2c4409594d
+    # further proving the bewildering maze of indirection in Rails ActionMailer.
+    class Mailer < ActionMailer::Base
+      def mail(...)
+        super(...)
+      end
+
+      def self.message_delivery(**)
+        ActionMailer::MessageDelivery.new self, :mail, **
+      end
+    end
+
     class Base
       delegate \
           :deliver,
           :deliver_now,
           :deliver_later,
           :message,
-        to: :action_mailer_message_delivery
+        to: :message_delivery
 
       def to = nil
       def from = nil
@@ -27,10 +39,8 @@ def body = ""
       def mailto = MailTo.href(to:, from:, cc:, bcc:, subject:, body:)
       alias :mail_to :mailto
 
-      # This is a bizzare work around for a commit that broke https://github.com/rails/rails/commit/c594ba4ffdb016c7b2a22055f41dfb2c4409594d
-      # further proving the bewildering maze of indirection in Rails ActionMailer.
-      private def action_mailer_message_delivery
-        ActionMailer::MessageDelivery.new(ActionMailer::Base, :mail,
+      private def message_delivery
+        Rails::Mailer.message_delivery(
           to:,
           from:,
           cc:,

spec/supermail_spec.rb

@@ -58,4 +58,36 @@ def initialize(to:, from:, subject:, body:, cc: [], bcc: [])
       expect(result).to include("bcc=%5B%22bcc%40example.com%22%5D")
     end
   end
+
+  describe "delivery" do
+    before do
+      ActionMailer::Base.delivery_method = :test
+      ActionMailer::Base.deliveries.clear
+      ActiveJob::Base.queue_adapter = :test
+    end
+
+    describe "#deliver_now" do
+      it "delivers the email through ActionMailer" do
+        expect {
+          email.deliver_now
+        }.to change { ActionMailer::Base.deliveries.count }.by(1)
+      end
+    end
+
+    describe "#deliver" do
+      it "delivers the email through ActionMailer" do
+        expect {
+          email.deliver
+        }.to change { ActionMailer::Base.deliveries.count }.by(1)
+      end
+    end
+
+    describe "#deliver_later" do
+      it "enqueues the email for delivery through ActiveJob" do
+        expect {
+          email.deliver_later
+        }.to change { ActiveJob::Base.queue_adapter.enqueued_jobs.size }.by(1)
+      end
+    end
+  end
 end