Improve newsletter model spec list_of_recipients
This commit is contained in:
@@ -48,15 +48,19 @@ describe Newsletter do
|
||||
end
|
||||
|
||||
describe '#list_of_recipients' do
|
||||
let(:erased_user) { create(:user, username: 'erased_user') }
|
||||
|
||||
before do
|
||||
create(:user, newsletter: true, username: 'newsletter_user')
|
||||
create(:user, newsletter: false)
|
||||
erased_user.erase
|
||||
newsletter.update(segment_recipient: 'all_users')
|
||||
end
|
||||
|
||||
it 'returns list of recipients excluding users with disabled newsletter' do
|
||||
expect(newsletter.list_of_recipients.count).to eq(1)
|
||||
expect(newsletter.list_of_recipients.first.username).to eq('newsletter_user')
|
||||
expect(newsletter.list_of_recipients).not_to include(erased_user)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user