diff --git a/app/services/gpo_reminder_sender.rb b/app/services/gpo_reminder_sender.rb index b8eabb9a1eb..8c84058b241 100644 --- a/app/services/gpo_reminder_sender.rb +++ b/app/services/gpo_reminder_sender.rb @@ -7,6 +7,7 @@ def send_emails(for_letters_sent_before) reminder_eligible_range = IdentityConfig.store.usps_confirmation_max_days.days.ago..for_letters_sent_before profiles_due_for_reminder(for_letters_sent_before).each do |profile| + next if profile.user.active_profile profile.gpo_confirmation_codes.all.each do |gpo_code| next if gpo_code.reminder_sent_at next unless reminder_eligible_range.cover?(gpo_code.created_at) diff --git a/spec/jobs/gpo_reminder_job_spec.rb b/spec/jobs/gpo_reminder_job_spec.rb index 156614819e5..4a193b5df23 100644 --- a/spec/jobs/gpo_reminder_job_spec.rb +++ b/spec/jobs/gpo_reminder_job_spec.rb @@ -62,5 +62,15 @@ user_id: due_for_reminder_user.uuid, ) end + + context 'when the user has another active profile' do + let!(:active_profile) do + create(:profile, :active, user: due_for_reminder_user) + end + + it 'does not send an email' do + expect { perform }.not_to change { ActionMailer::Base.deliveries.count } + end + end end end