feat(dunning): Permit creation of payment request without email

app/mailers/payment_request_mailer.rb

@@ -6,6 +6,10 @@ class PaymentRequestMailer < ApplicationMailer
   def requested
     @payment_request = params[:payment_request]
     @organization = @payment_request.organization
+
+    return if @payment_request.email.blank?
+    return if @organization.email.blank?
+
     @customer = @payment_request.customer
     @invoices = @payment_request.invoices
     @payment_url = ::PaymentRequests::Payments::GeneratePaymentUrlService.call(payable: @payment_request).payment_url

app/models/payment_request.rb

@@ -10,7 +10,6 @@ class PaymentRequest < ApplicationRecord
   belongs_to :organization
   belongs_to :customer, -> { with_discarded }
 
-  validates :email, presence: true
   validates :amount_cents, presence: true
   validates :amount_currency, presence: true
 

spec/mailers/payment_request_mailer_spec.rb

@@ -8,7 +8,7 @@
   let(:organization) { create(:organization, document_number_prefix: 'ORG-123B') }
   let(:first_invoice) { create(:invoice, total_amount_cents: 1000, organization:) }
   let(:second_invoice) { create(:invoice, total_amount_cents: 2000, organization:) }
-  let(:payment_request) { create(:payment_request, invoices: [first_invoice, second_invoice]) }
+  let(:payment_request) { create(:payment_request, organization:, invoices: [first_invoice, second_invoice]) }
 
   before do
     first_invoice.file.attach(
@@ -57,6 +57,24 @@
         .with(payable: payment_request)
     end
 
+    context "when payment request email is nil" do
+      before { payment_request.update(email: nil) }
+
+      it "returns a mailer with nil values" do
+        mailer = payment_request_mailer.with(payment_request:).requested
+        expect(mailer.to).to be_nil
+      end
+    end
+
+    context "when organization email is nil" do
+      before { organization.update(email: nil) }
+
+      it "returns a mailer with nil values" do
+        mailer = payment_request_mailer.with(payment_request:).requested
+        expect(mailer.to).to be_nil
+      end
+    end
+
     context "when no payment url is available" do
       let(:payment_url_result) do
         BaseService::Result.new.tap do |result|

spec/models/payment_request_spec.rb

@@ -31,11 +31,6 @@
       expect(payment_request).to be_valid
     end
 
-    it "is not valid without email" do
-      payment_request.email = nil
-      expect(payment_request).not_to be_valid
-    end
-
     it "is not valid without amount_cents" do
       payment_request.amount_cents = nil
       expect(payment_request).not_to be_valid