Skip to content

Commit 2d1d4f3

Browse files
committed
refactor(tests): convert eligibility inquiry email test to delivery assertions
Replaces spy-based mock test with email delivery verification. Changes: - 'sends an eligibility check email' - verifies email delivery count and recipient Removes allow().to receive() and have_received() mocks
1 parent f90baf2 commit 2d1d4f3

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

spec/models/eligibility_inquiry_spec.rb

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,14 @@
99
expect(eligibility_inquiry.issued_by).to eq(admin)
1010
end
1111

12-
it 'sends an attendance warning email' do
13-
allow(MemberMailer).to receive(:eligibility_check).with(member, member.email).and_call_original
12+
it 'sends an eligibility check email' do
13+
expect {
14+
described_class.create(member: member, issued_by: admin)
15+
}.to change { ActionMailer::Base.deliveries.count }.by(1)
1416

15-
described_class.create(member: member, issued_by: admin)
16-
17-
expect(MemberMailer).to have_received(:eligibility_check).with(member, member.email)
17+
email = ActionMailer::Base.deliveries.last
18+
expect(email.to).to include(member.email)
19+
expect(email.subject).to include('Eligibility')
1820
end
1921
end
2022
end

0 commit comments

Comments
 (0)