Include reply-to, cc + bcc in logging output (#20)

Author: eelcoj

lib/courrier/email/providers/logger.rb

@@ -21,9 +21,7 @@ def format_email_using(options)
           <<~EMAIL
             #{separator}
             Timestamp: #{Time.now.strftime("%Y-%m-%d %H:%M:%S %z")}
-            From:      #{@options.from}
-            To:        #{@options.to}
-            Subject:   #{@options.subject}
+            #{meta_fields(from: options)}
 
             Text:
             #{@options.text || "(empty)"}
@@ -35,6 +33,27 @@ def format_email_using(options)
         end
 
         def separator = "-" * 80
+
+        def meta_fields(from:)
+          fields = [
+            [:from, "From"],
+            [:to, "To"],
+            [:reply_to, "Reply-To"],
+            [:cc, "Cc"],
+            [:bcc, "Bcc"],
+            [:subject, "Subject"]
+          ]
+
+          fields.map do |field, label|
+            value = from.send(field)
+
+            next if value.nil? || value.to_s.strip.empty?
+
+            "#{label}:".ljust(11) + value
+          rescue NoMatchingPatternError
+            nil
+          end.compact.join("\n")
+        end
       end
     end
   end

test/courrier/email/providers/logger_test.rb

@@ -7,7 +7,8 @@ class LoggerTest < Minitest::Test
     def test_formats_email_for_logging
       email = TestEmail.new(
         from: "[email protected]",
-        to: "[email protected]"
+        to: "[email protected]",
+        reply_to: "[email protected]"
       )
       logger = Logger.new(options: email.options)
 
@@ -20,6 +21,23 @@ def test_formats_email_for_logging
       assert_includes output, "HTML:\n<p>Test HTML Body</p>"
     end
 
+    def test_formats_optional_meta_fields_for_logging
+      email = TestEmail.new(
+        from: "[email protected]",
+        to: "[email protected]",
+        reply_to: "[email protected]",
+        cc: "[email protected]",
+        bcc: "[email protected]"
+      )
+      logger = Logger.new(options: email.options)
+
+      output = capture_logger_output { logger.deliver }
+
+      assert_includes output, "Reply-To:  [email protected]"
+      assert_includes output, "Cc:        [email protected]"
+      assert_includes output, "Bcc:       [email protected]"
+    end
+
     private
 
     def capture_logger_output