claimant does not receive rejection if no email

app/controllers/admin/decisions_controller.rb

@@ -85,7 +85,10 @@ def send_claim_result_email
     return if @claim.awaiting_qa?
 
     ClaimMailer.approved(@claim).deliver_later if @claim.latest_decision.result == "approved"
-    ClaimMailer.rejected(@claim).deliver_later if @claim.latest_decision.result == "rejected"
+
+    if @claim.latest_decision.result == "rejected" && @claim.email_address.present?
+      ClaimMailer.rejected(@claim).deliver_later
+    end
 
     if @claim.latest_decision.result == "rejected" && @claim.has_early_years_policy?
       ClaimMailer.rejected_provider_notification(@claim).deliver_later

spec/features/admin/admin_reject_claim_spec.rb

@@ -110,25 +110,51 @@
   end
 
   context "early years claim" do
-    let!(:claim) do
-      create(
-        :claim,
-        :submitted,
-        policy: Policies::EarlyYearsPayments
-      )
+    context "claimant has not completed their half" do
+      let!(:claim) do
+        create(
+          :claim,
+          :submitted,
+          policy: Policies::EarlyYearsPayments,
+          email_address: nil
+        )
+      end
+
+      scenario "rejecting sends email to provider only when claimant yet to complete" do
+        visit admin_claim_tasks_path(claim)
+        click_on "Approve or reject this claim"
+        choose "Reject"
+        check "Claim cancelled by employer"
+
+        expect {
+          click_button "Confirm decision"
+        }.to change { enqueued_jobs.count { |job| job[:job] == ActionMailer::MailDeliveryJob } }.by(1)
+
+        expect(page).to have_content("Claim has been rejected successfully")
+      end
     end
 
-    scenario "rejecting sends email to claimant + provider" do
-      visit admin_claim_tasks_path(claim)
-      click_on "Approve or reject this claim"
-      choose "Reject"
-      check "Claim cancelled by employer"
-
-      expect {
-        click_button "Confirm decision"
-      }.to change { enqueued_jobs.count { |job| job[:job] == ActionMailer::MailDeliveryJob } }.by(2)
-
-      expect(page).to have_content("Claim has been rejected successfully")
+    context "claimant has completed their half" do
+      let!(:claim) do
+        create(
+          :claim,
+          :submitted,
+          policy: Policies::EarlyYearsPayments
+        )
+      end
+
+      scenario "rejecting sends email to claimant + provider" do
+        visit admin_claim_tasks_path(claim)
+        click_on "Approve or reject this claim"
+        choose "Reject"
+        check "Claim cancelled by employer"
+
+        expect {
+          click_button "Confirm decision"
+        }.to change { enqueued_jobs.count { |job| job[:job] == ActionMailer::MailDeliveryJob } }.by(2)
+
+        expect(page).to have_content("Claim has been rejected successfully")
+      end
     end
   end
 end