From 484c9709b67685c95de351a39e3dfb140acd3681 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Fri, 5 May 2017 14:56:00 -0400 Subject: Misc spec coverage improvements (#2821) * Dont use raise_error by itself (avoids warning) * Add coverage for AccountFilter * Improve coverage and refactor for Subscription#lease_seconds * Improve coverage and refactor for NotificationMailer * Simplify assignment of min/max threshold on subscription --- spec/mailers/notification_mailer_spec.rb | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) (limited to 'spec/mailers/notification_mailer_spec.rb') diff --git a/spec/mailers/notification_mailer_spec.rb b/spec/mailers/notification_mailer_spec.rb index 3beaebeb1..faa3197c8 100644 --- a/spec/mailers/notification_mailer_spec.rb +++ b/spec/mailers/notification_mailer_spec.rb @@ -62,4 +62,34 @@ RSpec.describe NotificationMailer, type: :mailer do end end + describe 'follow_request' do + let(:follow_request) { Fabricate(:follow_request, account: sender, target_account: receiver.account) } + let(:mail) { NotificationMailer.follow_request(receiver.account, Notification.create!(account: receiver.account, activity: follow_request)) } + + it 'renders the headers' do + expect(mail.subject).to eq('Pending follower: bob') + expect(mail.to).to eq([receiver.email]) + end + + it 'renders the body' do + expect(mail.body.encoded).to match("bob has requested to follow you") + end + end + + describe 'digest' do + before do + mention = Fabricate(:mention, account: receiver.account) + Fabricate(:notification, account: receiver.account, activity: mention) + end + let(:mail) { NotificationMailer.digest(receiver.account, since: 5.days.ago) } + + it 'renders the headers' do + expect(mail.subject).to match('notification since your last') + expect(mail.to).to eq([receiver.email]) + end + + it 'renders the body' do + expect(mail.body.encoded).to match('brief summary') + end + end end -- cgit